savejson st issue in matlab 2017 about jsonlab HOT 2 CLOSED

hi, this bug was previously reported (Issue #34), and was fixed in the commit 86ef12a back in March.

can you check out the latest version, and try it again?

from jsonlab.

Thanks for taking a look. I only pulled the repo this week so I don't know how I would have gotten an old version. With this version (in 2017b), there's now no error but the values get dropped

data = loadjson('{"Test": ["GML", "XML"]}')
json = savejson('',data)
data = 
  struct with fields:

    Test: {["GML"]  ["XML"]}
json =
    '{
     	"Test": [
     		{
     		},
     		{
     		}
     	]
     }
     '

Just as a sanity check, here's 2016b for comparison:

json = savejson('',data)
data = 
  struct with fields:

    Test: {'GML'  'XML'}
json =
{
	"Test": [
		"GML",
		"XML"
	]
}

I put the side by side progressions of obj2json in text files in case that's useful
2016b_out.txt
2017b_out.txt

For me, it's easy to fix if I just add isstring() conditions to ln. 172 and 313 of savejson
elseif(ischar(item)||isstring(item))
if(~ischar(item)&&~isstring(item))
Hopefully that doesn't mess anything else up
Thanks for the nice code!

from jsonlab.