b03tz / StoreLocator

StoreLocator is a MODX addon that let's your customers search for stores in their vicinity.
http://www.scherpontwikkeling.nl
6 stars 10 forks source link

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

Open vierkantemeter opened 11 years ago

vierkantemeter commented 11 years ago

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 commented 11 years ago

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 (https://github.com/b03tz/StoreLocator/pull/14).