Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

CDATA XHTML handling #163

Open
everton-rosario opened this issue Aug 23, 2016 · 0 comments
Open

CDATA XHTML handling #163

everton-rosario opened this issue Aug 23, 2016 · 0 comments

Comments

@everton-rosario
Copy link
Contributor

Currently, loadHTML (DOMDocument) does not do anything on parsing cdata and the script content can include CDATA tags. This causes an issue on Elements\GeoTag->isValid() and the code omit script in CDATA even we use the original transform function.

One of the workaround for developers is to write

<script type="text/javascript">
// <![CDATA[
script here
// ]]>
</script>

And this is a typical solution for browsers who cannot understand XHTML.

On the other hand, we can remove '' string from the original content. do you want to do this on SDK in this PR?

This is a side task from #161

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant