Page MenuHomePhabricator

Avoid using User ::addGroup, ::removeGroup, ::getAllGroups, ::clearNotification
Closed, DuplicatePublic

Description

User::addGroup, User::removeGroup, User::getAllGroups, User::clearNotification were soft deprecated and now need to be hard deprecated. So we need to remove using from the extensions:

  • Echo
  • LdapAuthentication
  • Wikibase
  • CentralAuth
  • AbuseFilter
  • Translate
  • Flow
  • UserMerge
  • MassMessage
  • GlobalCssJs
  • BetaFeatures

Event Timeline

Change 667602 had a related patch set uploaded (by Daimona Eaytoy; owner: Daimona Eaytoy):
[mediawiki/extensions/AbuseFilter@master] Stop using deprecated User::addGroup in tests

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

Change 667615 had a related patch set uploaded (by Vlad.shapik; owner: Vlad.shapik):
[mediawiki/extensions/Echo@master] Avoid using User::addGroup

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

Change 667632 had a related patch set uploaded (by Vlad.shapik; owner: Vlad.shapik):
[mediawiki/extensions/LdapAuthentication@master] Avoid using User ::addGroup, ::removeGroup, ::getAllGroups

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

Change 667615 merged by jenkins-bot:
[mediawiki/extensions/Echo@master] Avoid using User::addGroup

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

Change 667602 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Stop using deprecated User::addGroup in tests

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

There's already a subtask for group-related methods: T275148

Change 667632 merged by jenkins-bot:
[mediawiki/extensions/LdapAuthentication@master] Avoid using User ::addGroup, ::removeGroup, ::getAllGroups

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

Change 669132 had a related patch set uploaded (by Artrix; owner: Artrix):
[mediawiki/extensions/MassMessage@master] Avoid using User::addGroup

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

Change 669153 had a related patch set uploaded (by Artrix; owner: Artrix):
[mediawiki/extensions/Translate@master] Avoid using User::addGroup and User::removeGroup

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

Change 670929 had a related patch set uploaded (by Artrix; owner: Artrix):
[mediawiki/extensions/BetaFeatures@master] Avoid using User::addGroup() method

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

Change 669153 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Avoid using User::addGroup and User::removeGroup

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

ArTrix updated the task description. (Show Details)

Change 669132 abandoned by Ppchelko:
[mediawiki/extensions/MassMessage@master] Avoid using User::addGroup

Reason:
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/MassMessage/ /673719

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