Last modified: 2014-09-02 11:21:04 UTC
Filing as bug as it's a week now. Gerrit change #67531 was merged (and deployed, according to Chad), but it wasn't enough; probably thanks to Gerrit change #68110, it now works for core; but extensions are still broken. Example: https://git.wikimedia.org/commitdiff/mediawiki%2Fextensions%2FWikibase/127c6953860a41a8a03d73f97814f13855dee51e but old translation at [[d:MediaWiki:Wikibase-comment-update/nb]]
Chad, do you plan to look into this?
Yes, I was going to finish this up today. I never said https://gerrit.wikimedia.org/r/#/c/67531/ was deployed, I just thought someone said it was. Looking at 1.22wmf6, it doesn't look like it's been deployed. Deploying this will fix it for production.
(In reply to comment #2) > I never said https://gerrit.wikimedia.org/r/#/c/67531/ was deployed, I just > thought someone said it was. Sorry for misrepresenting your words, it wasn't intentional.
Would probably be worth checking in the puppet repo whether the update scripts there override the standard urls in this extension as well.
(In reply to comment #4) > Would probably be worth checking in the puppet repo whether the update > scripts > there override the standard urls in this extension as well. The update scripts do override the standard urls, to point to an already-checked-out copy of mediawiki/core and mediawiki/extensions in the local filesystem. So the addition of urlencode in https://gerrit.wikimedia.org/r/#/c/67531/1/LocalisationUpdate.class.php is probably going to break things for any extension that has i18n files in a subdirectory or any other characters affected by urlencode in its filename. I suppose we'll find out for sure tonight when l10nupdate runs against testwiki.
Related URL: https://gerrit.wikimedia.org/r/68546 (Gerrit Change I22613d2ebcff86d6e6c5f106baf01c790f413f80)
(In reply to comment #5) > I suppose we'll find out for sure tonight when l10nupdate runs against > testwiki. Example given in comment 0 has been correctly updated. Can this be closed as FIXED?
