fix: avoid escaping unnecessary chars in HTML report suppression regexes (#6749)

chadlwilson · web-flow · commit 9b42aedc376e · 2024-06-30T05:34:50.000-04:00
Signed-off-by: Chad Wilson &lt;chadw@thoughtworks.com&gt;
diff --git a/core/src/main/resources/templates/htmlReport.vsl b/core/src/main/resources/templates/htmlReport.vsl
@@ -152,7 +152,7 @@ Copyright (c) 2012 Jeremy Long. All Rights Reserved.
                 }
             }
             function escapeRegExp(text) {
-              return text.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, '\\$&');
+              return text.replace(/[[\]{}()*+?.\\^$|\s]/g, '\\$&');
             }
             function setCopyText(name, matchType, matchValue, suppressType, suppressVal) {
                 xml =  '<suppress>\n';

Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,7 @@ Copyright (c) 2012 Jeremy Long. All Rights Reserved.`
`152`	`152`	`}`
`153`	`153`	`}`
`154`	`154`	`function escapeRegExp(text) {`
`155`		`- return text.replace(/[-[\]{}()*+?.,\\^$\|#\s]/g, '\\$&');`
	`155`	`+ return text.replace(/[[\]{}()*+?.\\^$\|\s]/g, '\\$&');`
`156`	`156`	`}`
`157`	`157`	`function setCopyText(name, matchType, matchValue, suppressType, suppressVal) {`
`158`	`158`	`xml = '<suppress>\n';`