Page MenuHomePhabricator
Create Task
Maniphest T281993

extensions/ConfirmEdit - use AbstractAuthenticationProvider ::init, ::postInitSetup instead of the setters of AbstractAuthenticationProvider
Closed, ResolvedPublic
Actions

Description

  1. Provided that the following methods are used only in tests, all usages of deprecated AbstractAuthenticationProvider ::setLogger, ::setManager, ::setConfig, ::setHookContainer methods must be replaced with AbstractAuthenticationProvider::init.
  2. If it's overridden AbstractAuthenticationProvider::setConfig or AbstractAuthenticationProvider ::setLogger, ::setManager, ::setConfig, ::setHookContainer are used outside the test codebase of the extension, so replace it with AbstractAuthenticationProvider::postInitSetup to do any necessary setup.

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
Replace uses of the deprecated setters of AbstractAuthenticationProvidermediawiki/extensions/ConfirmEditmaster+12 -10
Customize query in gerrit

Related Objects
Search...

Event Timeline

Vlad.shapik created this task.May 5 2021, 2:54 PM
Vlad.shapik triaged this task as Medium priority.
Vlad.shapik updated the task description. (Show Details)May 5 2021, 4:05 PM
Vlad.shapik updated the task description. (Show Details)
Vlad.shapik renamed this task from extensions/ConfirmEdit - hard deprecate the setters of AbstractAuthenticationProvider class to extensions/ConfirmEdit - use AbstractAuthenticationProvider ::init, ::postInitSetup instead of the setters of AbstractAuthenticationProvider.May 6 2021, 2:56 PM
gerritbot added a comment.May 6 2021, 4:25 PM

Change 685849 had a related patch set uploaded (by Vlad.shapik; author: Vlad.shapik):

[mediawiki/extensions/ConfirmEdit@master] Replace uses of the deprecated setters of AbstractAuthenticationProvider

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

gerritbot added a project: Patch-For-Review.May 6 2021, 4:25 PM
Vlad.shapik moved this task from Unsorted pile to Waiting for Review on the Platform Team Workboards (MW Expedition) board.May 7 2021, 11:57 AM
gerritbot added a comment.May 25 2021, 7:46 PM

Change 685849 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@master] Replace uses of the deprecated setters of AbstractAuthenticationProvider

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

ReleaseTaggerBot added a project: MW-1.37-notes (1.37.0-wmf.9; 2021-06-07).May 25 2021, 8:00 PM
Maintenance_bot removed a project: Patch-For-Review.May 25 2021, 8:11 PM
Vlad.shapik closed this task as Resolved.May 26 2021, 8:37 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits