Merged #701

Check for array, rather than assuming an array
silvershop · Dec 6, 2018 · d866706 · d866706
2 parents 1bc44e9 + 1c99335
commit d866706
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/code/model/filters/MultiFieldPartialMatchFilter.php b/code/model/filters/MultiFieldPartialMatchFilter.php
@@ -54,7 +54,7 @@ function ($v) {
             }
         );
 
-        if (count($this->subfilters) > 0) {
+        if (is_array($this->subfilters)) {
             foreach ($this->subfilters as $f) {
                 $f->setModifiers($this->subfilterModifiers);
             }
@@ -85,7 +85,7 @@ public function setValue($value)
 
         parent::setValue($value);
 
-        if (count($this->subfilters) > 0) {
+        if (is_array($this->subfilters)) {
             foreach ($this->subfilters as $f) {
                 $f->setValue($value);
             }
@@ -111,7 +111,7 @@ public function apply(DataQuery $query)
         $orGroup = $query->disjunctiveGroup();
         $orGroup = parent::apply($orGroup);
 
-        if (count($this->subfilters) > 0) {
+        if (is_array($this->subfilters)) {
             foreach ($this->subfilters as $f) {
                 $orGroup = $f->apply($orGroup);
             }