The way how SparqlEntityStorageGraphHandler::getEntityTypeGraphUris() is implemented violates SparqlEntityStorageGraphHandlerInterface::getEntityTypeGraphUris(): the underlying code assumes that EntityTypeBundleInfoInterface::getBundleInfo() returns a non-empty array, but in case of missing entity types (or not-yet-installed entity types for example) it returns an empty array.
This means the foreach loop inside the first condition's body won't create the corresponding entity type ID key in SparqlEntityStorageGraphHandler::$cache['definitions'], which means the storage handler will throw exception if it is asked to load a missing entity - but the contact with EntityStorageInterface::load() contraints to return a NULL in these cases.
1. Install RDF Entity.
2. Try load a missing entity with RDF::load('foo-bar-baz');
Ensure SparqlEntityStorageGraphHandler::getEntityTypeGraphUris() always returns array, e.g. $this->cache['structure'][$entity_type_id] gets always set inside the first condition in SparqlEntityStorageGraphHandler::getEntityTypeGraphUris().
A triage and a review.
Nothing.
Nothing.
Nothing.
Needs review
2.0
Code
First attempt. Only attaching patch files, no MRs this time.
The last submitted patch, 2: sparql_entity_storage-3379328-2-test-only.patch, failed testing. View results →