CODEWITHSUNDEEP
regexcoldfusionreplacehtmlspecialchars
Dollique
Dollique

Reputation: 1012

Coldfusion ReReplace "&" but not htmlspecialchars

I need to replace all & with with & in a string like this:

Übung 1: Ü & Ä

or in html

Übung 1: Ü & Ä

Like you see htmlspecialchars in the string (but the & is not displayed as &), so I need to exclude them from my replace. I'm not so familiar with regular expressions. All I need is an expression that does the following:

Search for & that does either follow a (space) or does not follow some chars, excluding a space, which are ending with a ;. then replace that with &.

I tried something like this:

<cfset data = ReReplace(data, "&[ ]|[^(?*^( ));]", "&amp;", "ALL") />

but that replaces every char with the $amp;... ^^'

Sorry, I really don't get that regex things.

Upvotes: 6

Views: 3151

Answers (4)

Alex
Alex

Reputation: 282

For performance & issues free, just go with Decimal code point like so...

<cfset html = Replace(html, Chr(38), "&amp;", "all")>

Upvotes: -1

Cerbrus
Cerbrus

Reputation: 72867

I think it would be easier to simply replace all occurrences of & with &amp;, and then replace the wrongly replaced ones again:

<cfset data = ReReplace(ReReplace(data, "&", "&amp;", "ALL"), "&amp;([^;&]*;)", "&\1", "ALL") />

I haven't tested this in ColdFusion (since I have no clue how to), but it should work, because in JavaScript, the regex itself works:

var s = "I we&nt out on 1 se&123;p 2012 and& it was be&tter & than 15 jan 2012"
console.log(s.replace(/&/g, '&amp;').replace(/&amp;([^;&]*;)/g, '&$1'));
//"I we&amp;nt out on 1 se&123;p 2012 and&amp; it was be&amp;tter &amp; than 15 jan 2012"

So I assume the regex will also do its trick in CF.

Upvotes: 4

Peter Boughton
Peter Boughton

Reputation: 112170

Problem with existing attempt

The reason your attempted pattern &[ ]|[^(?*^( ));] is failing is primarily because you have a | but no bounding container - this means you are replacing &[ ] OR [^(?*^( ));] - and that latter will match most things - you are also misunderstanding how character classes work.

Inside [..] (a character class) there are a few simple rules:

  • if it starts with a ^ it is negated, otherwise the ^ is literal.
  • if there is a hyphen it is treated as a range (e.g. a-z or 1-5 )
  • if there is a backslash, it either marks a shorthand class (e.g. \w), or escapes the following character (inside a char class this is only required for [ ] ^ - \).
  • you are only matching a single character (subject to any qualifiers); there is no ordering/sequence inside the class, and duplicates of the same character are ignored.

Also, you don't need to put a space inside a character class - a literal space works fine (unless you are in free-spacing comment mode, which needs to be explicitly enabled).

Hopefully that helps you understand what was going wrong?

As for actually solving your problem...

Solution

To match an ampersand that does not start a HTML entity, you can use:

&(?![a-z][a-z0-9]+;|#(?:\d+|x[\dA-F]+);)

That is, an ampersand, followed by a negative lookahead for either of:

  • a letter, then a letter or a number, the a semicolon - i.e. a named entity reference

  • a hash, then either a number, or an x followed by a hex number, and finally a semicolon - i.e. a numeric entity reference.

To use this in CFML, to replace & with &amp; would be:

<cfset data = rereplaceNoCase( data , '&(?![a-z][a-z0-9]+;|##(?:\d+|x[\dA-F]+);)' , '&amp;' , 'all' ) />

Upvotes: 10

bphillips
bphillips

Reputation: 459

The other option you have is to not use REGEX at all. For the sample string you listed, you are simply tying to replace the html ampersand ("&"), without affecting the html entities. This can be accomplished just using REPLACE.

Remember that when using entities, there will be no spaces around the ampersand character, where as to convert an ampersand character to an HTML entity, there is typically a leading and trailing space. REPLACE will find every case of " & " and update, without affecting any of the "&Uuml" strings (e.g. no leading and trailing space).

<cfset html = "&Uuml;bung 1: &Uuml; & &Auml;">
<cfset parsedHtml = REPLACE(html," & ", " &amp; ","All")>

Upvotes: 0

Related Questions