-
Notifications
You must be signed in to change notification settings - Fork 181
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
js2xml outputs empty string #20
Comments
@jncr , The correct js object should be: const js = {
request: {
user: {
_text: 'username'
},
pass: {
_text: 'password'
},
numbers: {
number: [
{
_text: 1
},
{
_text: 2
}
]
}
}
}; Lastly, you need to pass const xml = convert.js2xml(js, {compact: true}); However, your sample input is simpler and I like it. I will try to add support to this kind of input so the parser can accept the following (note the usage of array in const js = {
request: {
user: 'username',
pass: 'password',
numbers: {
number: [1, 2]
}
}
}
} |
Sorry @jncr. Now I remember why I did not support this kind of input. Although input like const js = {name: 'Yousuf'} is simpler than: const js = {name: {_text: 'Yousuf'}} which both can be parsed into xml as: <name>Yousuf</name> But the problem occurs when there is some attributes with the node. Say we want the output xml to be: <name title="Mr">Yousuf</name> We cannot use your syntax to support this. But we can use this syntax to support this requirement: const js = {name: {_attributes: {title: 'Mr'}, _text: 'Yousuf'}} To further illustrate why const js = {name: {_attributes: {title: 'Mr'}, _text: 'Yousuf', rating: {_text: 'good'}}} which will generate following xml: <name title="Mr">
Yousuf
<rating>good</rating>
</name> I don't think this can be represented by your js object syntax. |
@jncr See also release notes. |
[email protected]
[email protected]
runkit here - https://runkit.com/jncr/xml-js. Something I'm doing wrong?
The text was updated successfully, but these errors were encountered: