Zorba

XQuery grammar needs EQNames

Bug #891453 reported by Paul J. Lucas
This bug report is a duplicate of:  Bug #997045: EQName regressions. Edit Remove
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zorba
Fix Committed
Medium
Matthias Brantner
Zorba 2.7 "Gaia"

Bug Description

According to Matthias, the XQuery grammar needs to be extended to support EQNames. There also needs to be an AtonicOrUntionType production.

Tags: parser task
Paul J. Lucas (paul-lucas)
Changed in zorba:
assignee: nobody → Nicolae Brinza (nbrinza)
Revision history for this message
Matthias Brantner (matthias-brantner) wrote :

Specificially, the ItemType production currently only includes the AtomicType production which should be extended to support AtomicOrUnionType.

Changed in zorba:
milestone: none → 2.2
Paul J. Lucas (paul-lucas)
Changed in zorba:
importance: Undecided → Medium
Nicolae Brinza (nbrinza)
Changed in zorba:
milestone: 2.2 → 2.5
Matthias Brantner (matthias-brantner)
Changed in zorba:
milestone: 2.5 → none
Dana Florescu (dflorescu)
Changed in zorba:
milestone: none → 2.7
Chris Hillery (ceejatec)
tags: added: task
Revision history for this message
Nicolae Brinza (nbrinza) wrote :

1. The EQNames grammar is implemented. The new syntax is also implemented in a branch, waiting to be merged.

2. The AtomicOrUnionType production is actually implemented indirectly through the following rules from xquery_parser.y:

ItemType ::= AtomicType

AtomicType ::= QNAME

QNAME ::= QNAME_SVAL
                      | EQNAME

, so there isn't anything that needs to be done, as far as I understand.

Matthias, could you please double-check, confirm and then close the bug?

Changed in zorba:
assignee: Nicolae Brinza (nbrinza) → Matthias Brantner (matthias-brantner)
status: New → Invalid
Revision history for this message
Chris Hillery (ceejatec) wrote :

"Invalid" actually is a closed state. Setting back to Confirmed until Matthias can double-check.

Changed in zorba:
status: Invalid → Confirmed
Revision history for this message
Matthias Brantner (matthias-brantner) wrote :

the EQName issue was fixed with merging the branch at lp:~nbrinza/zorba/error-messages.
I also confirm that the AtomicOrUnionType production is there.

Changed in zorba:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.