hi,
just as small note, because its not mentioned in the instructions.
On the Firefox-consol I saw this missing API-key-note and
I can't see the map ...
and btw.: its not supporting Openstreetmap (OSM is better! & the future!)
but anyway: the plugin should be helpfull!
best, eli
Zenphoto version 1.4.14
Comments
I wrote the original plugin, I can see the 'no API key' message in the Javascript console, but the map is still loading for me.
Best to open a ticket over here:
https://github.com/wongm/zenphoto-geotagger
One question for acrylian - does Zenphoto already have a shared config item where a user can enter a Google Maps API key? I can see the official 'GoogleMap' plugin has a 'gmap_api_key' setting already.
The key is based on the domain name of the website, so having each Zenphoto plugin that uses Google Maps maintain their own key config item will get messy.
I think the map does load but some coding stuff may not work completely. There is no generally shared option for this as it is plugin territory. But you can certainly just use the option of GoogleMaps if it is already set. But you need to check for it. If would stay set even if the GoogleMaps is disabled but you can never count on that of course.
To add you could add a check to the option class constructor of the theme and set it via setOptionDefault() if not set yet. In any case you need a plugin based option because someone might not use GoogleMaps.
I've fixed the issue with my latest release:
https://github.com/wongm/zenphoto-geotagger/commit/19012275e203ae4f4db27d3e48769cc76c9ccd21
I now hook into the 'gmap_api_key' option set by the official 'GoogleMap' ZenPhoto plugin, and added support for editing the value via my plugin.
Great!