(I think I) found a bug about jsonexport

[AckerApple]

Fix released as version 2.5.0

Please acquire. Please note we are going to a version 3.0.0 soon

[cbschuld]

@pietkamps - just downloaded jsonexport myself and encountered the exact same bug. Specifically if it is an empty value.

[max-bruhn]

i have encountered the same bug with my empty strings.
#41 (comment)

[AckerApple]

An online demo tool exists for jsonexport and I used it to check the mentioned data call outs.

Online tool link: https://kauegimenes.github.io/jsonexport/demo/

Here is valid JSON for this tickets mentioned issue. It does NOT convert to CSV correctly (bug confirmed)

[
    {
        "day": "1",
        "time": "12:00",
        "persons": [
            {
                "name": "Bob",
                "lastname": "Smith"
            },
            {
                "name": "James",
                "lastname": "piet"
            },
            {
                "name": "",
                "lastname": "Miller"
            },
            {
                "name": "David",
                "lastname": "Martin"
            }
        ]
    }
]

Invalid result line 3 and 4:

day,time,persons.name,persons.lastname
1,12:00,Bob,Smith
,,James,piet
,,David,Miller
,,,Martin

Expected Result for line 3 and 4:

day,time,persons.name,persons.lastname
1,12:00,Bob,Smith
,,James,piet
,,,Miller
,,David,Martin

I don't know when but I will attempt a fix in the near future.

Do you want a fix faster? This code is not that difficult, participate, make a pull request.

NOTE TO SELF: Upgrade demo web page. Don't require just JSON and use eval to cast javascript vars (consider security cost of running eval).

[kaue]

@pietkamps I finally got a little time to catch up with jsonexport :)
I just opened a PR that fixes this issue #67
@AckerApple can you do a code review?

[AckerApple]

NEW: You should be able to jump to this url to see proof of this fix