x_ite icon indicating copy to clipboard operation
x_ite copied to clipboard

NavigationInfo @type is a JSON String not a JSON array. Data from browser.toJSONString()

Open coderextreme opened this issue 1 year ago • 7 comments

image

Talk to Don and Vince if you disagree. I will back you, but it may require a few changes...

coderextreme avatar Aug 24 '23 13:08 coderextreme

It's also possible to accept either in JSON Schema. I will work on that too. I'll broach the discussion.

coderextreme avatar Aug 24 '23 13:08 coderextreme

The issue is not whether NavigationInfo.type should be coded differently, but whether MFString values should be coded differently, and that will probably not be the case.

create3000 avatar Aug 24 '23 14:08 create3000

I agree, but no one wants to discuss it frankly or honestly, they just avoid, avoid, avoid. So I'm looking to supporting both in JSON schema, just for NavigationInfo @type. I agree that all MFStrings should be coded the same.

coderextreme avatar Aug 24 '23 14:08 coderextreme

JSON schema has been tested, but I haven't changed the schema generation script yet.

coderextreme avatar Aug 24 '23 14:08 coderextreme

If you change MFString implementation, let me know. I am currently assuming that MFStrings are arrays. I have 2 branches of code, one, .json, with NavigationInfo.type as a string. The other, .x3dj as an array. I can merely substitute array for string in the schema if and when the time is right.

coderextreme avatar Sep 03 '23 01:09 coderextreme

Current X3dtoJson.xslt documentation has NavigatorInfo@type as an array.

coderextreme avatar Sep 12 '23 06:09 coderextreme

I've got changes to X3dToJson.xslt if you are interested. I will be pushing this through my pipelines.

coderextreme avatar Mar 31 '24 22:03 coderextreme