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.
Part of #2047951: [META] Remove calls to deprecated global $user and $GLOBALS['user']
replace
function update_access_allowed() {
global $user;
Comment | File | Size | Author |
---|---|---|---|
#9 | 2061949-9-global-user-depricated-update.patch | 478 bytes | joelpittet |
#4 | updatephp-remove-global-user-from-update-php-2061949-4.patch | 897 bytes | m1r1k |
#2 | updatephp-remove-global-user-from-update-php-2061949-2.patch | 477 bytes | m1r1k |
#1 | 2061949-1.patch | 728 bytes | Garbar |
Comments
Comment #1
Garbar CreditAttribution: Garbar commentedFix. Needs review.
Comment #2
m1r1k CreditAttribution: m1r1k commentedRerolling after #2062151: Create a current user service to ensure that current account is always available:
Comment #4
m1r1k CreditAttribution: m1r1k commentedAdd this patch #2076411: Remove the request scope from the current user service:
Comment #5
m1r1k CreditAttribution: m1r1k commentedPushed to retesting
Comment #6
m1r1k CreditAttribution: m1r1k commentedComment #7
catchComment #8
joelpittet#2: updatephp-remove-global-user-from-update-php-2061949-2.patch queued for re-testing.
Comment #9
joelpittetI'm RTBCing #2 the patch attached is the same just with a \ for the namespace. @see #2053489: Standardize on \Drupal throughout core
Comment #10
joelpittetretitling because the issue summary is meant for the update.php not the update module.
Comment #11
alexpottCommitted a18f9c6 and pushed to 8.x. Thanks!