Fix sanitization logic

Borrowed from https://github.com/sindresorhus/escape-string-regexp Fixes VerbalExpressions#352
shreyasminocha · Jul 10, 2022 · a228f1f · a228f1f
1 parent 0e0f1fb
commit a228f1f
Show file tree

Hide file tree

Showing 4 changed files with 578 additions and 616 deletions.
diff --git a/VerbalExpressions.js b/VerbalExpressions.js
@@ -54,13 +54,15 @@ class VerbalExpression extends RegExp {
 
         // Regular expression to match meta characters
         // https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/regexp
-        const toEscape = /([\].|*?+(){}^$\\:=[])/g;
+        const toEscape = /[|\\{}()[\]^$+*?.]/g;
 
         // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/lastMatch
         const lastMatch = '$&';
 
         // Escape meta characters
-        return value.replace(toEscape, `\\${lastMatch}`);
+        return value
+            .replace(toEscape, `\\${lastMatch}`)
+            .replace(/-/g, '\\x2d');
     }
 
     /**