Missing i18n in FlaggedRevs
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Ladsgroup
	Jun 5 2019, 1:38 PM

Description

Error

Look at https://en.wikibooks.org/wiki/Special:PendingChanges or https://ca.wikinews.org/wiki/Plut%C3%B3_deixa_de_ser_planeta

Very likely caused by making FlaggedRevs use extension-wip.json

Details

	Subject	Repo	Branch	Lines +/-
	Fix missing i18n due to extension-wip.json	mediawiki/extensions/FlaggedRevs	master	+18 -0
	Fix missing i18n due to extension-wip.json	mediawiki/extensions/FlaggedRevs	wmf/1.34.0-wmf.8	+18 -0

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved	Release	zeljkofilipin	T220733 1.34.0-wmf.8 deployment blockers
		Resolved		Reedy	T225092 Missing i18n in FlaggedRevs

Event Timeline

Definitely a train blocker.

Restricted Application added a subscriber: Liuxinyu970226. · View Herald TranscriptJun 5 2019, 1:39 PM

Do we need to add $wgMessageDirs and $wgExtensionMessagesFiles to FlaggedRevs.php so they can be picked up when pulled via extension-list?

Very likely caused by making FlaggedRevs use extension-wip.json

Is/Was it fine on .7?

Oh, ignore that. I couldn't see the revert in .7 of the wip patch

@Ladsgroup will this be fixed by train window tomorrow?

zeljkofilipin moved this task from Backlog 🪒 to Deep work 🌊 on the User-zeljkofilipin board.Jun 5 2019, 1:48 PM

These tags are not suitable for subtask of train.

In T225092#5236804, @zeljkofilipin wrote:

@Ladsgroup will this be fixed by train window tomorrow?

I have no idea, I didn't touch those parts of the code. @Reedy definitely knows better but I think fixing them should not be that hard

$wgMessagesDirs['FlaggedRevs'] = __DIR__ . '/i18n/flaggedrevs';
$wgMessagesDirs['RevisionReview'] = __DIR__ . '/i18n/revisionreview';
$wgMessagesDirs['Stabilization'] = __DIR__ . '/i18n/stabilization';
$wgMessagesDirs['ReviewedVersions'] = __DIR__ . '/i18n/reviewedversions';
$wgMessagesDirs['UnreviewedPages'] = __DIR__ . '/i18n/unreviewedpages';
$wgMessagesDirs['PendingChanges'] = __DIR__ . '/i18n/pendingchanges';
$wgMessagesDirs['ProblemChanges'] = __DIR__ . '/i18n/problemchanges';
$wgMessagesDirs['ReviewedPages'] = __DIR__ . '/i18n/reviewedpages';
$wgMessagesDirs['StablePages'] = __DIR__ . '/i18n/stablepages';
$wgMessagesDirs['ConfiguredPages'] = __DIR__ . '/i18n/configuredpages';
$wgMessagesDirs['QualityOversight'] = __DIR__ . '/i18n/qualityoversight';
$wgMessagesDirs['ValidationStatistics'] = __DIR__ . '/i18n/validationstatistics';
$wgMessagesDirs['FlaggedRevsApi'] = __DIR__ . '/i18n/api';
$wgExtensionMessagesFiles['FlaggedRevsMagic'] = "$langDir/FlaggedRevs.i18n.magic.php";
$wgExtensionMessagesFiles['FlaggedRevsAliases'] = "$langDir/FlaggedRevs.alias.php";

Change 514475 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/FlaggedRevs@master] Fix missing i18n due to extension-wip.json

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

gerritbot added a project: Patch-For-Review.Jun 5 2019, 1:56 PM

In T225092#5236838, @Reedy wrote:

$wgMessagesDirs['FlaggedRevs'] = __DIR__ . '/i18n/flaggedrevs';
$wgMessagesDirs['RevisionReview'] = __DIR__ . '/i18n/revisionreview';
$wgMessagesDirs['Stabilization'] = __DIR__ . '/i18n/stabilization';
$wgMessagesDirs['ReviewedVersions'] = __DIR__ . '/i18n/reviewedversions';
$wgMessagesDirs['UnreviewedPages'] = __DIR__ . '/i18n/unreviewedpages';
$wgMessagesDirs['PendingChanges'] = __DIR__ . '/i18n/pendingchanges';
$wgMessagesDirs['ProblemChanges'] = __DIR__ . '/i18n/problemchanges';
$wgMessagesDirs['ReviewedPages'] = __DIR__ . '/i18n/reviewedpages';
$wgMessagesDirs['StablePages'] = __DIR__ . '/i18n/stablepages';
$wgMessagesDirs['ConfiguredPages'] = __DIR__ . '/i18n/configuredpages';
$wgMessagesDirs['QualityOversight'] = __DIR__ . '/i18n/qualityoversight';
$wgMessagesDirs['ValidationStatistics'] = __DIR__ . '/i18n/validationstatistics';
$wgMessagesDirs['FlaggedRevsApi'] = __DIR__ . '/i18n/api';
$wgExtensionMessagesFiles['FlaggedRevsMagic'] = "$langDir/FlaggedRevs.i18n.magic.php";
$wgExtensionMessagesFiles['FlaggedRevsAliases'] = "$langDir/FlaggedRevs.alias.php";

I can't unsee this. My eyes hurt now and I will have nightmares tonight...

zeljkofilipin added a project: FlaggedRevs.Jun 5 2019, 1:57 PM

Change 514493 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/FlaggedRevs@wmf/1.34.0-wmf.8] Fix missing i18n due to extension-wip.json

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

PeterBowman subscribed.Jun 5 2019, 2:38 PM

Change 514493 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@wmf/1.34.0-wmf.8] Fix missing i18n due to extension-wip.json

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

Jdforrester-WMF assigned this task to Reedy.Jun 5 2019, 3:06 PM

Scap is still running, but FR is now in ExtensionMessages-1.34.0-wmf.8.php (but wasn't before), so should have the desired effect

reedy@deploy1001:/srv/mediawiki-staging$ grep Flagged wmf-config/ExtensionMessages-1.34.0-wmf.8.php 
  'FlaggedRevsMagic' => "$IP/extensions/FlaggedRevs/frontend/language/FlaggedRevs.i18n.magic.php",
  'FlaggedRevsAliases' => "$IP/extensions/FlaggedRevs/frontend/language/FlaggedRevs.alias.php",
  'FlaggedRevs' => "$IP/extensions/FlaggedRevs/i18n/flaggedrevs",
  'RevisionReview' => "$IP/extensions/FlaggedRevs/i18n/revisionreview",
  'Stabilization' => "$IP/extensions/FlaggedRevs/i18n/stabilization",
  'ReviewedVersions' => "$IP/extensions/FlaggedRevs/i18n/reviewedversions",
  'UnreviewedPages' => "$IP/extensions/FlaggedRevs/i18n/unreviewedpages",
  'PendingChanges' => "$IP/extensions/FlaggedRevs/i18n/pendingchanges",
  'ProblemChanges' => "$IP/extensions/FlaggedRevs/i18n/problemchanges",
  'ReviewedPages' => "$IP/extensions/FlaggedRevs/i18n/reviewedpages",
  'StablePages' => "$IP/extensions/FlaggedRevs/i18n/stablepages",
  'ConfiguredPages' => "$IP/extensions/FlaggedRevs/i18n/configuredpages",
  'QualityOversight' => "$IP/extensions/FlaggedRevs/i18n/qualityoversight",
  'ValidationStatistics' => "$IP/extensions/FlaggedRevs/i18n/validationstatistics",
  'FlaggedRevsApi' => "$IP/extensions/FlaggedRevs/i18n/api",

Slightly relatedly... How this works for CirrusSearch without the same relevant code... https://github.com/wikimedia/mediawiki-extensions-CirrusSearch/blob/wmf/1.34.0-wmf.8/CirrusSearch.php - But whatever :P

Confirmed fixed

ReleaseTaggerBot added a project: MW-1.34-notes (1.34.0-wmf.8; 2019-06-04).Jun 5 2019, 4:00 PM

Change 514475 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Fix missing i18n due to extension-wip.json