I am new to OpenLayers development, so I should begin for thanking elemoine, probins and crschmidt for their advice and comments on my pull requests.
On the subject of pull request #445 for issue #267 there appears to be an impasse: https://github.com/openlayers/openlayers/pull/445 The original ticket mentioned to separate problems: a) .desc attributes did not get exported as <desc> nodes, so the roundtrip GPX.write(GPX.read(someGPX)) did not return the original document or equivalent. b) <desc> nodes are written out as textnodes, not as [CDATA]. My code fixes a) but ignores b). It is controversial whether <desc> nodes should be written out as CDATA[]. From the but description and the comments at the pull request, probins thinks so, but crschmidt thinks that they should be textnodes. All the GPX xml literals in the GPX.html test are textnodes, and the GPX schema at suggests that textnodes are indeed the right way to encode the description element: <xsd:element name="desc" type="xsd:string" minOccurs="0">. Issue a) caused data loss and is now fixed, but I am not even sure what would be the severity of issue b) if it turns out to be a real issue. I am new here, so I am writing to the list to draw attention to the discussion, and to request advice on how to get the bug fix into master. Thanks for your attention, Javier Candeira _______________________________________________ Dev mailing list d...@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/openlayers-dev
