← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/fix-1046415 into lp:zorba

 

Review: Needs Information

- Is the empty string a valid name? Should we raise an error if the name of a pair is an empty string? Currently, the following query works fine: { "" : 1}("")
- What should happen if the key expression evaluates to null. Currently, the following query returns 1; { null : 1 }("null") (see test jsoniq/obj_constr_12.xq).
- I would like to get Paul's opinion on the error message. Maybe it can be made a bit more crisp without loosing information.
- I think the feature deserves more tests. I have committed a bunch into the branch.
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.


References