Merge pull request #113 from geckojsc/patch-1

Replace count() > 0 with !empty() (fix PHP 7.2 warnings)
fennb · Mar 15, 2018 · 405d125 · 405d125 · frankzvovu · Apr 10, 2018
2 parents 5cb4430 + 49c8da4
commit 405d125
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/Phirehose.php b/lib/Phirehose.php
@@ -606,14 +606,14 @@ protected function connect()
       }
 
       // Filter takes additional parameters
-      if (($this->method == self::METHOD_FILTER || $this->method == self::METHOD_USER) && count($this->trackWords) > 0) {
+      if (($this->method == self::METHOD_FILTER || $this->method == self::METHOD_USER) && !empty($this->trackWords)) {
         $requestParams['track'] = implode(',', $this->trackWords);
       }
       if ( ($this->method == self::METHOD_FILTER || $this->method == self::METHOD_SITE)
-            && count($this->followIds) > 0) {
+            && !empty($this->followIds)) {
         $requestParams['follow'] = implode(',', $this->followIds);
       }
-      if ($this->method == self::METHOD_FILTER && count($this->locationBoxes) > 0) {
+      if ($this->method == self::METHOD_FILTER && !empty($this->locationBoxes)) {
         $requestParams['locations'] = implode(',', $this->locationBoxes);
       }
       if ($this->count <> 0) {