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.
Follow-up of #2389335: Deprecate entity.query service and replace with using the entity storage's getQuery() method. Can likely be done in parallel as the other issue adds nothing new.
Comment | File | Size | Author |
---|---|---|---|
#4 | entity-query-leftovers-2849873-4-interdiff.txt | 689 bytes | Berdir |
#4 | entity-query-leftovers-2849873-4.patch | 63.38 KB | Berdir |
#2 | entity-query-leftovers-2849873-2.patch | 63.06 KB | Berdir |
Comments
Comment #2
BerdirThis updates the entity query tests and a few remaining mentiones/uses of entity.query service.
I know that some won't like that I converted this to \Drupal::entityQuery() but that was the easiest to replace with I don't see the point of doing some kind of pseudo-injection as I mentioned multiple times in #2066993: Use \Drupal consistently in tests.
Comment #4
BerdirComment #6
BerdirComment #10
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedUnsurprisingly.. doesn't apply anymore :/
Comment #15
TR CreditAttribution: TR commentedI think this issue is obsolete now? #2968519: The entity.query service is essentially deprecated because it relies on \Drupal\Core\Entity\Query\QueryFactory which is and #3087546: Remove deprecated entity manager seem to have fixed this in Drupal 9.
Comment #16
BerdirYes.