location.dk.inc map link providers don't have a terms of service link.

This causes an undefined index error on the admin/settings/location/maplinking page.

CommentFileSizeAuthor
#4 location.dk-1033878.patch771 byteshutch

Comments

rooby’s picture

rooby commented
Status: Active » Fixed

Committed to all branches
http://drupal.org/cvs?commit=485648
http://drupal.org/cvs?commit=485664
http://drupal.org/cvs?commit=485644
http://drupal.org/cvs?commit=485640

hutch’s picture

hutch commented

Would it make sense to add something like this?

    'google' => array(
      'name' => 'Google Maps',
      'url' => 'http://maps.google.dk',
      'tos' => 'http://www.google.com/intl/da_dk/help/terms_maps.html',
    ),
rooby’s picture

rooby commented
Status: Fixed » Active

Yeah, it probably would actually.

I wasn't paying much attention, the lack of tos was breaking my D7 version so I went for the quick fix and moved on :)

Can come back to this.

hutch’s picture

hutch commented
StatusFileSize
new location.dk-1033878.patch771 bytes

Here is the patch for google in location.dk.inc, made in DRUPAL-6--3

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Issue summary: View changes
Status: Active » Closed (outdated)

Closing old D6 issues as D6 is end of life