Fix warning from passing null to explode for includeParts

steven-fox · web-flow · commit 81fca60a70c5 · 2022-03-17T12:45:17.000-04:00
diff --git a/src/QueryBuilderRequest.php b/src/QueryBuilderRequest.php
@@ -38,8 +38,8 @@ public function includes(): Collection
 
         $includeParts = $this->getRequestData($includeParameterName);
 
-        if (! is_array($includeParts)) {
-            $includeParts = explode(static::getIncludesArrayValueDelimiter(), $this->getRequestData($includeParameterName));
+        if (! is_array($includeParts) && ! is_null($includeParts)) {
+            $includeParts = explode(static::getIncludesArrayValueDelimiter(), $includeParts);
         }
 
         return collect($includeParts)->filter();

Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,8 @@ public function includes(): Collection`
`38`	`38`
`39`	`39`	`$includeParts = $this->getRequestData($includeParameterName);`
`40`	`40`
`41`		`- if (! is_array($includeParts)) {`
`42`		`- $includeParts = explode(static::getIncludesArrayValueDelimiter(), $this->getRequestData($includeParameterName));`
	`41`	`+ if (! is_array($includeParts) && ! is_null($includeParts)) {`
	`42`	`+ $includeParts = explode(static::getIncludesArrayValueDelimiter(), $includeParts);`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`return collect($includeParts)->filter();`