Update Entity.php

nowackipawel · web-flow · commit 6f302930b98e · 2019-06-29T12:04:06.000+02:00
performance and remove checking to ensure CastException will be thrown when necessary
diff --git a/system/Entity.php b/system/Entity.php
@@ -588,8 +588,7 @@ private function castAsJson($value, bool $asArray = false)
 		$tmp = ! is_null($value) ? ($asArray ? [] : new \stdClass) : null;
 		if (function_exists('json_decode'))
 		{
-			$strlen = is_string($value) ? strlen($value) : 0;
-			if (($strlen > 1 && ((strpos($value, '[') === 0 && strrpos($value, ']') === $strlen - 1) || (strpos($value, '{') === 0 && strrpos($value, '}') === $strlen - 1) || (strpos($value, '"') === 0 && strrpos($value, '"') === $strlen - 1))) || is_numeric($value))
+			if ((is_string($value) && strlen($value) > 1 && in_array($value{0}, ['[','{','"'])) || is_numeric($value))
 			{
 				$tmp = json_decode($value, $asArray);
 

Original file line number	Diff line number	Diff line change
`@@ -588,8 +588,7 @@ private function castAsJson($value, bool $asArray = false)`
`588`	`588`	`$tmp = ! is_null($value) ? ($asArray ? [] : new \stdClass) : null;`
`589`	`589`	`if (function_exists('json_decode'))`
`590`	`590`	`{`
`591`		`- $strlen = is_string($value) ? strlen($value) : 0;`
`592`		`- if (($strlen > 1 && ((strpos($value, '[') === 0 && strrpos($value, ']') === $strlen - 1) \|\| (strpos($value, '{') === 0 && strrpos($value, '}') === $strlen - 1) \|\| (strpos($value, '"') === 0 && strrpos($value, '"') === $strlen - 1))) \|\| is_numeric($value))`
	`591`	`+ if ((is_string($value) && strlen($value) > 1 && in_array($value{0}, ['[','{','"'])) \|\| is_numeric($value))`
`593`	`592`	`{`
`594`	`593`	`$tmp = json_decode($value, $asArray);`
`595`	`594`