CarlosSimoes / geoxml3

Automatically exported from code.google.com/p/geoxml3
0 stars 0 forks source link

kml files not properly marked for reloading #43

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Parse two kml files.
2. Make a change to both kml files and parse them again.

What is the expected output? What do you see instead?
I expect to see the two modified kml files.
Instead I see the modified first kml file, the previously parsed second kml 
file, and the modified second kml file. The second kml file was not marked for 
reloading, the first one was. See line 88 of the geoxml3.js script, which reads 
"thisDoc.reload    = true;"

What version of the product are you using? On what operating system?
http://geoxml3.googlecode.com/svn/branches/polys/geoxml3.js
r60 | geocodezip@gmail.com | 2011-10-02 04:06:03 +0200 (Sun, 02 Oct 2011) 
Linux, Google Chrome

Please provide any additional information below.
Changed the code. Attached my modified version.

Index: geoxml3-polys-read-only/geoxml3.js
===================================================================
88c88
<           docs.splice(j, 1);

---
>           //docs.splice(j, 1);
94a95
>         internals.docSet.push(thisDoc);
98d98
<       internals.docSet.push(thisDoc);

Original issue reported on code.google.com by verm...@nurc.nato.int on 10 Nov 2011 at 1:37

Attachments:

GoogleCodeExporter commented 9 years ago
Do you have a test case that shows the issue?

Original comment by geocodezip on 10 Nov 2011 at 4:06

GoogleCodeExporter commented 9 years ago
Nope, sorry. I tried to explain how to reproduce the problem as best as I 
could. But I don't have a simple html/JavaScript page that shows the problem. 
My code is part of an application built on top of the (fantastic) wt library.

Btw, I hardly know JavaScript.

Original comment by verm...@nurc.nato.int on 10 Nov 2011 at 4:19

GoogleCodeExporter commented 9 years ago
fixed.
Test page:
http://www.geocodezip.com/geoxml3_test/v3_geoxml3_reloadTest_linkto.html?lat=28.
477488&lng=-81.234067&zoom=16&type=m
(requires modifying kml on the server)

Original comment by geocodezip on 12 Nov 2011 at 4:36