Merge pull request #126 from spideyfusion/php8-compatibility

Fix interaction with the Sockets API under PHP 8.x
fennb · May 10, 2021 · b7a4171 · b7a4171
2 parents 405d125 + 8d9b94e
commit b7a4171
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,3 +2,5 @@
 *.un~
 .cache
 .cache/*
+composer.lock
+vendor/
diff --git a/lib/Phirehose.php b/lib/Phirehose.php
@@ -518,8 +518,8 @@ public function consume($reconnect = TRUE)
         pcntl_signal_dispatch();
       }
 
-      // Some sort of socket error has occured
-      $this->lastErrorNo = is_resource($this->conn) ? @socket_last_error($this->conn) : NULL;
+      // Some sort of socket error has occurred
+      $this->lastErrorNo = is_resource($this->conn) ? @socket_last_error() : NULL;
       $this->lastErrorMsg = ($this->lastErrorNo > 0) ? @socket_strerror($this->lastErrorNo) : 'Socket disconnected';
       $this->log('Phirehose connection error occured: ' . $this->lastErrorMsg,'error');