Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#6 | uuid_dbselect_to_EFQ_in_getter_functions_2238157_6.patch | 4.26 KB | c_lehel |
Comments
Comment #1
bartram CreditAttribution: bartram commentedAttached is a patch to use EFQ rather than SQL.
Comment #2
skwashd CreditAttribution: skwashd commentedMarking as needs review so the test bot can check it.
Comment #4
skwashd CreditAttribution: skwashd commentedThanks 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.
This can be chained like so:
nitpick - should only be a single space here.
Comment #5
mihai_brb CreditAttribution: mihai_brb commentedWe 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
Comment #6
c_lehel CreditAttribution: c_lehel commentedWe reviewed the code, cleaned it up a bit. We also added fixes to the get_id_by_uuid function.