Page MenuHomePhabricator

Remove CentralAuthUtils after removing all uses
Closed, ResolvedPublic

Description

T270828: CentralAuthUtils should use dependency injection added a proper service to replace static methods in CentralAuthUtils. Once all references to CentralAuthUtils have been removed, that class should be removed.

See codesearch

I made a rough estimate of all locations on MediaWiki.org: https://www.mediawiki.org/wiki/User:Majavah/CentralAuth#Replace_CentralAuthUtils_with_a_service

Event Timeline

taavi created this task.
taavi moved this task from Unsorted to Working on on the User-Majavah board.

@Majavah are you working on this?

taavi removed taavi as the assignee of this task.Dec 21 2021, 6:51 AM

@Majavah are you working on this?

Not actively, feel free to have a look if you want

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

[mediawiki/extensions/CentralAuth@master] Remove CentralAuthUtils and all remaining usages

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

Change 755022 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] Remove CentralAuthUtils and all remaining usages

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

Zabe claimed this task.