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

Update fail on comments while value contains single (double?) quotes #154

Closed
dthonon opened this issue May 4, 2021 · 4 comments
Closed

Update fail on comments while value contains single (double?) quotes #154

dthonon opened this issue May 4, 2021 · 4 comments
Labels
bug Something isn't working

Comments

@dthonon
Copy link
Owner

dthonon commented May 4, 2021

In GitLab by @fred.lpo07 on May 4, 2021, 09:44

The update process fails when the value contains the same characters as the value delimiters.

ex:

site,id_universal,path,operation,value
site1,123456,$['data']['sightings'][0]['observers'][0]['comment'],replace,'It's first time in that place'
@dthonon
Copy link
Owner Author

dthonon commented Jun 17, 2021

In GitLab by @jgc_lpoaura on Jun 17, 2021, 15:05

est-ce qu'il y a une solution pour contourner ce problème ?

@dthonon
Copy link
Owner Author

dthonon commented Oct 10, 2021

mentioned in commit 9b0d08f

@dthonon
Copy link
Owner Author

dthonon commented Oct 10, 2021

Fixed in 9b0d08f.
Note that value must be enclosed in double-quote: ".

@dthonon
Copy link
Owner Author

dthonon commented Oct 24, 2021

mentioned in commit 4f34adc

@dthonon dthonon closed this as completed May 5, 2022
dthonon added a commit that referenced this issue May 6, 2022
@dthonon
Merge branch 'develop' into 'master'
4f34adc 
Develop into master for v2.9.0

Closes #173, #68, and #154

See merge request lpo/Client_API_VN!66
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dthonon