Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

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.

like image 730
Dollique Avatar asked Jan 04 '13 08:01

Dollique


2 Answers

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.

like image 38
Cerbrus Avatar answered Oct 14 '22 14:10

Cerbrus


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' ) />
like image 92
Peter Boughton Avatar answered Oct 14 '22 13:10

Peter Boughton