Add Token module support for latitude and longitude [#2072263]

Initial support for system-wide tokens:

[locationmap:latitude]
[locationmap:longitude]

Useful for Location Map issue: #1825468: Integration with Meta Tags module - Drupal 7

Working OK for adding custom (geo) meta tags from metatag module.
Related issue: #1278146: [META] Ensure all meta tags are supported (no patches, just planning)

Comment	File	Size	Author
#1	locationmap--2072263--initial-token-support--1.patch	1.9 KB	amontero
#1
	locationmap--initial-token-support--0.patch	1.82 KB	amontero

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #0.0

amontero

Barcelona

CreditAttribution: amontero commented 23 August 2013 at 01:32

Issue summary:

View changes

Linking issue

Comment #1

amontero

Barcelona

CreditAttribution: amontero commented 26 August 2013 at 18:10

Title:

Initial support for system-wide tokens

» Add Token module support for latitude and longitude

File	Size
locationmap--2072263--initial-token-support--1.patch	1.9 KB

Added descriptions to get rid of status page warnings.

Comment #1.0

amontero

Barcelona

CreditAttribution: amontero commented 26 August 2013 at 18:10

Issue summary:

View changes

Edit

Comment #2

babbage CreditAttribution: babbage commented 25 March 2014 at 07:11

Issue summary:

View changes

Sounds like a suitable idea for a patch. For future reference, this should have been posted to #1825468: Integration with Meta Tags module - Drupal 7 rather than creating a new issue, since it directly addresses the issue raised in that thread. Given other extreme commitments, may be a little while before I can test this, though I am not the only person with commit rights here so this may happen sooner. :)

Edit: Oh, ok, perhaps what you're saying is that Token support is necessary but not sufficient for Meta Tags support? In that case, perhaps a separate issue was indeed correct. My bad. :)

Comment #3

amontero

Barcelona

CreditAttribution: amontero commented 3 May 2014 at 11:51

Parent issue:

» #1825468: Integration with Meta Tags module - Drupal 7

Hi babbage.
Thanks for your feedback. Even if you could not test the patch, did you review the code?
Is there anything else needed to get this commited? (apart from third party testing)
TIA.

Comment #4

rupertj CreditAttribution: rupertj commented 24 May 2014 at 14:10

Status:

Needs review

» Closed (fixed)

Thanks - Looks good to me, so I've committed it. The only thing I did change was the wording that appears in the token UI. IE: "Location Map Tokens" => "Location map" to match the case & style of the other options, and "Latitude" => "Latitude of map marker." just for a bit more information.