Revert followup change for #337585 -- the only thing an empty macro should return is the autogenerated id.

#802102 by rooby - When editing locations, after clicking to remove a marker new markers are not properly added.

Fix #337585 -- Behavior flags on macros weren't using the site defaults. This most often manifested as missing "nomousezoom".

Fix #703480, patch by dnotes: Brown paper bag fix in gmap_module_invoke: Don't nuke the return value repeatedly when cycling through hook implementers.
Fixes an issue where all the behavior flags disappear.

(cherry picked from commit 9a79fc758f8a7f9d5b692d725722abfe2c98997f)

(cherry picked from commit 21ffd90556103f3f1a045a9c8216926401c62ff3)

Fix #582934, patch by me, rerolled by pivica and hutch: PHP 5.3 compatibility for hook_gmap(). (Manual backport)

* Backport bec's gmap_polyutil work.
* Minor whitespace change.

Fix #676108. Original patch by greenSkin, heavily modified by me to hopefully maximize compatibility.

In theory, this should work in both keys_api and keys now.
(cherry picked from commit 2831abcbe55a2b60dbeaeb52e895d0877ef20615)

(cherry picked from commit 62cb91318ddbf045a98b55bd02682b0320cf0ccc)

(cherry picked from commit bd26544327fea2a7a76f78ce963f4ace0c12758d)

(cherry picked from commit 751d05ac637eba7c25155e5e14ed6fd26418314e)

Stop confusing the marker system by blindly wrapping marker.text in a div, even
if it didn't exist.

This lets us work around the fact that gmap_widget_setup() completely fails to work on CCK fields.

Example:
<code>
  $remap = array(
    'edit-field-latitude-0-value' => array('classes' => array('gmap-locpick_latitude'), 'id' => 'gmap-loc0-locpick_latitude0'),
    'edit-field-longitude-0-value' => array('classes' => array('gmap-locpick_longitude'), 'id' => 'gmap-loc0-locpick_longitude0'),
    'edit-field-address-0-value' => array('classes' => array('gmap-locpick_address'), 'id' => 'gmap-loc0-locpick_address0'),
  );
  drupal_add_js(array('gmap_remap_widgets' => $remap), 'setting');

  /* ... stuff copied from gmap_set_location(), etc... */
</code>

HOPEFULLY it will work okay.
The reason I was reluctant to change this is the interactions between drupal_add_js()
and hook_filter(), described at
http://mostrey.be/hookfilter-versus-drupaladdjs

If anyone has troubles with macros after this change, I will have to revisit gmap's filter
stuff.

Usability: Don't show a JS error when attempting to display a map before the API key has been configured.

This should help when visiting the settings page for the first time, etc. The immediate JS error isn't very nice.

This fixes the annoying jerky / flashing marker when using locpick, and makes the whole thing feel a lot smoother.

Also remove old API.txt file -- the contents had been moved to advanced help, but I forgot to cvs rm on this branch.

Rename permissions:
"show node map" -> "view node map"
"show user map" -> "view user map"
"user locations" -> "view user location details"

#397986, reported by VeryMisunderstood: Fixing php notice / serious performance issue in marker titles.

Fix #155104 by Scott Falconer: Add "Zoom to cluster area" support.

Allow falling back to the default Clusterer marker if desired.

Allow entering larger numbers in some marker manager settings.

Allow choosing between the original and the new "zoom to marker area" behavior on Clusterer.

http://groups.google.com/group/Google-Maps-API/msg/33df59945c139afe?dmode=source

This file needs some serious work still, it will definately need the most work before dynamic bootstrapping works.

(Header changes only.)