Problem/Motivation
Several code comments refer to \Drupal\Update\UpdateManager::getProjects()
. The correct namespace is \Drupal\update
(lower case "update", referring to the Update module).
These comments seem to have been added in #2449809: Remove usage of update_get_projects().
We briefly made the same mistake in a use
statement (not a comment) in #3015810: Properly deprecate UPDATE_* constants, but that was promptly corrected.
Proposed resolution
Search for \Drupal\Update
and correct the capitalization. You can use the following command to search:
grep -r '\\Drupal\\Update' core
Remaining tasks
User interface changes
None
API changes
None
Data model changes
None
Comment | File | Size | Author |
---|---|---|---|
#2 | 3100071-2.patch | 8.11 KB | ravi.shankar |
Comments
Comment #2
ravi.shankar CreditAttribution: ravi.shankar at OpenSense Labs commentedHere is a patch for this issue please review.
Comment #3
ravi.shankar CreditAttribution: ravi.shankar at OpenSense Labs commentedComment #4
dwwPatch looks good.
After applying patch, the codebase is free of references to
Drupal\Update
.All changes in the patch are in comments, so bot results are meaningless. ;)
RTBC.
Thanks!
-Derek
Comment #5
alexpottCrediting @benjifisher for creating this issue and @dww for reviewing.
Backporting to 8.8.x as a docs only fix.
Committed and pushed 04e5bbbc98 to 9.0.x and c8c2b9632b to 8.9.x and 32dc816fcc to 8.8.x. Thanks!