Safari and mutiple posts

[GoogleCodeExporter]

Hi 
This took a while for me to find out.
When displaying multiple posts at the same location, Firfox displays all the 
messages at once in the 
map; great!
but Safari (Mac) puts out following error code:

Request for 
http://www.spamgalantay.com/wordpress/wp-content/plugins/geo-mashup/geo-
query.php?post_id=1146 failed: Error - DOM Exception 4

For every post you have to click awy this message until the latest post finally 
appears. I overcome 
the problem by shifting the position of every post slightly.
 best Rod

Original issue reported on code.google.com by r.galan...@fhbb.ch on 20 Feb 2007 at 9:09

[GoogleCodeExporter]

in geomashup.js (line 170 - 172):

    var newItem = request.responseXML.getElementsByTagName('item')[0];
    var channel = GeoMashup.locations[point].xmlDoc.getElementsByTagName('channel')[0];
    channel.appendChild(newItem); //throws an error because you are not allowed to mix nodes from 
different xml documents

replace with:

    var newItem = request.responseXML.getElementsByTagName('item')[0];
    var channel = GeoMashup.locations[point].xmlDoc.getElementsByTagName('channel')[0];

    // you have to import nodes before you can use them 
    var importedNode = GeoMashup.locations[point].xmlDoc.importNode(newItem, true);

    channel.appendChild(importedNode);

best,
sigurd (scriptamac.at)

Original comment by sig...@scriptamac.at on 9 May 2007 at 8:38

[GoogleCodeExporter]

Many thanks for that, Sigurd. Of course, IE requires a little extra nastiness. 
The
attached version seems functional. I'll get another bugfix release out soon.

Original comment by dylankk...@gmail.com on 10 May 2007 at 11:58

• Changed state: Fixed

Attachments:

• geo-mashup.js

[GoogleCodeExporter]

Final fix with extra Safari title handling in 1.0.3.

Original comment by dylankk...@gmail.com on 17 May 2007 at 9:58

• Changed state: Verified