Bug #2692: Incorrect output from xml-to-json function, for an empty array inside an array - Saxon - Saxonica Developer Community

Actions

Send by e-mail Copy link

Bug #2692

closed

Incorrect output from xml-to-json function, for an empty array inside an array

Added by Debbie Lockett over 8 years ago. Updated over 8 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Debbie Lockett

Category:

XSLT conformance

Sprint/Milestone:

9.7.0.5

Start date:

2016-04-01

Due date:

% Done:

100%

Estimated time:

Legacy ID:

Applies to branch:

9.7

Fix Committed on Branch:

9.7

Fixed in Maintenance Release:

9.7.0.5

Platforms:

Description

Raised by a user on the support mailing list.

Applying the xml-to-json() function to the following input:

<number>3</number>

<array/>

<number>4</number>

Returns the JSON output:

[3,[]4]

i.e. there is a comma separator missing after the empty array.

Actions

Copy link

Updated by Michael Kay over 8 years ago

Also applies where an empty map is output within an array.

Actions

Copy link

Updated by Debbie Lockett over 8 years ago

Status changed from New to In Progress
Assignee set to Debbie Lockett

Actions

Copy link

Updated by Debbie Lockett over 8 years ago

Tests xml-to-json-A014, xml-to-json-A015, xml-to-json-B014, xml-to-json-B015 added to the XSLT 3.0 test suite. Testing both empty arrays and empty maps inside an array.

Actions

Copy link

Updated by Debbie Lockett over 8 years ago

Status changed from In Progress to Resolved
Fix Committed on Branch 9.7 added

The problem was that JsonReceiver was not setting atStart to false after processing an empty map or array.

Fix made to JsonReceiver.endElement() to set atStart=false unconditionally.

Patch committed on 9.7 and dev branches.

Actions

Copy link

Updated by Debbie Lockett over 8 years ago

Another user has pointed out that the same bug affects maps containing an empty map or array as the value of a pair (which is not the last).

e.g. xml-to-json(json-to-xml({A":{},"B":1})) returns {"A":{}"B --- with a missing comma.

The patch already committed for this bug fixes this issue. Further new tests (xml-to-json-A017, A018, B017, B018) have been added to the XSLT 3.0 test suite.

Actions

Copy link