Skip to content

Fix nested key deletion #34

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.

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

Merged
merged 1 commit into from
Feb 9, 2020
Merged

Fix nested key deletion #34

merged 1 commit into from
Feb 9, 2020

Conversation

nalundgaard
Copy link
Owner

Resolves #33.

Adds a clause for the jsn__delete value to short-circuit the automatic path creation used in the value-setting path. For backward compatibility, I implemented an internal return_if_object/2 function to preserve throwing not_an_object errors that happen when you try to delete binary keys from arrays, etc.

Resolves #33.

Adds a clause for the `jsn__delete` value to short-circuit the automatic path creation used in the value-setting path. For backward compatibility, I implemented an internal `return_if_object/2` function to preserve throwing `not_an_object` errors that happen when you try to delete binary keys from arrays, etc.
@jadeallenx
Copy link
Collaborator

:shipit:

@nalundgaard nalundgaard merged commit 4b66ab4 into master Feb 9, 2020
@nalundgaard nalundgaard deleted the delete_nested branch February 9, 2020 17:34
nalundgaard added a commit that referenced this pull request Feb 9, 2020
* Fix nested key deletion (#34)
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.

delete adds entries when using deep paths
2 participants