Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BUG: closing parenthesis ")" breaks parsing arguments #57

Closed
yghern opened this issue Apr 26, 2019 · 1 comment
Closed

BUG: closing parenthesis ")" breaks parsing arguments #57

yghern opened this issue Apr 26, 2019 · 1 comment

Comments

@yghern
Copy link

yghern commented Apr 26, 2019

Steps:

  1. Create field random with subfields that contain closing parenthesis, e.g.
    "function_name": "random('function1()', 'function2()')"

Expected:
The field should have been filled with one of the provided values, e.g. function1() or function2()

Actual:
Parsing argument breaks after first closing parenthesis disregarding any quoting, .e.g. 'function1(
yghern pushed a commit to yghern/json-data-generator that referenced this issue Apr 26, 2019
Fix for issue everwatchsolutions#57: parsing arguments after ) sign
188cc5d
@yghern
Copy link
Author

yghern commented Apr 26, 2019

Fixed in: #58

andrewserff added a commit that referenced this issue Sep 24, 2019
@andrewserff
Merge pull request #58 from yghern/master
f569d7d 
Fix for issue #57: parsing arguments after ) sign
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@andrewserff @yghern