Skip to content

ISSUES-86: fix arguments checked - #87

Closed
sergeevik wants to merge 1 commit into
dashjoin:mainfrom
sergeevik:bugfix/ISSUES-86
Closed

ISSUES-86: fix arguments checked#87
sergeevik wants to merge 1 commit into
dashjoin:mainfrom
sergeevik:bugfix/ISSUES-86

Conversation

@sergeevik

@sergeevik sergeevik commented Jan 27, 2026

Copy link
Copy Markdown
Contributor
  • fix comparison question symbol (it is a char, not a string)
  • fixed argument validation in 'partialApplyProcedure': null arguments are not passed to the unboundArgs array

@sergeevik
ISSUES-86: fix arguments checked
a2ca7cb 
- fix comparison question symbol (it is a char, not a string)
- fixed argument validation in 'partialApplyProcedure': null arguments are not passed to the unboundArgs array
@sergeevik

sergeevik commented Jan 27, 2026

Copy link
Copy Markdown
Contributor Author

fix of #86

aeberhart added a commit that referenced this pull request Nov 4, 2026
@aeberhart @claude
Add test cases from #80, #85, #87
dd89e91 
Tests contributed by @sergeevik in the earlier PRs, adapted to the
existing API. All pass with the partial application changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Nov 4, 2026
Closed
Merged
@aeberhart

aeberhart commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Thanks a lot @sergeevik for reporting #86 and for this PR! It is now fixed on main by #132: as in jsonata-js, only ? leaves a parameter unbound, and missing args are bound to undefined. Your test cases were added to OldPrsTest. Closing this one as superseded.

@aeberhart aeberhart closed this Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sergeevik @aeberhart