Merge pull request #297 from pragatheeswarans/fix-splitcharacter-regexp

RichardLindhout · web-flow · commit df50c8bf006d · 2023-06-22T17:10:40.000+02:00
Fix for passing dot as splitcharacter to RegExp constructor
diff --git a/src/TextInputMask.tsx b/src/TextInputMask.tsx
@@ -7,6 +7,10 @@ function detectCharacter(mask: string): string {
   return c || ''
 }
 
+function escapeForRegExp(value: string): string {
+  return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+}
+
 function TextInputWithMask(
   {
     onChangeText,
@@ -40,7 +44,7 @@ function TextInputWithMask(
     )
     const replaceValue = format.match(/\W/)
     const replace = `$1${splitCharacter}$2${splitCharacter}$3$4`.replace(
-      new RegExp(splitCharacter, 'g'),
+      new RegExp(escapeForRegExp(splitCharacter), 'g'),
       (replaceValue ?? '') as string
     )
 

Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,10 @@ function detectCharacter(mask: string): string {`
`7`	`7`	`return c \|\| ''`
`8`	`8`	`}`
`9`	`9`
	`10`	`+function escapeForRegExp(value: string): string {`
	`11`	`+ return value.replace(/[.*+?^${}()\|[\]\\]/g, '\\$&')`
	`12`	`+}`
	`13`	`+`
`10`	`14`	`function TextInputWithMask(`
`11`	`15`	`{`
`12`	`16`	`onChangeText,`
`@@ -40,7 +44,7 @@ function TextInputWithMask(`
`40`	`44`	`)`
`41`	`45`	`const replaceValue = format.match(/\W/)`
`42`	`46`	const replace = `$1${splitCharacter}$2${splitCharacter}$3$4`.replace(
`43`		`- new RegExp(splitCharacter, 'g'),`
	`47`	`+ new RegExp(escapeForRegExp(splitCharacter), 'g'),`
`44`	`48`	`(replaceValue ?? '') as string`
`45`	`49`	`)`
`46`	`50`