Project

Profile

Help

Feature #4991

closed

Do indentation of JSON with xml-to-json($json-xml, map { 'indent' : true() })

Added by Martin Honnen about 3 years ago. Updated 4 days ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Category:
-
Sprint/Milestone:
Start date:
2021-05-16
Due date:
% Done:

0%

Estimated time:
Applies to JS Branch:
2, Trunk
Fix Committed on JS Branch:
2, Trunk
Fixed in JS Release:
SEF Generated with:
Platforms:
Company:
-
Contact person:
-
Additional contact persons:
-

Description

The function xml-to-json takes an optional second argument with a map for options, the one being defined is a boolean value for the key indent.

While the Java and .NET product honour that option and do indent any produced JSON, Saxon-JS 2 doesn't do any indentation, see also https://saxonica.plan.io/boards/5/topics/8052?r=8053#message-8053.

It would be nice if the implementation could be enhanced to support indentation as otherwise reading the result of xml-to-json that Saxon-JS 2 produces is very hard as you are faced with one long line of JSON data.

Actions #2

Updated by Norm Tovey-Walsh about 2 years ago

  • Sprint/Milestone set to SaxonJS 3.0
Actions #3

Updated by Norm Tovey-Walsh 7 days ago

Indeed, the implementation of xml-to-json diligently checks that the indent option is a boolean then...never mentions it again. :-(

Actions #6

Updated by Norm Tovey-Walsh 4 days ago

  • Status changed from New to Resolved
  • Applies to JS Branch Trunk added
  • Fix Committed on JS Branch 2, Trunk added

I've added support for the indent option. It's not exactly what JavaScript does when asked to produce indented JSON, but it certainly introduces sufficient line breaks and indentation to be readable.

Please register to edit this issue

Also available in: Atom PDF Tracking page