mergeAttrs true behaviour changes

[otemnov]

Not sure what is changed in latest release but before following xmlUrl attribute was parsed as single string. Now it is parsed as array.
Input:

<outline xmlUrl="http://www.futurity.org/feed/"/>

Output:

{
     'xmlUrl': [ 
        'http://www.futurity.org/feed', 
        pathname: '/' 
    ]
}

is it expected behaviour/change?

[Leonidas-from-XIV]

Which xmlUrl attribute? If this changed, this only changed in the 0.1 to 0.2 transition.

[otemnov]

Here is example of xml tag:

<outline xmlUrl="http://www.futurity.org/feed/"/>

[otemnov]

I'm using heroku so after today deploy it stoped work as before becasue of provided xml is parsed as

{ xmlUrl: [ 'http://www.futurity.org/feed', pathname: '/' ] }

it was expected to be just

{ xmlUrl: 'http://www.futurity.org/feed' }

heroku do clear npm install every release, so I did not catch it locally
previous release was 4 days ago

[otemnov]

evenmore - looks like every tag attributes are parsed now as array

[Leonidas-from-XIV]

Can't reproduce at all:

{parseString} = require 'xml2js'
xml = '<outline htmlUrl="futurity.org" text="Futurity.org" title="Futurity.org" type="rss" xmlUrl="http://www.futurity.org/feed/"/>'
parseString xml, (err, result) ->
    console.dir result

Parses correctly to:

{ outline: 
   { '$': 
      { htmlUrl: 'futurity.org',
        text: 'Futurity.org',
        title: 'Futurity.org',
        type: 'rss',
        xmlUrl: 'http://www.futurity.org/feed/' } } }

Please provide example code to reproduce your issue.

[otemnov]

looks like it is related to mergeAttrs setting

var xml2js = require('xml2js');
var parser = new xml2js.Parser({
    mergeAttrs: true
});
parser.parseString('<outline xmlUrl="http://www.futurity.org/feed/"/>', function (err, result) {
    console.dir(result);
});

[Leonidas-from-XIV]

Alright, that would have shortened the search. Check what happened in #101.

[otemnov]

oki, thanks for help. Did not get initially that is could be related to mergeAttrs.
Probably could be good to add #101 as breaking change

[Leonidas-from-XIV]

Yes, I agree. I should add that to the readme.