text serialization with jsoniq fails
Bug #1034990 reported by
Dennis Knochenwefel
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Zorba |
Fix Released
|
High
|
Matthias Brantner | ||
Bug Description
example query:
-------
import schema namespace output = "http://
serialize( { "message": "test" },
<output:
<
<
</output:
-------
returns:
"getNodeKind" not implemented for item type "jdm:object"; raised
at /home/pilot/
http://
Related branches
lp:~zorba-coders/zorba/bug-1034990
- Till Westmann: Approve
- Matthias Brantner: Approve
-
Diff: 235 lines (+114/-1)10 files modifiedChangeLog (+1/-0)
include/zorba/pregenerated/diagnostic_list.h (+2/-0)
modules/org/jsoniq/www/pregenerated/errors.xq (+7/-0)
src/api/serialization/serializer.cpp (+45/-1)
src/api/serialization/serializer.h (+7/-0)
src/diagnostics/diagnostic_en.xml (+7/-0)
src/diagnostics/pregenerated/diagnostic_list.cpp (+3/-0)
src/diagnostics/pregenerated/dict_en.cpp (+3/-0)
test/rbkt/ExpQueryResults/zorba/jsoniq/serializer-JNSE0022.xml.res (+1/-0)
test/rbkt/Queries/zorba/jsoniq/serializer-JNSE0022.xq (+38/-0)
| tags: | added: jsoniq |
Revision history for this message
| Matthias Brantner (matthias-brantner) wrote | #1 |
| Changed in zorba: | |
| assignee: | Ghislain Fourny (gislenius) → Matthias Brantner (matthias-brantner) |
| milestone: | none → 2.7 |
Revision history for this message
| Matthias Brantner (matthias-brantner) wrote | #2 |
| Changed in zorba: | |
| status: | New → In Progress |
| importance: | Undecided → High |
| Changed in zorba: | |
| status: | In Progress → Fix Committed |
| Changed in zorba: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The problem is that the text serializer is not able to deal with JSON items.
We need to change the spec saying that it's an error if the text serialization encounters a JSON item.
After the spec was changed, this needs to be implemented.
Also, for usability, we need to make sure that
- fn:trace serialization should use the hybrid serialization method
- the default serialization method is hybrid