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.

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

Fix handling of nulls passed via context #46

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

rayokota
Copy link
Contributor

@rayokota rayokota commented Aug 3, 2024

This is a port of the corresponding fix in the Python implementation of JSONata, see this.

The fix allows a null obtained from the context to be properly handled, so that the following two expressions return the same result (where a is undefined):

     jsonata("$split(a, '-')").evaluate(Map.of());

     jsonata("a ~> $split('-')").evaluate(Map.of());

@aeberhart aeberhart merged commit 35c4812 into dashjoin:main Sep 18, 2024
1 check passed
@uw4
Copy link
Contributor

uw4 commented Sep 18, 2024

Thanks a lot for backporting! Sorry it took some time to merge, preparing for new release...

@uw4 uw4 added this to the 0.9.8 milestone Oct 4, 2024
@uw4 uw4 added the bug Something isn't working label Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants