CDATA section ignored in KML description

[ghost]

When I load a KML file the KML description is ignored when it's inside a CDATA section.

This works just fine,

      <description>
           Here is some text
      </description>

but not this one;

      <description>
        <![CDATA[
             Here is some text
        ]]>
      </description>

It seems an old issue in togeojson.js and should be fixed https://github.com/mapbox/togeojson/issues/42 Using the latest togeojson.js file doesn't solved my problem.

Is this a leaflet.filelayer issue or togeojson.js ?

Regards,

Robertico

[leplatrem]

Hi Robertico !

This part parses the XML into nodes and gives it to togeojson : https://github.com/makinacorpus/Leaflet.FileLayer/blob/gh-pages/leaflet.filelayer.js#L80-L82

If the problem was fixed in togeojson, we might pass the raw XML instead of parsing it here...

Could you try to change this and see if it helps ?

Thanks for reporting!

[ghost]

Hi,

I'm not sure I understand you correctly. Do I need to comment out these lines? When I do so, it doesn't work at all.

Sorry, I'm not a professional programmer :-)

[leplatrem]

Well, I upgraded togeojson towards 0.8.1, and it works wel, it strips out CDATA out of texts !

[ghost]

I doubted my findings, but... this doesn't work.

      <description>
        <![CDATA[
             Here is some text
        ]]>
      </description>

but this does !

<description><![CDATA[Here is some text]]></description>

So it must be a single line without line breaks. As far as I know they're both valid. Nevertheless a bug?

[leplatrem]

In the example I tried it worked. But this way :

<description><![CDATA[

             Here is some text
]]></description>

Otherwise, it would be a bug in togeojson...

[ghost]

Thanks. Created a new issue on togeojson for this.

[SBats]

I close this issue. If someone has still problems with it please reopen !