s434730/RandomSec
3
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix unescape GREL function in javascript mode. Closes #1681.

Browse Source
This commit is contained in:
Antonin Delpeuch 2018-08-07 10:57:00 +01:00
parent d271b430ec
commit 87b6aadbad
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
main/src/com/google/refine/expr/functions/strings/Unescape.java
View File

@ -62,7 +62,7 @@ public class Unescape implements Function {
} else if ("csv".equals(mode)) { } else if ("csv".equals(mode)) {
return StringEscapeUtils.unescapeCsv(s); return StringEscapeUtils.unescapeCsv(s);
} else if ("javascript".equals(mode)) { } else if ("javascript".equals(mode)) {
return StringEscapeUtils.escapeEcmaScript(s); return StringEscapeUtils.unescapeEcmaScript(s);
} else if ("url".equals(mode)) { } else if ("url".equals(mode)) {
try { try {
return URLDecoder.decode(s,"UTF-8"); return URLDecoder.decode(s,"UTF-8");

6
main/tests/server/src/com/google/refine/tests/expr/functions/strings/ToFromConversionTests.java
View File

@ -178,4 +178,10 @@ public class ToFromConversionTests extends RefineTest {
Assert.assertEquals(invoke("escape", Double.parseDouble("1.23"), "javascript"),"1.23"); Assert.assertEquals(invoke("escape", Double.parseDouble("1.23"), "javascript"),"1.23");
} }
@Test
public void testUnescape() {
Assert.assertEquals(invoke("unescape", "Ä", "html"),"Ä");
Assert.assertEquals(invoke("unescape", "\\u00C4", "javascript"),"Ä");
}
} }