Page MenuHomePhabricator

Prepare User group methods for hard deprecation
Closed, ResolvedPublic

Description

The following methods in User are being replaced with UserGroupManager.

The goal of this task is to hard-deprecate the following methods:

  • public function getGroups()
  • public function addGroup()
  • public function removeGroup()
  • public function getGroupMemberships()
  • public function getEffectiveGroups() => Done
  • public function getAutomaticGroups() => Done
  • public function getFormerGroups() => Done
  • public function addAutopromoteOnceGroups() => Done
  • public static function getAllGroups()
  • public static function getImplicitGroups()

CodeSearch: https://codesearch.wmcloud.org/search/?q=%5Cb%28getGroups%7CaddGroup%7CremoveGroup%7CgetGroupMemberships%7CgetAllGroups%7CgetImplicitGroups%29%5Cb&files=%5C.php&excludeFiles=&repos=

Steps:

  1. use https://codesearch.wmcloud.org/search/ tool to find all the usages of these methods in extensions. Make a list, WMF deployed extensions separately, not WMF deployed extensions separately (you can select Wikimedia deployed) in the tool
  2. All usages of deprecated methods in the deployed extensions need to be replaced with corresponding UserGroupManager methods. You can obtain UserGroupManager via MediaWikiServices::getInstance()->getUserGroupManager(), but it's preferred to inject it if possible.
  3. Patches for all extensions that are not deployed should be created, but there's no requirement for them to get merged.

WMF deployed

  • extensions / Translate
  • extensions / Wikibase
  • extensions / CheckUser
  • extensions / SecurePoll
  • extensions / UserMerge
  • extensions / WikibaseMediaInfo
  • operations/mediawiki-config
  • extensions / AbuseFilter
  • extensions / EventBus
  • extensions / LdapAuthentication
  • extensions / OAuth
  • extensions / Flow
  • extensions / BetaFeatures
  • extensions / CentralAuth
  • extensions / GlobalCssJs
  • extensions / CentralNotice

not WMF deployed

  • extensions / LDAPGroups
  • extensions / WikiLexicalData
  • extensions / BlueSpiceFoundation
  • extensions / SimpleSAMLphp
  • extensions / AbsenteeLandlord
  • extensions / AccessControl
  • extensions / ApprovedRevs
  • extensions / BatchUserRights
  • extensions / BlueSpiceDashboards
  • extensions / BlueSpiceFlaggedRevsConnector
  • extensions / BlueSpicePageAccess
  • extensions / BlueSpiceUserManager
  • extensions / BlueSpiceSMWConnector
  • extensions / ConfirmAccount
  • extensions / Farmer
  • extensions / ImportUsers
  • extensions / LinkFilter
  • extensions / RevisionCommentSupplement
  • extensions / SocialProfile
  • extensions / WikiToLDAP
  • mediawiki / vagrant
  • skins / Cosmos
  • skins / Tweeki
  • extensions / Auth_remoteuser
  • extensions / GlobalNotice
  • extensions / GroupsSidebar
  • extensions / GroupWhitelist
  • extensions / HAWelcome
  • extensions / Lockdown
  • extensions / LockDownEnglishPages
  • extensions / PageForms
  • extensions / PrivateDomains
  • extensions / Quantcast
  • extensions / RightFunctions
  • extensions / SemanticACL
  • extensions / SemanticPageSeries
  • extensions / ShoutWikiAds
  • extensions / ShoutWikiAPI
  • extensions / StaffPowers
  • extensions / UserFunctions
  • extensions / WikiForum
  • extensions / WSOAuth
  • extensions / DisableAccount
  • extensions / InviteSignup
  • extensions / Shibboleth

Details

SubjectRepoBranchLines +/-
mediawiki/coremaster+27 -9
mediawiki/extensions/WikiLambdamaster+13 -3
mediawiki/extensions/UserMergemaster+1 -1
mediawiki/extensions/UserGroupsmaster+6 -3
mediawiki/extensions/AccessControlmaster+4 -2
mediawiki/extensions/LDAPGroupsREL1_39+3 -4
mediawiki/coremaster+37 -32
mediawiki/extensions/BlueSpiceEchoConnectormaster+2 -2
mediawiki/extensions/LDAPGroupsmaster+3 -4
mediawiki/extensions/WatchAnalyticsmaster+8 -1
mediawiki/extensions/GlobalUserrightsmaster+1 -1
mediawiki/coremaster+2 -1
mediawiki/extensions/TwitterLoginmaster+5 -1
mediawiki/extensions/PageAuthorsmaster+2 -1
mediawiki/extensions/MediaWikiChatmaster+13 -4
mediawiki/extensions/GroupsSidebarREL1_37+7 -2
mediawiki/extensions/EditAccountREL1_37+10 -2
mediawiki/extensions/SemanticPageSeriesmaster+5 -1
mediawiki/extensions/GroupsSidebarmaster+7 -2
mediawiki/extensions/EditAccountmaster+10 -2
mediawiki/extensions/HAWelcomemaster+3 -1
mediawiki/coremaster+98 -59
mediawiki/extensions/Flowmaster+31 -4
mediawiki/extensions/MassMessagemaster+5 -3
mediawiki/extensions/Translatemaster+35 -3
mediawiki/extensions/Echomaster+53 -29
mediawiki/extensions/Echomaster+40 -61
Show related patches Customize query in gerrit

Related Objects

StatusSubtypeAssignedTask
OpenNone
OpenNone
Resolved Pchelolo
ResolvedUmherirrender
ResolvedVlad.shapik
DuplicateNone
InvalidNone
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedZabe
ResolvedDaimona
InvalidVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
Resolvedjeena
InvalidNone
ResolvedVlad.shapik
ResolvedVlad.shapik
InvalidNone
ResolvedVlad.shapik
ResolvedVlad.shapik
DuplicateNone
ResolvedVlad.shapik
ResolvedNone
InvalidNone
InvalidVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedZabe
ResolvedVlad.shapik
ResolvedVlad.shapik
InvalidNone
ResolvedVlad.shapik
ResolvedVlad.shapik
Resolvedashley
ResolvedVlad.shapik
ResolvedNone
ResolvedUniversal_Omega
ResolvedVlad.shapik
ResolvedVlad.shapik
InvalidNone
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
InvalidNone
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedNone
ResolvedUmherirrender

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Change 727514 merged by jenkins-bot:

[mediawiki/extensions/GroupsSidebar@master] Replace deprecated User::getEffectiveGroups

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

Change 727530 merged by jenkins-bot:

[mediawiki/extensions/SemanticPageSeries@master] Replace deprecated User::getEffectiveGroups

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

Change 817343 had a related patch set uploaded (by Zabe; author: Umherirrender):

[mediawiki/extensions/EditAccount@REL1_37] Replace deprecated User::getEffectiveGroups

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

Change 817343 merged by jenkins-bot:

[mediawiki/extensions/EditAccount@REL1_37] Replace deprecated User::getEffectiveGroups

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

Change 849625 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/GroupsSidebar@REL1_37] Replace deprecated User::getEffectiveGroups

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

Change 849625 merged by jenkins-bot:

[mediawiki/extensions/GroupsSidebar@REL1_37] Replace deprecated User::getEffectiveGroups

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

Change 856060 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/TwitterLogin@master] Replace deprecated User::addGroup

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

Change 856061 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/PageAuthors@master] Replace deprecated User::getGroups

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

Change 856062 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/MediaWikiChat@master] Replace deprecated User::getGroups

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

Change 856063 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] SpecialUserrights: Replace deprecated User::get/add/removeGroups

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

Change 856062 merged by jenkins-bot:

[mediawiki/extensions/MediaWikiChat@master] Replace deprecated User::getGroups

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

Change 856061 merged by jenkins-bot:

[mediawiki/extensions/PageAuthors@master] Replace deprecated User::getGroups

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

Change 856060 merged by jenkins-bot:

[mediawiki/extensions/TwitterLogin@master] Replace deprecated User::addGroup

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

Change 870663 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] tests: Replace deprecated User::getAllGroups

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

Change 870687 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/WatchAnalytics@master] Replace deprecated User::getAllGroups

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

Change 870688 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/GlobalUserrights@master] Replace deprecated User::getAllGroups

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

Change 870689 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/LDAPGroups@master] Replace deprecated User::getAllGroups

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

Change 870690 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/BlueSpiceEchoConnector@master] Replace deprecated User::getAllGroups

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

Change 870663 merged by jenkins-bot:

[mediawiki/core@master] tests: Replace deprecated User::getAllGroups

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

Change 870688 merged by Umherirrender:

[mediawiki/extensions/GlobalUserrights@master] Replace deprecated User::getAllGroups

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

Change 870687 merged by Umherirrender:

[mediawiki/extensions/WatchAnalytics@master] Replace deprecated User::getAllGroups

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

Change 856063 merged by jenkins-bot:

[mediawiki/core@master] SpecialUserrights: Replace deprecated User::get/add/removeGroups

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

Change 870689 merged by jenkins-bot:

[mediawiki/extensions/LDAPGroups@master] Replace deprecated User::getAllGroups

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

Change 870690 merged by Umherirrender:

[mediawiki/extensions/BlueSpiceEchoConnector@master] Replace deprecated User::getAllGroups

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

Change 881038 had a related patch set uploaded (by Hslater; author: Umherirrender):

[mediawiki/extensions/LDAPGroups@REL1_39] Replace deprecated User::getAllGroups

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

Change 881038 merged by Hslater:

[mediawiki/extensions/LDAPGroups@REL1_39] Replace deprecated User::getAllGroups

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

Change 953331 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/AccessControl@master] Replace deprecated User::getAllGroups

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

Change 953331 merged by jenkins-bot:

[mediawiki/extensions/AccessControl@master] Replace deprecated User::getAllGroups

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

Change 953332 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/UserGroups@master] Replace deprecated User::getGroups/User::addGroup

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

Change 953334 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/UserMerge@master] Replace deprecatd User::getGroups

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

Change 953336 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/WikiLambda@master] Replace deprecated User::addGroup

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

Change 953337 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] user: Hard-deprecate User::getGroups/addGroup/removeGroup

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

Change 953336 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Replace deprecated User::addGroup

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

Change 953334 merged by jenkins-bot:

[mediawiki/extensions/UserMerge@master] Replace deprecated User::getGroups

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

Change 953332 merged by jenkins-bot:

[mediawiki/extensions/UserGroups@master] Replace deprecated User::getGroups/User::addGroup

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

Change 953337 merged by jenkins-bot:

[mediawiki/core@master] user: Hard-deprecate user group related User functions

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