id	summary	reporter	owner	description	type	status	priority	milestone	component	version	severity	resolution	keywords	cc	distribution	status_field
1553	cjson parser too buggy for exchange formats (such as Journal Entry Bundles)	martin.langhoff	alsroot	"This commit http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/ee4535c98ae74347e7072909d49dcf8a5e16ca7b switched from JSON to CJSON.

The fallout of it is that now the reader of ""exchange"" formats such as Journal Entry Bundles is failing to read perfectly valid files.

JEBs are sometimes produced outside of Sugar, for example in Moodle code (which is PHP). PHP has no way to produce CJSON -- the _only_ CJSON writer in existence is written in Python, and CJSON is actually an OLPC/Sugar invention.

Attached is a patch fixing simple JEB imports, but it is very limited: if the JEB has a buddies entry, the CJSON parser still kicks in and fails to parse it.

Perhaps a better fix is to just revert the patch above. I just don't know the rationale.

More info on the fallout on this OLPC bug: http://dev.laptop.org/ticket/9651
"	defect	closed	Unspecified by Maintainer	Unspecified by Release Team	journal	0.84.x	Unspecified	fixed	r+	dsd	OLPC	Unconfirmed