Fox for returning trailing empty string on Google AppEngine (https://code.google.com/p/googleappengine/issues/detail?id=13565)

Marcel Overdijk · Marcel Overdijk · commit 7cfff744848c · 2017-02-19T22:04:32.000+01:00
diff --git a/rivescript-core/src/main/java/com/rivescript/util/StringUtils.java b/rivescript-core/src/main/java/com/rivescript/util/StringUtils.java
@@ -96,7 +96,9 @@ public static String join(String[] array, String delimiter) {
 	public static String quoteMetacharacters(String str) {
 		String[] unsafe = "\\.+*?[^]$(){}=!<>|:".split("");
 		for (String c : unsafe) {
-			str = str.replaceAll(Pattern.quote(c), "\\\\\\" + c);
+			if (c != null && c.length() > 0) {
+				str = str.replaceAll(Pattern.quote(c), "\\\\\\" + c);
+			}
 		}
 		return str;
 	}

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,9 @@ public static String join(String[] array, String delimiter) {`
`96`	`96`	`public static String quoteMetacharacters(String str) {`
`97`	`97`	`String[] unsafe = "\\.+*?[^]$(){}=!<>\|:".split("");`
`98`	`98`	`for (String c : unsafe) {`
`99`		`- str = str.replaceAll(Pattern.quote(c), "\\\\\\" + c);`
	`99`	`+ if (c != null && c.length() > 0) {`
	`100`	`+ str = str.replaceAll(Pattern.quote(c), "\\\\\\" + c);`
	`101`	`+ }`
`100`	`102`	`}`
`101`	`103`	`return str;`
`102`	`104`	`}`