Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

StoreLocator not showing stores on map when locale is set to nl_NL #13

Open
vierkantemeter opened this issue May 8, 2013 · 1 comment

Comments

@vierkantemeter
Copy link

Just found out that when my locale (MODX system setting) is set to 'nl_NL', there are no stores showed on the map. This locale setting will automatically change the dot (.) to a comma in a coordinate, like: 51.913953 to 51,913953. And than Google Maps don't know what to do with it because it expects a dotted coordinate. I saw this happened in the page source code, (see below) but it will also occur somewhere else in your codes.

With locale 'nl_NL':
< a onclick="storeLocatorMap.setCenter(new google.maps.LatLng(51,91412, 4,580343)); storeLocatorMap.setZoom(13);" style="cursor: pointer;">• Location A< /a >

With locale 'en':
< a onclick="storeLocatorMap.setCenter(new google.maps.LatLng(51.91412, 4.580343)); storeLocatorMap.setZoom(13);" style="cursor: pointer;">• Location A< /a >

So, after changing the locale to 'en', the stores are shown on the map again. My server (CentOS 5.5) requires 'nl_NL' as locale, not just 'nl'. So using 'nl' is not an option.

Please fix StoreLocator so it's usable when locale is set to 'nl_NL', or something similar :)

I've also discussed this here, with a temporary fix: http://forums.modx.com/thread/74885/support-topic-for-storelocator-1-0-pl-1?page=4#dis-post-465567

MODX 2.2.6
StoreLocator 1.1.0

@Jako
Copy link

Jako commented Jun 15, 2013

Had the same problem with german locale:

Fast and ugly fix:
In sl.scriptstoremarker.tpl change placeholders like this
[[+store.latitude:replace=,==.]] and [[+store.longitude:replace=,==.]]

And in sl.storerow.tpl
[[+onclick:replace=,==.:replace=. ==, ]]

But it would be better to change the snippet code with sprintf (#14).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants