fix: fixed an issue where only the first special character of each type would be escaped when building queries

zachary-D · zachary-D · commit cf5bb636af18 · 2020-04-10T23:24:55.000-04:00
Closes #25
diff --git a/lib/quoteString.js b/lib/quoteString.js
@@ -15,10 +15,10 @@ function QuoteString(string) {
     }
     string = string.toString();
     string = string.replace(/\\/g, '\\\\');
-    string = string.replace('\x00', '\\x00');
-    string = string.replace('\n', '\\n');
-    string = string.replace('\r', '\\r');
-    string = string.replace("'", "''");
+    string = string.replace(/\x00/g, '\\x00');
+    string = string.replace(/\n/g, '\\n');
+    string = string.replace(/\r/g, '\\r');
+    string = string.replace(/'/g, "''");
     return "'" + string + "'";
 }
 

Original file line number	Diff line number	Diff line change
`@@ -15,10 +15,10 @@ function QuoteString(string) {`
`15`	`15`	`}`
`16`	`16`	`string = string.toString();`
`17`	`17`	`string = string.replace(/\\/g, '\\\\');`
`18`		`- string = string.replace('\x00', '\\x00');`
`19`		`- string = string.replace('\n', '\\n');`
`20`		`- string = string.replace('\r', '\\r');`
`21`		`- string = string.replace("'", "''");`
	`18`	`+ string = string.replace(/\x00/g, '\\x00');`
	`19`	`+ string = string.replace(/\n/g, '\\n');`
	`20`	`+ string = string.replace(/\r/g, '\\r');`
	`21`	`+ string = string.replace(/'/g, "''");`
`22`	`22`	`return "'" + string + "'";`
`23`	`23`	`}`
`24`	`24`