Map Link Doesn't show up - phone problem also fixed

I changed the if statement to

if (!$variables['hide']['map_link']) {

and it worked.

When I make this substitution I just get the white screen. Does anyone have any more suggestions or a more descriptive example?

thanks

I had this same problem in 5.x-3.0. Fix in #2 worked for me as well.

The fix in #2 did nothing for me. I even went so far as to change the code to always call the location_map_link function, and still no map links showing up. I can't see where to fix the phone number problem.

 // Map link.
  //$variables['map_link'] = '';
  //  if (isset($variables['hide']['map_link']) && !$variables['hide']['map_link']) {
 // if (!$variables['hide']['map_link']) {
    $variables['map_link'] = location_map_link($location);
 // }

fletch11...You probably left out the other curly bracket. White screen means error. Change it to look like this (I commented out the first if statement, and replaced it with the new one. (this did not work for me, btw)

  // Map link.
  //$variables['map_link'] = '';
  //  if (isset($variables['hide']['map_link']) && !$variables['hide']['map_link']) {
  if (!$variables['hide']['map_link']) {
    $variables['map_link'] = location_map_link($location);
  }

I did fix this problem using the instructions in Comment #2. One important note. You cannot have the location field "Country Name" hidden in the Content Type > Locative > Display Fields settings, or it won't show map links. That is a requirement of looking up the map. Now, I am trying to debug the phone number problem!

I figured out how to fix the phone problem.

In location.tpl.php, near line 28, after this piece of code where it's printing the country...

<?php if ($country_name) { ?>
<div class="country-name"><?php echo $country_name; ?></div>
<?php } ?>

Add this, so it prints out the phone number:

<?php if ($phone) { ?>
<div class="phone"><?php echo $phone; ?></div>
<?php } ?>

subscribing.. great fixes

This works for 5x.3.0 too.
please commit changes

The missing map link problem exists for the location-5.x-3-0 as well. For me, the patch given in #2 does not work properly, It does show the map link successfully, but the opposite problem now exists. The map link is always shown, despite the display settings. This is because the $variables['hide'] array has numerical indexes, with the values being the name of the field to hide. So, instead of checking if $variables['hide']['map_link'] exists, we need to check if the value 'map_link' is in the array $variables['hide'] - a subtle difference. A patch is attached.

If the above patch does not work for you, try this one. I had some issues by not doing a strict in_array comparison in the previous patch. Here is an updated patch.

I applied the patch #14 and it works for me; thanks! I previously was unable to get the "See Map" and the Google Maps link to show until I applied the patch. I am using Drupal 6.9.

I started with 6.x-3.0 on D6.9. Similar to the original poster I couldn't ever get the map link to appear. Made sure all the permissions were set and a map link provider was checked in location settings.

Then I upgraded to the most recent 6.x-3.x-dev. Still no map link. Tried both of the patches, in #13 and #14, and they don't seem to make a difference, at least for me (I'm jealous of dustonio!). Turned off all caching and cleared stuff just to be sure, each time.

I've actually never seen the map link, so I'm not sure exactly what it's supposed to look like, but there's certainly nothing that would resemble such a thing being displayed.

There's so much that's fantastic about this module, and I love it...but the mythical map link does sound like a nice feature. :)

Hi quixoticlife:

in collection settings of your content type have you allowed the collection of countries? This assumes you are using cck, but the map link does not work if country info is not collected or force defaulted.

Interesting! I figured it wouldn't work if it wasn't collected, but I didn't guess that it would fail with force-default.

Nonetheless, after setting all fields to "Allow," I still don't see a map link. Three of the fields (city, state, country) have defaults -- would that disable the link?

Sorry, I meant to say that it should work if the country field is set to 'force default' but won't if a country is not collected altogether. Can't say I know what would cause this if you've tried setting all fields to 'Allow' and manually entered valid locations in each field. This patch seemed to work for me after trying a lot of things. Have you modified your .tpl.php file for the node type? Also I found that using contemplate required the addition of custom code and that otherwise the link wouldn't show up.

Which patch worked for you, #13 or #14? I haven't created a .tpl.php file for the content type, and node.tpl.php, while slightly modified, is still displaying the straight $content value. I'm not using contemplate.

#14 worked for me... not sure what else to try - you might want to test another theme or disable all unneccessary modules to see if you get a response.

I can confirm the phone number display problem, I can also confirm that adding the code from commen #9 fixes it. This should definitely be committed. bdragon?

The patch in #14 makes the map link show up. I don't use the phone number so can't confirm anything about that. If this doesn't fix the phone number, I think that is a separate issue, because the failure of the map link to display is clearly the incorrect logic that is fixed by #14.

Confirmed #14 patch.

Committed something a bit different, because I figure it will be slightly more efficient than in_array.

http://drupal.org/cvs?commit=170589
http://drupal.org/cvs?commit=170591
http://drupal.org/cvs?commit=170593

Regarding the phone problem, I've been running with this patch for a while. Does anyone have any +1s or -1s for it?

#26 would be better if using instead of

#14 works so now the Google maps links displays.

Thanks!
DaveA

still have the same effect.. updated to the last dev and even have problems to find the equivalent program code in order to change.

is there any better explanation available or will the downloadable code updated soon ? i urgently need the location including a functional gmap.

the phone and fax stuff is working; the map shows up while editing and disappears while in "normal" homepage made.

any quick help is appriciated.

steffen

edit: found "Then, make sure permissions on "files," "files/js" and "files/js/gmap_markers.js" are set to 777 - that's working for me - so far, anyway." this solved my problem.

The patch in #26 works, but could use a bit of formatting cleanup. There is no space between the field label and the field.

Map Link disappears from display when "Country" (not to be confused with "Country Name") checkbox is checked.

Appears fine if "Country" checkbox (and "Map Link" checkbox) is left unchecked.

Confirmed #31. Definitely something screwy with these checkboxes. I'd bet it's just a typo somewhere.

duplicate? #387198: No Phone Field in User Accounts

Same bug exists in 5.x

I can confirm that the dev version fixes the issue.
For some reason the last time I ran drush updates it installed the 3.0 version. Should that one really be the recommended version with all those bugs? At any rate I am sooo excited that the dev version is a happy camper! Thanks for the fixes!

Hmmm was the patch in #26 included in the dev release?

Patch in #26 doesn't appear to be committed in 6.x-3.x-dev. Also, I can confirm that the bug in #31 still exists in dev from today (6-12-2009).

Same. Can we please get this in an RC release too?

Applied patch #26 and it works for me with 3.1-rc1.

@skyredwang in #27 I think you posted some HTML we can't see. Please repost.

Formatting mentioned in #30

and

Checkbox problem in #31

Need addressed.

Someone needs to address those and re-roll this againt head.

The cvs checkout command is on the location project page [edit: http://drupal.org/project/location] under a deveopler tab [edit: it's actually called CVS Instructions]. And google for:
drupal apply patch
and
drupal create patch
if you (someone) wants to take a stab at this.

If anyone needs help making the re-roll patch post back here or try #drupal-geo for help on irc

Has this fix been committed? No map links display for me.

Yes. the fix has been committed.

I am running location -6.x-3.1-r1 module

re: map_link

I am allowing collection of all fields and there is data entered for name, street, city, province, state and country. If I display all the fields (in other words I do not hide anything) the map link shows up fine. But that is not what I want to do. I only want to display the map link. The data is all collected, I just don't want to display all of it. When I check to Display in full text and uncheck all but the map link, the map link is not displayed.

re: phone number

I do not have an option to collect phone number.

Thanks!
Diana