If you validate a page containing a gmap, it fails validation because ampersands in the google URL are not escaped. & needs to be represented as &. Only two lines need to be changed in the module, and a patch is included. (sorry if the patch is funky, I did try to follow guidelines.)

CommentFileSizeAuthor
gmap.module.patch1.29 KBhyperlogos

Comments

hyperlogos’s picture

hyperlogos commented

Also, on line 293 you have to add the type="text/javascript" property, or the page still won't validate. Sorry I didn't catch this and put it into my patch... but it's pretty obvious where it does, it's in the only <script language="javascript"> statement in the module.

webgeer’s picture

webgeer commented

This is now fixed on the 4.7 version, but I really don't have a clue what I'm doing with cvs and don't really know the best way of updating an older version. (I have a feeling it should be branched, but I don't know how that would work)

bdragon’s picture

bdragon commented

4.6 is no longer a supported Drupal version.

bdragon’s picture

bdragon commented
Status: Active » Closed (won't fix)