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

2012-09-13 Thread Ghislain Fourny 
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Ghislain Fourny 
Hi Cezar,

In general, an object constructor expression is not always valid JSON, as it is 
more general. For example, { 1 : 1 } is valid JSONiq although it is not valid 
JSON. A key expression, can be anything, and will be atomized and cast to 
xs:string.

Also, { null : 1 } (which is what is produced) is valid JSON, just like { 1 
: 1 }.

Accepting { null : 1 } in the same way makes the specification simpler, as we 
need not special-case.

Does it make sense?
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Cezar Andrei 
I understand, it make sense in this way but it's very dangerous, in this
way I don't think it helps the users on contrary can leave bad code falsely
working.

Cezar

On Thu, Sep 13, 2012 at 6:38 AM, Ghislain Fourny ghislain.fou...@28msec.com
 wrote:

 Hi Cezar,

 In general, an object constructor expression is not always valid JSON, as
 it is more general. For example, { 1 : 1 } is valid JSONiq although it is
 not valid JSON. A key expression, can be anything, and will be atomized and
 cast to xs:string.

 Also, { null : 1 } (which is what is produced) is valid JSON, just like
 { 1 : 1 }.

 Accepting { null : 1 } in the same way makes the specification simpler, as
 we need not special-case.

 Does it make sense?
 --
 https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
 Your team Zorba Coders is subscribed to branch lp:zorba.

 --
 Mailing list: https://launchpad.net/~zorba-coders
 Post to : zorba-coders@lists.launchpad.net
 Unsubscribe : https://launchpad.net/~zorba-coders
 More help   : https://help.launchpad.net/ListHelp


-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/fix-1046415-2012-09-13T14-49-49.139Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The attempt to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job fix-1046415-2012-09-13T14-49-49.139Z is finished.  The
  final status was:

  

  1 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Ghislain Fourny 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/fix-1046415-2012-09-13T15-19-50.542Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Validation queue job fix-1046415-2012-09-13T15-19-50.542Z is finished. The 
final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Voting does not meet specified criteria. Required: Approve  1, Disapprove  1, 
Needs Fixing  1, Pending  1. Got: 2 Approve, 1 Pending.
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Chris Hillery 
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Chris Hillery 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/fix-1046415-2012-09-13T20-14-47.795Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The attempt to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job fix-1046415-2012-09-13T20-14-47.795Z is finished.  The
  final status was:

  

  1 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Matthias Brantner 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/fix-1046415-2012-09-13T20-54-55.036Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The attempt to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job fix-1046415-2012-09-13T20-54-55.036Z is finished.  The
  final status was:

  

  1 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Matthias Brantner 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/fix-1046415-2012-09-14T04-04-53.838Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread Zorba Build Bot 
Validation queue job fix-1046415-2012-09-14T04-04-53.838Z is finished. The 
final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-13 Thread noreply 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-12 Thread Ghislain Fourny 
Hi Matthias,

- I think the empty string is a valid JSON name.
- The current semantics of { null : 1 }(null) in the spec is indeed to return 
1. I think I am fine with this.
- I uploaded more tests (somehow they did not get uploaded yesterday).

-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-12 Thread Matthias Brantner 
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-11 Thread Matthias Brantner 
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.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-11 Thread Chris Hillery 
As far as I can make out, an empty string is a valid key for an object. 
jsonlint.com at least agrees.

null is not a valid key, and we should raise an error if such a case arises. 
Also, the fact that using the *string* null as an accessor worked in that 
case frightens me.
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-11 Thread Matthias Brantner 
 As far as I can make out, an empty string is a valid key for an object. 
 jsonlint.com at least agrees.
Fair enough.

 null is not a valid key, and we should raise an error if such a case arises. 
 Also, the fact that using the *string* null as an accessor worked in that 
 case frightens me.
According to what we said, null is cast to a string. That's why the accessor 
works. I don't see anything being
particularly wrong with that. I just wanted to raise it to make sure we are all 
on the same page.

Matthias
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-07 Thread Ghislain Fourny 
Ghislain Fourny has proposed merging lp:~zorba-coders/zorba/fix-1046415 into 
lp:zorba.

Requested reviews:
  Chris Hillery (ceejatec)
  Matthias Brantner (matthias-brantner)
Related bugs:
  Bug #1046415 in Zorba: jsoniq: pair name needs to cast to string
  https://bugs.launchpad.net/zorba/+bug/1046415

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283

Fixes bug 1046415.
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/compiler/translator/translator.cpp'
--- src/compiler/translator/translator.cpp	2012-08-31 12:40:20 +
+++ src/compiler/translator/translator.cpp	2012-09-07 14:10:24 +
@@ -11364,12 +11364,10 @@
   nameExpr = wrap_in_atomization(nameExpr);
 
   nameExpr = theExprManager-
-  create_promote_expr(theRootSctx,
-  nameExpr-get_loc(),
-  nameExpr,
-  GENV_TYPESYSTEM.STRING_TYPE_ONE,
-  PromoteIterator::JSONIQ_PAIR_NAME, // JNTY0001
-  NULL);
+  create_cast_expr(theRootSctx,
+   nameExpr-get_loc(),
+   nameExpr,
+   GENV_TYPESYSTEM.STRING_TYPE_ONE);
 
   valueExpr = theExprManager-
   create_treat_expr(theRootSctx,

=== modified file 'test/rbkt/Queries/zorba/jsoniq/jnty0001-1.spec'
--- test/rbkt/Queries/zorba/jsoniq/jnty0001-1.spec	2012-06-19 14:58:25 +
+++ test/rbkt/Queries/zorba/jsoniq/jnty0001-1.spec	2012-09-07 14:10:24 +
@@ -1,1 +1,1 @@
-Error: http://www.jsoniq.org/errors:JNTY0001
+Error: http://www.w3.org/2005/xqt-errors:XPTY0004

=== removed file 'test/rbkt/Queries/zorba/jsoniq/jnty0001-2.spec'
--- test/rbkt/Queries/zorba/jsoniq/jnty0001-2.spec	2012-06-19 14:58:25 +
+++ test/rbkt/Queries/zorba/jsoniq/jnty0001-2.spec	1970-01-01 00:00:00 +
@@ -1,1 +0,0 @@
-Error: http://www.jsoniq.org/errors:JNTY0001

=== removed file 'test/rbkt/Queries/zorba/jsoniq/jnty0001-2.xq'
--- test/rbkt/Queries/zorba/jsoniq/jnty0001-2.xq	2012-06-19 14:58:25 +
+++ test/rbkt/Queries/zorba/jsoniq/jnty0001-2.xq	1970-01-01 00:00:00 +
@@ -1,5 +0,0 @@
-copy $o := { foo : bar }
-modify (
-  insert json { 1 : bar } into $o
-)
-return $o
\ No newline at end of file

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-07 Thread Ghislain Fourny 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-07 Thread Matthias Brantner 
Review: Needs Fixing

It works great for the usual cases, e.g. if the name is an integer.

However, the following query raises a strange error:

{ { foo : bar}  : foo }

[jerr:JNTY0004]: object items do not have typed value. This might be hard to 
understand by users.
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-07 Thread Zorba Build Bot 
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/fix-1046415-2012-09-07T22-57-28.435Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-07 Thread Zorba Build Bot 
Voting does not meet specified criteria. Required: Approve  1, Disapprove  1, 
Needs Fixing  1, Pending  1. Got: 1 Needs Fixing, 1 Pending.
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

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

2012-09-07 Thread Zorba Build Bot 
The proposal to merge lp:~zorba-coders/zorba/fix-1046415 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-1046415/+merge/123283
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp