The following replacements should be made in all deployed extensions
After that, the methods in PermissionManager should be hard-deprecated.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • Pchelolo | T234921 Factor group membership management out of User class | |||
| Resolved | • Pchelolo | T252621 Move Autopromotion logic to UserGroupManager service | |||
| Resolved | • Pchelolo | T254537 UserGroupManager/PermissionManager cyclic dependency | |||
| Resolved | Peter.ovchyn | T271593 Replace specific PermissionManager calls with GroupPermissionsLookup calls in all deployed extensions |
I note, this is going to cause more potential backport problems for the LTS.
We haven't even fully migrated to PermissionManager I think... And when we backport to 1.31 currently, we need to remember to replace permission manager calls with applicable calls (User::isAllowed() is still in master, not hard deprecated either, soft deprecated since 1.34).
So when PermissionManager methods are hard deprecated, that means we're using GroupPermissionsLookup everywhere applicable... But then it means for future bug reports, and likely security patches... GroupPermissionsLookup changes will need changing for PermissionMananger, or possibly User::isAllowed() (though REL1_31 goes EOL in the middle of this year.
It may be worth considering to see what we can backport of GroupPermissionsLookup into 1.35, even if it ends up being a thin wrapper as a compat layer...
The 3 methods in question are much less widely used then isAllowed, about 50 usages altogether, but sure, we can back port GroupPermissionsLookup, it's rather small.
Change 656204 had a related patch set uploaded (by Peter.ovchyn; owner: Peter.ovchyn):
[mediawiki/extensions/CodeReview@master] Replace static User::getGroupsWithPermission in favour of GroupPermissionsLookup
Change 656904 had a related patch set uploaded (by Vlad.shapik; owner: Peter.ovchyn):
[mediawiki/extensions/CodeReview@master] Replace static User::getGroupsWithPermission in favour of GroupPermissionsLookup
Change 656904 abandoned by Vlad.shapik:
[mediawiki/extensions/CodeReview@master] Replace static User::getGroupsWithPermission in favour of GroupPermissionsLookup
Reason:
Change 656204 merged by jenkins-bot:
[mediawiki/extensions/CodeReview@master] Replace static User::getGroupsWithPermission in favour of GroupPermissionsLookup