Page MenuHomePhabricator

Hard-deprecate soft-deprecated User methods
Open, MediumPublic

Description

User class has 48 soft-deprecated methods. In order to replace it with UserIdentity and Authority where possible we need to start removing usages of these methods.

This is an umbrella task for that work.

Strategy:

  1. Select a batch of @deprecated methods, usually it's better to group methods by service where the implementation was moved.
  2. Find an existing ticket, or file a new one, about hard-deprecating those methods.
  3. Remove usages in deployed extensions and core, hard-deprecate methods. When removing usages use DI if possible. When User type hints or PHPDoc is used and there's no longer need for a full User - update the type hints to accept UserIdentity.
  4. Repeat from point 1.

Related Objects

StatusSubtypeAssignedTask
OpenNone
OpenVlad.shapik
ResolvedVlad.shapik
OpenVlad.shapik
ResolvedVlad.shapik
DuplicateNone
InvalidNone
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedZabe
ResolvedDaimona
InvalidVlad.shapik
OpenVlad.shapik
OpenVlad.shapik
Resolvedjeena
InvalidNone
OpenVlad.shapik
OpenVlad.shapik
InvalidNone
OpenNone
OpenVlad.shapik
DuplicateNone
OpenVlad.shapik
OpenVlad.shapik
InvalidNone
OpenVlad.shapik
OpenVlad.shapik
OpenVlad.shapik
OpenVlad.shapik
ResolvedZabe
OpenVlad.shapik
OpenVlad.shapik
InvalidNone
OpenVlad.shapik
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
ResolvedVlad.shapik
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
ResolvedVlad.shapik
ResolvedVlad.shapik
ResolvedVlad.shapik
DuplicateVlad.shapik
ResolvedVlad.shapik
OpenVlad.shapik
ResolvedVlad.shapik
ResolvedZabe
ResolvedZabe
OpenZabe
OpenZabe
OpenNone
InvalidNone

Event Timeline

Pchelolo triaged this task as Medium priority.Feb 9 2021, 3:50 AM

Change 671702 had a related patch set uploaded (by Zabe; owner: Zabe):
[mediawiki/core@master] Hard deprecate User::clearNotification()

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

Change 671702 merged by jenkins-bot:
[mediawiki/core@master] Hard deprecate User::clearNotification()

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

Change 672804 had a related patch set uploaded (by Zabe; owner: Zabe):
[mediawiki/core@master] Hard deprecate User::getFirstEditTimestamp()

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

Change 673272 had a related patch set uploaded (by Zabe; owner: Zabe):
[mediawiki/core@master] Hard deprecate User::isIPRange()

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

Change 673272 merged by jenkins-bot:
[mediawiki/core@master] Hard deprecate User::isIPRange()

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

Change 672804 merged by jenkins-bot:
[mediawiki/core@master] Hard deprecate User::getFirstEditTimestamp() and ::getLatestEditTimestamp()

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

Change 674959 had a related patch set uploaded (by Zabe; author: Zabe):
[mediawiki/core@master] Hard deprecate User::isValidUserName()

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

Change 674959 merged by jenkins-bot:
[mediawiki/core@master] Hard deprecate User::isValidUserName()

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

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

[mediawiki/core@master] Hard deprecated User::getRights()

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

Change 693179 merged by jenkins-bot:

[mediawiki/core@master] Hard deprecate User::getRights()

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

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

[mediawiki/core@master] Hard deprecate User::listOptionKinds()

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

Change 686498 merged by jenkins-bot:

[mediawiki/core@master] Hard deprecate User::listOptionKinds()

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