GitHub user justinchuch opened a pull request:
https://github.com/apache/tinkerpop/pull/936
Apply StringEscapeUtils in GroovyTranslator#convertToString(String) [tp33]
Preface:
There's character escaping issue when translating.
What's changed:
- Merge the changes from tp32
- org.apache.tinkerpop.gremlin.groovy.jsr223.GroovyTranslatorTest
Adjust the test case to verify the changes in tp33.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/justinchuch/tinkerpop groovy-translator-tp33
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/936.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 #936
----
commit aff2037b5973c62c0292ab3065761e923899aa57
Author: Justin Chu <15710241+justinchuch@...>
Date: 2018-09-21T19:12:26Z
Apply StringEscapeUtils in GroovyTranslator#convertToString(String)
commit 02e412d92d6f6c7004bdeef206e3d7adb9bd148a
Author: Justin Chu <15710241+justinchuch@...>
Date: 2018-09-24T15:41:36Z
Merge branch 'groovy-translator-tp32' into groovy-translator-tp33
commit 3f221968d79e2db6e170eec9285cb75d8cb203f0
Author: Justin Chu <15710241+justinchuch@...>
Date: 2018-09-24T16:44:17Z
Adjust test cases for tp33
----
---
