Should parse long number string as string instead of number

Checklist

• Have you asked your question on Stackoverflow or similar forum?
• Are you running the latest version?
• Have you included sample input, output, error, and expected output?
• Have you checked if you are using correct configuration?
• Did you try online tool?

Input

<?xml version="1.0" encoding="UTF-8"?>
<Root>
  <LongNumberString>420926189200190257681175017717</LongNumberString>
</Root>

Code

const { getTraversalObj, convertToJson } = require('fast-xml-parser');

const options = { attributeNamePrefix: '$', ignoreAttributes: false };

const x2j = xml => convertToJson(getTraversalObj(xml, options), options);

const xml = `<?xml version="1.0" encoding="UTF-8"?>
<Root>
  <LongNumberString>420926189200190257681175017717</LongNumberString>
</Root>
`;

const json = x2j(xml);

console.log(json);

Output

{ Root: { LongNumberString: 4.209261892001902e+29 } }

expected data

{ Root: { LongNumberString: "420926189200190257681175017717" } }

Would you like to work on this issue?

• Yes
• No

Bookmark this repository for further updates.

Description

I can help to enhance the parser if you think this is correct. IMO, the scientific notation can not represent the value in the XML. The type should be string if the number string is too long (more than Number.MAX_SAFE_INTEGER).

[amitguptagwl]

@ShinyChang please use option parseTrueNumberOnly : true

[amitguptagwl]

Thanks for the suggestion. Will implement it in a future version.