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.
This function bypasses the entity API, so let's convert it to entity_query.
Comment | File | Size | Author |
---|---|---|---|
#6 | user-2024985-6.patch | 3.88 KB | tim.plunkett |
#1 | user-2024985-1.patch | 3.96 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettWhen converting admin/people to a view, it seems I left this in by mistake.
Comment #2
BerdirAs discussed in IRC, we'll still need EFQ support for the user roles, which I expected to be added here but as this is just dead code, let's remove it.
Comment #3
tim.plunkettComment #4
catch#1: user-2024985-1.patch queued for re-testing.
Comment #6
tim.plunkettThe code I was deleting was changed. It's still being 100% deleted.
Comment #7
catchThanks for the re-roll. Committed/pushed to 8.x, thanks!
Is there an issue open for EFQ roles support?
Comment #8
BerdirI guess we can use #1751274: Do not query user_roles directly for that?