phptek / silverstripe-jsontext

x Sorry, these patches are not available anymore due to data migration. Please run a fresh inspection.

Completed

Push — master ( 2afc86...6ca8f1 )

by Russell

created 2016-06-29 23:34 UTC

Changes

	15 issues (9 major, 2 minor, 4 informational) were introduced. 10 issues were fixed.
	2 new patches (Doc Comments, Unused Use Statements) are available.
	5 added classes/operations
		A	JSONText::isValidJson() added
		A	JSONText::isValidDBValue() added
		A	JSONTextTest::testIsValidExpression() added
		A	JSONTextTest::testIsValidJson() added
		A	JSONTextTest::testIsValidDBValue() added
B	↗	A	JSONTextQueryTest::testQueryWithMatchOnInt() improved
B	↗	A	JSONTextSetValueTest::testSetValueOnSourceArray() improved

7.97

▲ + 0.04

Head	Russ Michell 2016-06-30 11:32 GMT+1200 Modified behaviour of setValue(). Mostly refuses to accept invalid JSON in the first place apart from zero-length strings which SS needs to clear DBField types with.
Base	Russ Michell 2016-06-29 00:14 UTC Simplified some logic. Removed repetition.