Markers Do Not Save Using IE

Shane Birley - May 3, 2008 - 22:27
Project:cck_map
Version:5.x-3.3
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:needs review
Description

I have been testing a patch I had done for this module and noticed that if you are submitting with Internet Explorer, the markers are not saving. Works fine in Firefox.

#1

slybud - June 19, 2008 - 14:17

I've found an alternative, not fully tested, let me know if it works for you

- In the subfolder "icons", i've created a gif gopy of the png marker
marker_white.png => marker_white.gif

- In the file cck_map.js, line 225
marker_white.png => marker_white.gif

And it works fine in IE7 (an 6 probably)

#2

piersg - July 25, 2008 - 20:50

Made this change but no luck for me. IE (and Firefox) reports error in cck_map.js at line 387/388
IE:Line 388 Char 7 Error: Object Required Code: 0
Firefox: Error: document.getElementById("edit-" + this.prefix + "-" + i + "-lon") is null

Is it related to this? http://drupal.org/node/200429 Made this fix too but still doesn't work :(

#3

AlxM - October 16, 2008 - 19:56

Hi,

i'm also having this problem, but only in IE. Tried changing the marker to .gif with no luck.

The error im getting is similar to piersg. IE Line: 413 Error: Object doesn't support this property or method Code: 0.

does anyone have any idea what the problem is as it works fine in firefox?

#4

larowlan - November 27, 2008 - 11:52
Status:active» needs review

Hi
I had similar problems with IE (v7). After four hours of pain found the following fixes resolve the issue:
In gmapAdminController.prototype.initializePoints, declare the variables' scope as local first using the var keyword ie
var numPoints = ...
and on new line after this
var lon, lat, title, description, image;

In gmapAdminController.prototype.getCurrentDescription
change
description = desc.value; to
var description = desc.value;

Then do the other changes mentioned above regarding the replace function (this is in two places, lines 270 and 83).

Change
jsString = this.prefix.replace('-', '_', 'g') to
jsString = this.prefix.replace(/-/g, '_');

These changes are in the attached patch.

Basically there seems to be some sort of global namespace collision on the description variable in Internet Explorer.

Lee Rowlands
http://www.rowlands-bcs.com

AttachmentSize
cck_map.js_.patch 1.99 KB

#5

slybud - December 15, 2008 - 23:06

Thanks for your work but that hasn't worked for me : still got this problem in IE7 :(

Do you think it's related to the name of my cck (double underscore in it) : profil_coord_geo ?

Best regards

Sylvain Moreau

 
 

Drupal is a registered trademark of Dries Buytaert.