Regex to remove CDATA wrapper from an XHTML file

Question

I'm trying to remove the CDATA wrapper in the following script (content has to be wrapped by CDATA to pass XHTML validation):

<script id="tplTest" type="text/html">

//<![CDATA[ 
<p id="msg">Hello</p>
<p>Another test: <#= ddd.ArtID #></p> 
//]]>

</script>

JavaScript:

var strTmp = document.getElementById("tplTest").innerHTML;
var strNew = strTmp.replace(/[\/(\/!\[)\]CDATA]/g, "").replace(/[(\/\/\]\])]/g, "");

It removes most to the CDATA mark-up except for the start/end (<, >) tags:

< 
<p id="msg">Hello<p>
<p>nother test: <#= ddd.rtI #><p> 
>

Question: How should I modify the regex to additionally remove these leading and trailing <, > tags?

cllpse · Accepted Answer · 2009-06-26 15:13:52Z

5

You could just replace the raw string and skip using regular expressions all-together:

"FOO BAR".replace("FOO", ""); // replace "FOO" with "" (nothing)

In your case:

var stringToSanitize = "//<![CDATA[ xxx //]]>";

var sanitizedString = stringToSanitize
                      .replace("//<![CDATA[", "")
                      .replace("//]]>", "");

Regular expressions in JavaScript are slow. So on top of you getting your problem solved, you might see a slight speed-increase using my example.

edited Jun 26, 2009 at 15:13

answered Jun 26, 2009 at 15:07

cllpse

21.8k37 gold badges134 silver badges170 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Alex Martelli · Accepted Answer · 2009-06-26 15:09:07Z

0

Isn't it enough to just add a < after the first slash in the first replace and a '>' after the last slash in the last replace? If your regex dialect takes these angle brackets as magic characters (few do) you can use \< and \> respectively, i.e., escape them with backslashes.

answered Jun 26, 2009 at 15:09

Alex Martelli

887k175 gold badges1.3k silver badges1.4k bronze badges

Collectives™ on Stack Overflow

Regex to remove CDATA wrapper from an XHTML file

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related