When set coordinates instead of address fields the Google Maps link is not correct for some countries such as Netherlands.

Steps to reproduce:

  1. Select Netherlands country
  2. Fill Latitude and Longitude

On the output it render next link:
<a href="http://maps.google.nl?q=%2C+%2C+%2C+nl">Google Maps</a>

CommentFileSizeAuthor
#4 interdiff-2781227-2-1.txt1.91 KBozin
#4 location-google-maps-link-2781227-2.patch14.54 KBozin
#2 location-google-maps-link-2781227-1.patch14.3 KBozin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ozin created an issue. See original summary.

ozin’s picture

ozin
Primary language Ukrainian
Location 🇺🇦Lviv
CreditAttribution: ozin as a volunteer and at Lemberg Solutions commented
Version: 7.x-3.7 » 7.x-3.x-dev
Status: Active » Needs review
FileSize
location-google-maps-link-2781227-1.patch14.3 KB

There is a patch which fix Google link for list of countries. Please check.

ozin’s picture

ozin
Primary language Ukrainian
Location 🇺🇦Lviv
CreditAttribution: ozin as a volunteer and at Lemberg Solutions commented
Title: Wrong Google Maps link for Netherlands and some other countries when coordinates are set. » Wrong Google Maps link for Netherlands and some other countries when coordinates are set only.
ozin’s picture

ozin
Primary language Ukrainian
Location 🇺🇦Lviv
CreditAttribution: ozin as a volunteer and at Lemberg Solutions commented
FileSize
location-google-maps-link-2781227-2.patch14.54 KB
interdiff-2781227-2-1.txt1.91 KB
ozin’s picture

ozin
Primary language Ukrainian
Location 🇺🇦Lviv
CreditAttribution: ozin as a volunteer and at Lemberg Solutions commented

Fixed notices.

gg24’s picture

gg24 CreditAttribution: gg24 as a volunteer and at QED42 commented
Status: Needs review » Reviewed & tested by the community

I able to reproduce the issue. Seems patch works as intended. Issuing RTBC.

Thanks!

ikeigenwijs’s picture

ikeigenwijs CreditAttribution: ikeigenwijs commented

# Works like a charm