| Comment | File | Size | Author |
|---|---|---|---|
| #7 | editor-remove-global-user-from-editor-module-2062171-7.patch | 713 bytes | m1r1k |
| #1 | 2062171-1.patch | 851 bytes | Garbar |
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | editor-remove-global-user-from-editor-module-2062171-7.patch | 713 bytes | m1r1k |
| #1 | 2062171-1.patch | 851 bytes | Garbar |
Comments
Comment #1
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 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!