GitHub user justinchuch opened a pull request:
https://github.com/apache/tinkerpop/pull/934
Apply StringEscapeUtils in GroovyTranslator#convertToString(String)
Preface:
There's character escaping issue when translating.
What's changed:
-
`org.apache.tinkerpop.gremlin.groovy.jsr223.GroovyTranslator#convertToString`
Apply `StringEscapeUtils.escapeJava(String)` for String object.
- `org.apache.tinkerpop.gremlin.groovy.jsr223.GroovyTranslatorTest`
Add a few test cases to verify the changes.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/justinchuch/tinkerpop groovy-translator-tp32
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/934.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #934
----
commit 3416e52db8c2f81239a774e2dd17b09a9e471b14
Author: Justin Chu <15710241+justinchuch@...>
Date: 2018-09-21T19:12:26Z
Apply StringEscapeUtils in GroovyTranslator#convertToString(String)
----
---
