Skip to content

Conversation

@kunwar24
Copy link

In my case, I needed to pass a string as a parameter that contained values such as "(test)". The code was failing as it terminated the search for arguments at the first ")".

@robtweed
Copy link
Owner

I can see that this helps for your specific case where the function arguments might, themselves, contain a ), but of course the argument list will also get messed up if they contain a comma. It's probably better if this logic is modified to cater for all the potential scenarios. Perhaps the ideas here could be used: https://stackoverflow.com/questions/36902396/get-function-parameters-from-a-js-function-which-is-a-string ?

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.

2 participants