Setting project to Wikimedia-Site-requests, as this request is about settings / configuration of a Wikimedia website.

Please follow https://meta.wikimedia.org/wiki/Requesting_wiki_configuration_changes when requesting such kinds of changes. Thanks!

Hi again!

We locally discussed situation about "mapframe" and agreed that we want to use this wiki-instrument in future.

Please note there are not enough developers to develop new features for this extension. We're happy to enable it on other wikis, but we can't fulfil new features request, if you'll have any. Obviously, bugs will be remedied. Once you acknowledge this information, we can enable the extension.

Ok. We are acknowledged this information and you can enable the extension.

Thanks a lot!

Hmm... It's supposed to be already enabled, see below:

'wgKartographerEnableMapFrame' => [
	'default' => true,
	// Disable for FlaggedRevs wikis with $wgFlaggedRevsOverride=true (T191585)
	'bswiki' => false,
	'dewiki' => false,
	'huwiki' => false,
	'idwiki' => false,
	'kawiki' => false,
	'plwiki' => false,
	'sqwiki' => false,
	'trwiki' => false,
	'zh_classicalwiki' => false,
],

But it clearly isn't enabled.

[urbanecm@mwmaint1002 ~]$ mwrepl kawiki
Welcome to HipHop Debugger!
Type "help" or "?" for a complete list of commands.

Note: no server specified, debugging local scripts only.
If you want to connect to a server, launch with "-h" or use:
  [m]achine [c]onnect <servername>

hphpd> var_dump($wgKartographerEnableMapFrame);
var_dump($wgKartographerEnableMapFrame);
bool(false)

hphpd> ^Dquit
[urbanecm@mwmaint1002 ~]$

In T229726#5390694, @Urbanecm wrote:

Hmm... It's supposed to be already enabled, see below:
[...]
'kawiki' => false,
[...]

Did you remove this line? If not, this should be disabled as it is.

In T229726#5390696, @MSantos wrote:

In T229726#5390694, @Urbanecm wrote:

Hmm... It's supposed to be already enabled, see below:
[...]
'kawiki' => false,
[...]

Did you remove this line? If not, this should be disabled as it is.

Eh, I'm just blind, thanks :). I see T191585 says there are problems if wgFlaggedRevsOverride=true, which is the case for kawiki. I don't know what kind of problems that means, @Catrope, could you comment here, please?

Hi dear friends!

Is this task finished or not?

@Surprizi: Not. Please read the previous comment before your comment again. Thanks!

Yes, combining Kartographer with FlaggedRevsOverride causes some problems. We'll still enable Kartographer on your wiki if you want it, but you should know about this bug so that you can make an informed decision.

The bug happens when you have a <mapframe> tag with JSON inside it (used for markers on the map), this JSON is edited (to move/change the markers), and then a user views an older version of the page. In that situation, the map that was edited will break after some time (about an hour), and the user will see a blank grey rectangle instead of the map. On most wikis, users rarely view older versions of pages, but on wikis with FlaggedRevs it happens all the time: when a page has unapproved edits, most readers will be shown the approved/stable version by default, rather than the latest/unstable version. If the pending (unapproved) edits changed the markers on a map, these readers may see a broken map.

This page on test2wiki illustrates the situation in which the bug happens, but unfortunately it doesn't actually demonstrate the bug because static maps are not enabled there (I'll change that early next week, so that it breaks the way it's supposed to).

In summary:

• When viewing the latest (possibly unapproved) version of a page
  • Everything works fine
• When viewing an older (approved) version of a page
  • If the markers on the map were changed between the older (approved) and latest (unapproved) versions of the page
    • Then the map will break, and display a grey rectangle (but not immediately, it'll take up to an hour)
  • If the markers on the map were not changed
    • Then the map will display just fine

Thanks a lot!

Any update?

@Urbanecm: By who exactly, on which specific question?

@Aklapper I stalled the task above pending on community's "informed decision", if they want the extension to be enabled even with the bug @Catrope described in T229726#5419327, that will probably affect them.

Hello to all. Sorry, I just saw a specific proposal so the question, give me about 5-7 days and I will inform our community and will return with a specific answer. Thanks! :)

Thank you, @Mehman97. Let us know.

Hi @Urbanecm! I informed Georgian Wiki community about what was said above. The community decided that you need to enable this feature despite the bugs, so you can hardily start integrating new maps in Georgian Wikipedia. Thanks.

Change 546673 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[operations/mediawiki-config@master] Enable mapframe at kawiki

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

Change 546673 merged by Urbanecm:
[operations/mediawiki-config@master] Enable mapframe at kawiki

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

Mentioned in SAL (#wikimedia-operations) [2019-10-28T18:30:11Z] <urbanecm@deploy1001> Synchronized wmf-config/InitialiseSettings.php: SWAT: 30111f3: Enable mapframe at kawiki (T229726) (duration: 00m 53s)

In T229726#5419327, @Catrope wrote:

This page on test2wiki illustrates the situation in which the bug happens, but unfortunately it doesn't actually demonstrate the bug because static maps are not enabled there (I'll change that early next week, so that it breaks the way it's supposed to).

It took me a year and a day instead of a week, but I've now changed the configuration on test2wiki so that the linked page correctly showcases the bug.