Part of #2047951: #2047951: [META] Remove calls to deprecated global $user and $GLOBALS['user']

CommentFileSizeAuthor
#3 node-page-remove-global-user-2062243-3.patch518 bytesm1r1k
#2 2062243-remove-global-user.patch609 bytesjohnmcc
#1 2062243-remove-global-user.patch497 byteschertzog
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chertzog’s picture

chertzog
he/him
Primary language English
Location Harrisburg, PA, USA
CreditAttribution: chertzog commented
Status: Active » Needs review
FileSize
2062243-remove-global-user.patch497 bytes
johnmcc’s picture

johnmcc CreditAttribution: johnmcc commented
FileSize
2062243-remove-global-user.patch609 bytes

Looks good to me, but the patch in #1 won't apply for me. I re-rolled the patch to apply to the latest dev version.

m1r1k’s picture

m1r1k CreditAttribution: m1r1k commented
FileSize
node-page-remove-global-user-2062243-3.patch518 bytes

Reroll after #2062151: Create a current user service to ensure that current account is always available

johnmcc’s picture

johnmcc CreditAttribution: johnmcc commented

Looks good and applies cleanly.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Assigned: Maxis » Unassigned
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix

This one is ready to go in.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.