Last modified: 2013-01-08 00:18:24 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T37522, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 35522 - Yandex maps patch review & merge
Yandex maps patch review & merge
Status: RESOLVED WONTFIX
Product: MediaWiki extensions
Classification: Unclassified
Maps (Other open bugs)
unspecified
All All
: Low normal with 2 votes (vote)
: ---
Assigned To: Jeroen De Dauw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-27 15:22 UTC by Jeroen De Dauw
Modified: 2013-01-08 00:18 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Comment 1 Niklas Laxström 2012-03-27 15:30:06 UTC
On quick skim:

JavasScript repeated multiple times (copied from old version of extension before it was fixed?). Hard coded cyrillic text.


+ * Maps language codes to Google Maps API v2 compatible values.
Is this needed for Yandex too?


+ * This groupe contains all Yandex Maps related files of the Maps extension.
groupe

Indentation in Maps_Settings.php
Comment 2 Jeroen De Dauw 2012-03-27 15:37:43 UTC
Yeah... this needs quite some cleanup. I first want to see it work to begin with :)
Comment 3 Yury Katkov 2012-03-30 11:27:06 UTC
http://vedmaka-smw.dyndns.org/wiki/STestMap - it works I promise! Alexey works in my team ;)
Comment 4 Jeroen De Dauw 2013-01-08 00:18:24 UTC
Marking wontfix as this seems to be dead. Can open a new bug if there is new code to review

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links