Page MenuHomePhabricator

Add functionality to move Message bundles
Closed, ResolvedPublic8 Estimated Story Points

Description

Add functionality to rename Message Bundle pages. The related translations should also be renamed.

Details

SubjectRepoBranchLines +/-
mediawiki/extensions/Translatemaster+108 -89
mediawiki/extensions/Translatemaster+107 -29
mediawiki/extensions/Translatemaster+14 -3
mediawiki/extensions/Translatemaster+171 -130
mediawiki/extensions/Translatemaster+357 -108
mediawiki/extensions/Translatemaster+18 -23
mediawiki/extensions/Translatemaster+121 -100
mediawiki/extensions/Translatemaster+155 -75
mediawiki/extensions/Translatemaster+300 -37
mediawiki/extensions/Translatemaster+296 -197
mediawiki/extensions/Translatemaster+84 -101
mediawiki/extensions/Translatemaster+62 -5
mediawiki/extensions/Translatemaster+67 -21
mediawiki/extensions/Translatemaster+56 -23
mediawiki/extensions/Translatemaster+66 -39
mediawiki/extensions/Translatemaster+27 -15
mediawiki/extensions/Translatemaster+4 -4
mediawiki/extensions/Translatemaster+8 -6
mediawiki/extensions/Translatemaster+27 -27
mediawiki/extensions/Translatemaster+134 -56
mediawiki/extensions/Translatemaster+24 -3
Show related patches Customize query in gerrit

Event Timeline

abi_ changed the task status from Open to In Progress.Feb 3 2022, 3:21 PM
abi_ claimed this task.

Change 759515 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add support to move message bundle

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

Change 760915 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Rename TranslatablePageMove to TranslatableBundleMover

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

Change 760916 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Rename TranslatablePageMoveJob to TranslatableBundleMoveJob

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

Nikerabbit set the point value for this task to 8.Feb 9 2022, 2:15 PM

Change 761623 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add TranslatableBundle interface

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

Change 761962 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] TranslatablePage: Move functions used during move to a separate file

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

Change 762399 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add MessageBundle

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

Change 762528 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add service to cache TranslatableBundles

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

Change 759515 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@master] Add support to move message bundle

Reason:

Subsequent patches implement this in a different manner.

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

Change 760917 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Rename MoveTranslatablePageSpecialPage to MoveTranslatableBundleSpecialPage

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

Change 760918 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Rename moveTranslatablePage script to moveTranslatableBundle

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

Change 767056 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add service to cache TranslatableBundles

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

Change 762528 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@master] Add service to cache TranslatableBundles

Reason:

Reordered: Ieb9f358b2b64b2e599d58125802cbfeaae83c216

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

Change 760915 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Rename TranslatablePageMove to TranslatableBundleMover

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

Change 760916 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Rename TranslatablePageMoveJob to TranslatableBundleMoveJob

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

Change 760917 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Rename MoveTranslatablePageSpecialPage to MoveTranslatableBundleSpecialPage

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

Change 760918 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Rename moveTranslatablePage script to moveTranslatableBundle

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

Change 769275 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add TranslatableBundle interface

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

Change 761623 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@master] Add TranslatableBundle interface

Reason:

Abandoned in favor of: Ief3f72a901c07075188d4a7ef9b25ed1209b1bd1

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

Change 769385 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add TranslatableBundleFactory

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

Change 769408 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add PageMoveLogger

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

Change 769698 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add interface TranslatableBundleStore and TranslatablePageStore

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

Change 770450 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add method to fetch subpages grouped by type

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

Change 769275 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add TranslatableBundle interface

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

Change 769385 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add TranslatableBundleFactory

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

Change 769408 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add PageMoveLogger

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

Change 772397 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add MessageBundle

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

Change 772841 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Rename PageTranslationLogFormatter to TranslatableBundleLogFormatter

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

Change 772842 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Move methods to manage revtags to TranslatbaleBundle

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

Change 774011 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add SubpageListBuilder

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

Change 777743 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Add RevTagStore

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

Change 772842 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@master] Move methods to manage revtags to TranslatableBundleStore

Reason:

In favor of Ia84e22c0aa96f7dc9ba11be9f5711e1ac89256a4

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

Change 761962 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@master] TranslatablePage: Move methods used during move operation to services

Reason:

Took another approach to handle this.

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

Change 762399 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@master] Add MessageBundle class for moving message bundles.

Reason:

Took another approach to handle this.

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

Change 769698 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add interface TranslatableBundleStore and TranslatablePageStore

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

Change 770450 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add method to fetch subpages grouped by type

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

Change 777743 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add RevTagStore

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

Change 772397 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add support to move Message bundles

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

Change 772841 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Rename PageTranslationLogFormatter to TranslatableBundleLogFormatter

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

Change 774011 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Add SubpageListBuilder

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

Patches for this change have been deployed on Translatewiki. I've tested moving / deleting of translatable pages, and moving of message bundles.

I'm leaving this open to collect any other issues that might have been introduced.

I've tested this change on Metawiki. Works fine.

Change 793479 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] MessageBundle: Clear caches and rebuild index on move

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

Change 793479 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] MessageBundle: Clear caches and rebuild index on move

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

Change 767056 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@master] Add service to cache TranslatableBundles

Reason:

No longer needed.

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