search

NaturalIntelligence / fast-xml-parser

Validate XML, Parse XML and Build XML rapidly without C/C++ based libraries and no callback.
https://naturalintelligence.github.io/fast-xml-parser/
MIT License
2.43k stars 297 forks source link

fix: get oneListGroup to work as expected for array of strings #662

Closed a-rasin closed 3 days ago

a-rasin commented 1 week ago

If I have oneListGroup enabled I would expect that:

{
  "a": ["(first)", "second"]
}

Would give me:

<a>(first)(second)</a>

But I get:

<a>
  <a>(first)</a>
  <a>(second)</a>
</a>

This commit fixes that issue.

Type

Please mention the type of PR

Note : Please ensure that you've read contribution guidelines before raising this PR. If your PR is in progress, please prepend [WIP] in PR title. Your PR will be reviewed when [WIP] will be removed from the PR title.

Bookmark this repository for further updates.

coveralls commented 1 week ago

Coverage Status

coverage: 98.168% (+0.006%) from 98.162% when pulling e2d730d6bda3a745d5832c53091cbbdd39f6c652 on a-rasin:fix/oneListGroupArray into b8e40c842bf63b7d4d706f016d00e259cbc4379e on NaturalIntelligence:master.