marshalString fails

[pgiraud]

I tried to marshal back an unmarshaled xml document without any success.
https://github.com/pgiraud/ogc-schemas/blob/owscontext_marshal/scripts/tests/OWC/0.3.1/OWC_0_3_1.js#L25-L28

I get the following error:

Error: Argument [undefined] must be an object.
    at Object.Jsonix.Util.Ensure.ensureObject (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:678:10)
    at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalElementNode (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5081:22)
    at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalDocument (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5071:8)
    at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalString (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5060:18)
    at /home/pierre/public_html/ogc-schemas/scripts/tests/OWC/0.3.1/OWC_0_3_1.js:26:15
    at /home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5145:5
    at fs.js:271:14
    at Object.oncomplete (fs.js:107:15)

I haven't figured out when the error appears exactly. Using a similar code in the browser and iterating through breakpoints didn't help me. ensureObject is called several times.

Any help would be appreciated.

[highsource]

Please marshal result, not result.value. result defines the root element
with name which is a qname of the element. value is a content.

On Fri, Sep 5, 2014 at 4:58 PM, Pierre GIRAUD notifications@github.com
wrote:

I tried to marshal back an unmarshaled xml document without any success.

https://github.com/pgiraud/ogc-schemas/blob/owscontext_marshal/scripts/tests/OWC/0.3.1/OWC_0_3_1.js#L25-L28

I get the following error:

Error: Argument [undefined] must be an object.
at Object.Jsonix.Util.Ensure.ensureObject (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:678:10)
at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalElementNode (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5081:22)
at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalDocument (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5071:8)
at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalString (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5060:18)
at /home/pierre/public_html/ogc-schemas/scripts/tests/OWC/0.3.1/OWC_0_3_1.js:26:15
at /home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5145:5
at fs.js:271:14
at Object.oncomplete (fs.js:107:15)

I haven't figured out when the error appears exactly. Using a similar code
in the browser and iterating through breakpoints didn't help me.
ensureObject is called several times.

Any help would be appreciated.

—
Reply to this email directly or view it on GitHub
#11.

[highsource]

BTW marshalString gives the result back, it is synchron.

On Fri, Sep 5, 2014 at 5:40 PM, Aleksei Valikov aleksei.valikov@gmail.com
wrote:

Please marshal result, not result.value. result defines the root element
with name which is a qname of the element. value is a content.

On Fri, Sep 5, 2014 at 4:58 PM, Pierre GIRAUD notifications@github.com
wrote:

I tried to marshal back an unmarshaled xml document without any success.

https://github.com/pgiraud/ogc-schemas/blob/owscontext_marshal/scripts/tests/OWC/0.3.1/OWC_0_3_1.js#L25-L28

I get the following error:

Error: Argument [undefined] must be an object.
at Object.Jsonix.Util.Ensure.ensureObject (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:678:10)
at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalElementNode (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5081:22)
at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalDocument (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5071:8)
at Object.Jsonix.Context.Marshaller.Jsonix.Class.marshalString (/home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5060:18)
at /home/pierre/public_html/ogc-schemas/scripts/tests/OWC/0.3.1/OWC_0_3_1.js:26:15
at /home/pierre/public_html/ogc-schemas/scripts/node_modules/jsonix/jsonix.js:5145:5
at fs.js:271:14
at Object.oncomplete (fs.js:107:15)

I haven't figured out when the error appears exactly. Using a similar
code in the browser and iterating through breakpoints didn't help me.
ensureObject is called several times.

Any help would be appreciated.

—
Reply to this email directly or view it on GitHub
#11.

[highsource]

https://github.com/highsource/ogc-schemas/blob/master/scripts/tests/OWC/0.3.1/OWC_0_3_1.js#L24-L25

[pgiraud]

OK, stupid me.
One further question though. In the console the document generated by the marshaler start with:

<owc:OWSContext version="0.3.1" id="ows-context-ex-1-v3"><p0:General>[...]

See the p0 prefix for General? Is it an intended result?
Also shouldn't the namespaces be defined in the root node?

[highsource]

p0 is not quite intended, prefix mappings don't fully work at the moment.
There are two feature related to that:
*
http://confluence.highsource.org/display/JSNX/User+Guide#UserGuide-CreatingJsonixContext

• this does not seem to work at the moment
• namespace mappings could also be derived from the data itself.
  Please file the issues.

As for the namespace on the root node - it IS actually defined in the DOM,
but XMLDOM probably has a serializer bug:
jindw/xmldom#97

On Fri, Sep 5, 2014 at 6:02 PM, Pierre GIRAUD notifications@github.com
wrote:

OK, stupid me.
One further question though. In the console the document generated by the
marshaler start with:

<owc:OWSContext version="0.3.1" id="ows-context-ex-1-v3">p0:General[...]

See the p0 prefix for General? Is it an intended result?
Also shouldn't the namespaces be defined in the root node?

—
Reply to this email directly or view it on GitHub
#11 (comment)
.

[pgiraud]

I don't know whether I should file an issue in Jsonix or here.

[pgiraud]

It may help to know that while the prefix is incorrect for General, it is correct for its BoundingBox child node.

[highsource]

File an issue in Jsonix, I'll investigate.

Am 08.09.2014 um 12:12 schrieb Pierre GIRAUD notifications@github.com:

It may help to know that while the prefix is incorrect for General, it is correct for its BoundingBox child node.

—
Reply to this email directly or view it on GitHub.

[highsource]

This should be fixed in Jsonix 2.1.0.

[pgiraud]

Looks much better!

There are still some remaining "pX" prefixes though. It seems valid however since each time a "pX" prefix is used, the corresponding xmlns attribute is declared in the node.

In our case, p0 actually corresponds to a namespace that has already been declared. It would be prettier to use it.

By the way, thanks again for all your great work.

[highsource]

Please PR the example which produces pX-prefixes, I'll investigate.
Actually it should use the already declared prefix.
In the next version you could declare all the prefixes you want to use, see
this version:
highsource/jsonix#28

(Yet to be released.)

On Mon, Sep 15, 2014 at 9:10 AM, Pierre GIRAUD notifications@github.com
wrote:

Looks much better!

There are still some remaining "pX" prefixes though. It seems valid
however since each time a "pX" prefix is used, the corresponding xmlns
attribute is declared in the node.

In our case, p0 actually corresponds to a namespace that has already been
declared. It would be prettier to use it.

By the way, thanks again for all your great work.

—
Reply to this email directly or view it on GitHub
#11 (comment)
.

[pgiraud]

No need for a PR ;-) The code is already there.
https://github.com/highsource/ogc-schemas/blob/master/scripts/tests/OWC/0.3.1/OWC_0_3_1.js#L27

[highsource]

Ok, I'll check.

On Mon, Sep 15, 2014 at 9:48 AM, Pierre GIRAUD notifications@github.com
wrote:

No need for a PR ;-) The code is already there.

https://github.com/highsource/ogc-schemas/blob/master/scripts/tests/OWC/0.3.1/OWC_0_3_1.js#L27

—
Reply to this email directly or view it on GitHub
#11 (comment)
.

[highsource]

highsource/jsonix#30