Page MenuHomePhabricator

Phase out deprecated Title::isValidMoveTarget
Closed, ResolvedPublic

Description

Title::isValidMoveTarget has been deprecated since 1.25 in favor of MovePage, but is still used in a couple of places. All usages should be replaced, and the method should be hard-deprecated (should call wfDeprecated()), and scheduled for removal in 1.34.

Event Timeline

BPirkle added a subscriber: BPirkle.

Change 494789 had a related patch set uploaded (by Holger Knust; owner: Holger Knust):
[mediawiki/core@master] Hard-deprecation of Title::isValidMoveTarget

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

Change 494789 had a related patch set uploaded (by Holger Knust; owner: Holger Knust):
[mediawiki/core@master] Hard-deprecation of Title::isValidMoveTarget

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

Just to check, what I gathered from the Scrum of Scrums notice last week is that this is blocked on the removal of the UserMerge extension (T216089)?

Daimona added a subscriber: Daimona.

This was done.

Change 494789 abandoned by Thiemo Kreuz (WMDE):
[mediawiki/core@master] Hard-deprecation of Title::isValidMoveTarget

Reason:
Apparently the method does not exist any more since 1.35.

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