The attached patch adds a leaflet formatter for location fields (from the location_cck submodule of location) as well as integration with leaflet_views.

CommentFileSizeAuthor
#9 leaflet-2062873-9-location_module_support.patch8.08 KBlily.yan
#6 leaflet-2062873-6-location_module_support.patch7.67 KBlily.yan
#1 leaflet-2062873-1-location_module_support.patch6.55 KBmstrelan
leaflet_location_module_support.patch6.38 KBmstrelan

Comments

mstrelan’s picture

mstrelan commented
StatusFileSize
new leaflet-2062873-1-location_module_support.patch6.55 KB

Not sure why but sometimes the items passed to leaflet_process_location() in the previous patch were not actually location items. This is resolved in the attached patch.

rdeboer’s picture

rdeboer
Location Melbourne
commented

Hi Michael,
Thanks for contributing this patch.
As far as the integration of leaflet_views is concerned, are you aware of IP Geolocation Views and Maps?
Rik

mstrelan’s picture

mstrelan commented

Wow, no, I was not aware of that module. It looks fantastic, thanks.

rdeboer’s picture

rdeboer
Location Melbourne
commented
Status: Needs review » Closed (won't fix)

Covered by IPGV&M.

kpaxman’s picture

kpaxman commented
Issue summary: View changes
Status: Closed (won't fix) » Needs work

We wanted to be able to use Leaflet with Location as well, and found that IPGV&M didn't let us use it as a field formatter, so we came back to this patch. It didn't apply against the current version, so we're working on an updated version and will add it here when it's ready.

lily.yan’s picture

lily.yan commented
StatusFileSize
new leaflet-2062873-6-location_module_support.patch7.67 KB

Here is the patch mentioned above.

kpaxman’s picture

kpaxman commented
Status: Needs work » Needs review
itamair’s picture

itamair commented

Tnx and sorry ... It seems that I am the only active maintainer for the Leaflet module, but actually I am taking care of the 8.x branch,
and not able to follow, fix bugs and face features requests, and even review functionalities.
Actually I don't even have a Drupal 7 playground (and don't have time to set it up) to test them.
Evan I am not able to grant new maintainer privileges.

So ... I would be able just to commit new patches to dev, and just if they are 99% tested and reviewed by you external contributors.
So do that, and eventually under your responsibility ask me to commit to dev if really needed (will try to do that in my spare time).

lily.yan’s picture

lily.yan commented
StatusFileSize
new leaflet-2062873-9-location_module_support.patch8.08 KB

This patch includes the changes of leaflet views module in order to make sure leaflet views can support location correctly.