HomePhabricator

EPIC: Remove the Revision class entirely

Description

EPIC: Remove the Revision class entirely

Following soft deprecation in 1.31, hard deprecation
in 1.35, and removal of remaining deprecated uses,
the entire Revision class, and the tests for it, can
now be removed. This will be followed by a patch to
clean up old references in comments - this is just
dealing with the code.

There were some other tests in RevisionDbTest.php
for Title::getNextRevisionID and Title::getRelativeRevisionID,
which could have been kept, but since those methods are
tested separately in TitleTest, no need to keep these, so
just remove the entire file.

Also remove the RevisionTestModifyableContent and
RevisionTestModifyableContentHandler classes that were
only loaded for use in RevisionDbTest - not sure how
the stable interface policy applies to classes that
are only loaded within tests, so mention that in the
release notes as a breaking change.

Bug: T247143
Change-Id: I05a297d28b85c7413384979f5c908be318098d9d

Details

Provenance
DannyS712Authored on May 2 2021, 10:26 PM
Parents
rMWb36f5a4175c9: Localisation updates from https://translatewiki.net.
Branches
Unknown
Tags
Unknown
ChangeId
I05a297d28b85c7413384979f5c908be318098d9d