This class introduces unhelpful indirection.
Description
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | None | T182943 Move namespaced extensions to use PSR-4 autoloading | |||
Open | None | T278278 Adopt PSR-4 namespaces in WMF-deployed extensions/skins | |||
Open | None | T377752 Test coverage of root directories missing | |||
Open | None | T193166 Adopt PSR-4 namespaces in Translate and modernize code | |||
Resolved | abi_ | T312007 Remove the RevTag class |
Event Timeline
Change 811320 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):
[mediawiki/extensions/Translate@master] Simplify the RevTag table
Change 811580 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):
[mediawiki/extensions/CentralNotice@master] Remove usage of Translate RevTag class
Change 819005 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):
[mediawiki/extensions/Translate@master] Remove RevTag class and add tag constants in RevTagStore
Change 811320 abandoned by Abijeet Patro:
[mediawiki/extensions/Translate@master] Simplify the RevTag class
Reason:
In favor of I45c22bc7622dc65455eec0ed4238b34b97c75693
Change 819005 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Replace usage of RevTag class with add tag constants in RevTagStore
Change 826127 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):
[mediawiki/extensions/Translate@master] Remove RevTag class
We've added some constants to the RevTagStore class to remove usage of the RevTag class from the Translate extension. This patch has been deployed.
The RevTag class is still used on the CentralNotice extension, the patch for which needs to be reviewed, merged, and deployed.
I've also submitted a separate patch to remove the class once the CentralNotice patch is deployed.
Change 811580 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@master] Remove usage of Translate RevTag class
Change 826127 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Remove RevTag class
Change 842900 had a related patch set uploaded (by MacFan4000; author: Abijeet Patro):
[mediawiki/extensions/CentralNotice@REL1_38] Remove usage of Translate RevTag class
Change 842900 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@REL1_38] Remove usage of Translate RevTag class