-
Notifications
You must be signed in to change notification settings - Fork 13
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
Seed to randomize should not require an integer #49
Labels
Milestone
Comments
This was referenced Oct 14, 2024
brontolosone
added a commit
to brontolosone/web-forms
that referenced
this issue
Oct 14, 2024
brontolosone
added a commit
to brontolosone/web-forms
that referenced
this issue
Oct 14, 2024
brontolosone
added a commit
to brontolosone/web-forms
that referenced
this issue
Oct 14, 2024
2 tasks
brontolosone
added a commit
to brontolosone/web-forms
that referenced
this issue
Oct 15, 2024
brontolosone
added a commit
to brontolosone/web-forms
that referenced
this issue
Oct 15, 2024
For blank case, add test to original JR implementation and match behavior; might want to specify in spec |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Per spec,
randomize
accepts an optional seed. The current implementation currently checks that the seed is an integer, and throws (a string, a tangentially related my-bad) if it's not. This was derived from the implementation in OpenRosa XPath Evaluator, so I'd expect this bug affects Enketo as well. Which is somewhat surprising, because I discovered the bug by using XLSForms' prominent example for stable-randomization.Steps to reproduce
Expected behavior
randomize
will stably randomize options, without errorNaN
or""
?), but this would potentially involve spec clarificationObserved behavior
(From memory, happy to repro again if needed) Form load fails without explanation.
The text was updated successfully, but these errors were encountered: