Can't concat multiple files

[rafalagunas]

Here is my code, I'm trying to concat multiple files in xfiles folder.

jsonConcat( { src: "./xfiles", dest: "./result.json" }, function(json) { console.log(json); } );

this is what returns on console:
null

result.json only contains one of the files.

Thank you.

[lukejamestaylor]

I'm also experiencing the same issue. If the change function(json) to function(err, json), you can then log out the JSON file.

Still it only returns 1 file for me.

[GochoMugo]

Is it possible to provide the files to be able to reproduce this bug.

Also note that the library is forgiving in that if a file could not be parsed, it will be skipped over.

[guyaumetremblay]

EDIT: After digging, I found this so I replace my comment with this better one:

@GochoMugo Get the same issue here. The bug seems to be cause by one of the 3 replaces in the concat method, line 144 of json-concat.js.

As each of my json file contain an object, the result should be an array of object. But, by replacing }{ by a single coma, the result is only one big object.

For example, taking these two files:

File A

{
    "name": "John"
} 

File B

{
    "name": "Doe"
} 

Final string after before the replace:

{
    "name": "John"
}
{
    "name": "Doe"
} 

And after the replace:

{
    "name": "John"
,
    "name": "Doe"
} 

Expected result:

[
    {
        "name": "Doe"
    },
    {
        "name": "Doe"
    } 
]

Without the replace, that should work as the JSON.parse will throw an exception for invalid format and your logic will use the contentArray variable.

[guyaumetremblay]

@GochoMugo Just saw that the logic in the catch loop on the item in contentArray and merge them into a single object... so it'll not create an array of object.

[steveDL]

Has this bug been fixed?