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

Remove all entries:
core/modules/user/lib/Drupal/user/Access/LoginStatusCheck.php: return (bool) $GLOBALS['user']->id();

CommentFileSizeAuthor
#1 user-LoginStatusCheck-remove-global-user-2078419-1.patch616 bytesm1r1k
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

m1r1k’s picture

m1r1k CreditAttribution: m1r1k commented
Status: Active » Needs review
FileSize
user-LoginStatusCheck-remove-global-user-2078419-1.patch616 bytes

Here is a patch

Status: Needs review » Needs work

The last submitted patch, user-LoginStatusCheck-remove-global-user-2078419-1.patch, failed testing.

m1r1k’s picture

m1r1k CreditAttribution: m1r1k commented
Status: Needs work » Postponed

That means current patch also depends on #2076411: Remove the request scope from the current user service

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Issue summary: View changes
Status: Postponed » Needs work
Related issues: +#2076411: Remove the request scope from the current user service

No longer a barrier.

m1r1k’s picture

m1r1k CreditAttribution: m1r1k commented
Assigned: m1r1k » Unassigned
Status: Needs work » Fixed
Issue tags: +Amsterdam 2014

Well, $_GLOBAL['user'] was removed already.

patrickd’s picture

patrickd CreditAttribution: patrickd commented
Issue tags: -Amsterdam 2014 +Amsterdam2014

correcting amsterdam tag (No space)

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

Excellent.. I was trying to clean up the issue queue. Postponed issues had been ignored.

Status: Fixed » Closed (fixed)

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