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?