drupal_urlencode() is no longer defined in 7.x

Does replacing it with urlencode() make it work properly?

Here is a patch on current 7.x-1.x

I found the same problem/fix. This patch needs to be committed but I don't think it has yet? apologies if I'm wrong...

Setting this issue to 'Needs Review' for patch in #3

drupal_urlencode is drupal_encode_path in drupal 7.
See here on api.drupal.org (comment #1): http://api.drupal.org/api/drupal/includes--common.inc/function/drupal_ur...

Good spot @phoenix. Patch in #3 is not applicable, uses php urlencode function. Just change that for drupal_encode_path instead...

Here is a patch as per #7, I can't test at the moment, rebuilding server.
Should work...

Looks good to me, how wrong can a one liner go ;)

#9 works here too

#9 is fine for me too thanks.

I added this tiny function to my module, and everything works:

function drupal_urlencode($address) {
return drupal_encode_path($address);
}

With this function present no need to patch GMap module.

Two obvious questions come to mind:

1. Without a patch (or the function that I suggest) the gmap_geocode function is useless. The change is so simple, why GMap 7.x-1.x-dev has not been fixed yet?

2. Question to Drupal core guys: why to change the API function name, so some modules have to be patched? Sounds like unnecessary change without respect to backwards compatibility...

patch #9 is applied to 7.x-1.x branch. Thanks a lot!