Page MenuHomePhabricator

Collection needs uses of global $wgUser removed
Closed, ResolvedPublic

Description

See parent task

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 561238 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Collection@master] Remove uses of global $wgUser

https://gerrit.wikimedia.org/r/561238

Change 561238 merged by jenkins-bot:
[mediawiki/extensions/Collection@master] Remove uses of global $wgUser and global $wgCommunityCollectionNamespace

https://gerrit.wikimedia.org/r/561238

Change 562398 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Collection@master] Remove use of global $wgUser in templates

https://gerrit.wikimedia.org/r/562398

I didn't think to search for that. Thanks for catching it. Patch provided to fix

Neither did I... I thought our codesniffer rules would actually forbid $GLOBALS, but well...

Change 562398 merged by jenkins-bot:
[mediawiki/extensions/Collection@master] Remove use of global $wgUser in templates

https://gerrit.wikimedia.org/r/562398