You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.
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?
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
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
The text was updated successfully, but these errors were encountered: