Change 426322 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/Kartographer@master] Add lang parameter to <mapframe>

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

Change 426322 merged by jenkins-bot:
[mediawiki/extensions/Kartographer@master] Add lang parameter to <mapframe>

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

I don't think https://gerrit.wikimedia.org/r/426322 should have been merged. Besides the no tests, it uses user language, not content language. As such, this ends up with weird behavior: if content author created a <mapframe lang=xx>, it will always show the map in xx, but if they didn't specify lang param, it will use User's language, not the content language.

In T192187#4131245, @Yurik wrote:

I don't think https://gerrit.wikimedia.org/r/426322 should have been merged. Besides the no tests, it uses user language, not content language. As such, this ends up with weird behavior: if content author created a <mapframe lang=xx>, it will always show the map in xx, but if they didn't specify lang param, it will use User's language, not the content language.

It does use the content language, not the user language. $this->getLanguage() ends up using $this->parser->getTargetLanguage(), which is the page language (which is generally the content language, except in weird cases like translated pages).

In T192187#4131256, @Catrope wrote:

It does use the content language, not the user language. $this->getLanguage() ends up using $this->parser->getTargetLanguage(), which is the page language (which is generally the content language, except in weird cases like translated pages).

My apologies, missed the second file's change, and thought you used $context->getLanguage(). All's good.

Tests: https://gerrit.wikimedia.org/r/426339

Change 426368 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/Kartographer@master] Also pass language code through leaflet for dynamic maps

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

Change 426389 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/Kartographer@master] Respect lang attribute in VisualEditor modules

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

Change 426368 merged by jenkins-bot:
[mediawiki/extensions/Kartographer@master] Also pass language code through leaflet for dynamic maps

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

Change 426389 merged by jenkins-bot:
[mediawiki/extensions/Kartographer@master] Respect lang attribute in VisualEditor modules

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

Checked in betalabs for enwiki and cawiki.

I used - ja for Japanese language.

<mapframe width="700" height="400" zoom="9" longitude="8.5062" latitude="46.8151" align="right" lang="ja" />

(1)
enwiki (beta) - displays available labels in Japanese on a page and on a map full view - e.g. https://en.wikipedia.beta.wmflabs.org/wiki/Mavetuna14
cawiki (beta) - - displays available labels in Japanese on a page but not on a map full view -
e.g. https://ca.wikipedia.beta.wmflabs.org/wiki/Mavetuna (https://ca.wikipedia.beta.wmflabs.org/wiki/Mavetuna#/map/0)
@Catrope - should lang be recognized in full map view on cawiki(beta)?

(2) When the lang parameter is not recognized, the fallback will be to the content lang of a wiki.

Change 428404 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/Kartographer@master] Make lang attribute work for static maps too

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

Change 428404 merged by jenkins-bot:
[mediawiki/extensions/Kartographer@master] Make lang attribute work for static maps too

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

Checked in cawiki betalabs(for static maps) and enwiki betalabs for view on a page and full-map view.

• maps will display available labels according to lang="[lang]" attribute
• unavailable/invalid or absent lang entry will result in the fallback to the wiki content lang

QA recommendation: Resolve.