Skip to content

fixed array escaping#2

Open
dloose wants to merge 1 commit intobcarrell:masterfrom
dloose:master
Open

fixed array escaping#2
dloose wants to merge 1 commit intobcarrell:masterfrom
dloose:master

Conversation

@dloose
Copy link

@dloose dloose commented Oct 4, 2016

I noticed the arrays don't get escaped properly if you have an escape function. The wrapQuotes function does map over the elements of the array, so something like [1, 2, 3] becomes "(1, 2, 3)". Then the the escape function was applied to the result string, yielding something like '\"(1, 2, 3)\"'. I munged together a solution that works in my limited testing. Not saying it's the best way to do it, but hopefully it helps illustrate the problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant