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 class is from before the proper interfaces for that were finalized IIRC. I'd like to see the conversion happening here, but you're right, that's out of scope for this particular issue.
Comments
Comment #1
Garbar CreditAttribution: Garbar commentedneed reviews
Comment #2
Wim LeersGreat — thanks! :)
Comment #3
catchSorry folks postponing on #2062151: Create a current user service to ensure that current account is always available.
Comment #4
Wim LeersComment #5
Wim LeersGarbar, #2062151: Create a current user service to ensure that current account is always available was finished and committed two days ago. Would you like to pick this up again? :)
Comment #6
Wim Leers.
Comment #7
m1r1k CreditAttribution: m1r1k commentedRerolling after #2062151: Create a current user service to ensure that current account is always available:
Comment #8
andypostLooks nice to go
Not sure why the dependencies are not injected here but it was before the patch
Comment #9
Wim LeersThis class is from before the proper interfaces for that were finalized IIRC. I'd like to see the conversion happening here, but you're right, that's out of scope for this particular issue.
RTBC+1
Comment #10
alexpottCommitted 51811d8 and pushed to 8.x. Thanks!