Page MenuHomePhabricator

Hard deprecate Revision::isMinor
Closed, ResolvedPublic

Description

Only remaining deployed use outside of core is in MobileFrontend's SpecialMobileHistory::showRow

  • Replace use in MobileFrontend
  • Replace core uses
  • Hard deprecate

Event Timeline

DannyS712 triaged this task as Medium priority.
DannyS712 moved this task from Unsorted to In progress on the User-DannyS712 board.

Change 589932 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/MobileFrontend@master] Convert SpecialMobileHistory::showRow to use RevisionRecord

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

@Pchelolo I've addressed the issues you raised on the patch - can you take another look?

Change 592320 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/core@master] Convert WikiPage::$mLastRevision to RevisionRecord

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

Change 592513 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/core@master] Add ContribsPager::tryCreatingRevisionRecord

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

Change 589932 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Convert SpecialMobileHistory::showRow to use RevisionRecord

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

Change 592513 merged by jenkins-bot:
[mediawiki/core@master] Add ContribsPager::tryCreatingRevisionRecord

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

Change 592320 merged by jenkins-bot:
[mediawiki/core@master] Convert WikiPage::$mLastRevision to RevisionRecord

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

Change 593620 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/core@master] Replace various uses of Revision objects

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

Change 593620 merged by jenkins-bot:
[mediawiki/core@master] Replace various uses of Revision objects

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

Remaining uses in core:

Core tests: RevisionStoreDbTestBase, RevisionDbTest, RevisionTest

Stalling - cannot proceed until after T250761: DifferenceEngine $mNewRev and $mOldRev are Revision objects is merged

DannyS712 changed the task status from Open to Stalled.May 1 2020, 6:23 PM

Change 598865 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/core@master] Hard deprecate Revision::getUserText and Revision::isMinor

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

DannyS712 changed the task status from Stalled to Open.May 26 2020, 10:13 PM

Change 598865 merged by jenkins-bot:
[mediawiki/core@master] Hard deprecate Revision::getUserText, ::isMinor, and ::isCurrent

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