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

change type mapping for item() in function signatures #799

Open
kcoleman-marklogic opened this issue Jul 30, 2018 · 0 comments
Open

change type mapping for item() in function signatures #799

kcoleman-marklogic opened this issue Jul 30, 2018 · 0 comments

Comments

@kcoleman-marklogic
Copy link
Contributor

Currently, the code that generates SJS apidoc from the XQuery function defns maps item() and item()? to String and String?. This is wrong in most or all cases. See fn.string for an example. It says it takes String? as input, but that's not so.

We should probably change this to use a non-existent abstract type like Item instead.

The use of String is right out of the original SJS functional spec. I have a query out to the devs about why they thought this was a good idea. Will update this bug once I confirm whether or not my suggested fix is a bad idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant