Page MenuHomePhabricator

In translatable page move reason: “Part of translatable page XXX”, XXX is wrong
Closed, ResolvedPublic1 Estimated Story PointsBUG REPORT

Description

Many thanks for fixing T333786, however, that introduces a small bug:
Pages in Translations: namespace and translation pages contain the reason “Part of translatable page XXX”. Previously, XXX was correctly the source page name (example).

Reproduction steps:

  • Prepare and mark the English page X for translation.
  • Move/Rename X to Y.
  • Review history of
    • Y/en
    • Translations:Y/1/en

What happens?
Log entry:

FuzzyBot moved page X/en to Y/en without leaving a redirect: Part of translatable page "X/en"
FuzzyBot moved page Translations:X/1/en to Translations:Y/1/en without leaving a redirect: Part of translatable page "Translations:X/1/en"

Example

What should have happened instead?
The (previous or current) source page name should still be mentioned as the translatable page. By the way, it should be linked.
Log entry expected:

FuzzyBot moved page X/en to Y/en without leaving a redirect: Part of translatable page "X"
FuzzyBot moved page Translations:X/1/en to Translations:Y/1/en without leaving a redirect: Part of translatable page "X"

Event Timeline

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

[mediawiki/extensions/Translate@master] Log correct old title in move reason for translation pages and units

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

abi_ triaged this task as Medium priority.
abi_ set the point value for this task to 1.

Change 932837 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Log correct old title in move reason for translation pages and units

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

Change 934648 had a related patch set uploaded (by Pols12; author: Pols12):

[mediawiki/extensions/Translate@master] Link old title in logged move reason for translation pages and units

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

Change 934648 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Link old title in logged move reason for translation pages and units

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