Merge pull request #76 from barthicus/fix-issue-75

xiaohutai · web-flow · commit b5a60a89d497 · 2018-02-23T10:38:50.000+01:00
Add empty array check before function count() call
diff --git a/src/Action/FetchAction.php b/src/Action/FetchAction.php
@@ -50,7 +50,7 @@ public function __construct(
      */
     protected function throwErrorOnNoResults($results, $message)
     {
-        if (! $results || count($results) === 0) {
+        if (! $results || (is_array($results) && count($results) === 0)) {
             throw new ApiInvalidRequestException(
                 $message
             );

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public function __construct(`
`50`	`50`	`*/`
`51`	`51`	`protected function throwErrorOnNoResults($results, $message)`
`52`	`52`	`{`
`53`		`- if (! $results \|\| count($results) === 0) {`
	`53`	`+ if (! $results \|\| (is_array($results) && count($results) === 0)) {`
`54`	`54`	`throw new ApiInvalidRequestException(`
`55`	`55`	`$message`
`56`	`56`	`);`