entity_get_uuid_by_id fails on entities without a base table.

Marking as needs review so the test bot can check it.

Thanks for the patch. It looks like it needs some work before it is ready to be committed. In addition to the test bot not being happy, there is some other stuff to fix.

1. +++ b/uuid.entity.inc
   @@ -469,12 +469,21 @@ function entity_get_uuid_by_id($entity_type, $ids, $revision = FALSE) {
   +    $result = $query->execute();
   

   This can be chained like so:

   $query = new EntityFieldQuery();
   $result = $query->entityCondition(...)
     ->propertyCondition(...)
     ->execute();
   

2. +++ b/uuid.entity.inc
   @@ -469,12 +469,21 @@ function entity_get_uuid_by_id($entity_type, $ids, $revision = FALSE) {
   +      return array_map(function ($entity) use ($uuid_key) { return $entity->$uuid_key;  }, $entities);
   

   nitpick - should only be a single space here.

We have the same problem with Mongo Node entities. EFQ seems to solve some cases where there is no base table, however we need to load the entity to get the UUID.
Would it be a solution to provide a hook here? For instance, for all Mongo Node entities we could hook in and run our own query in mongoDB and get exactly what is needed, without loading the entities.

Thanks,
Mihai

We reviewed the code, cleaned it up a bit. We also added fixes to the get_id_by_uuid function.