Fix php 8 deprecation warnings

voryx · Feb 18, 2022 · bcb7821 · bcb7821
1 parent b82fed2
commit bcb7821
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/composer.json b/composer.json
@@ -34,7 +34,7 @@
         "bin/thruway"
     ],
     "require": {
-        "php": ">=5.6",
+        "php": ">=7.1",
         "voryx/thruway-common": "^1.0.5",
         "thruway/client": "^0.5.0",
         "react/event-loop": "^1.0 || ^0.5 || ^0.4.3",

diff --git a/src/Authentication/AuthenticationDetails.php b/src/Authentication/AuthenticationDetails.php
@@ -210,12 +210,12 @@ public function setAuthExtra($authExtra)
      * @return mixed data which can be serialized by <b>json_encode</b>,
      * which is a value of any type other than a resource.
      */
-    public function jsonSerialize()
+    public function jsonSerialize() : array
     {
         return [
             'authid'      => $this->authId,
             'auth_method' => $this->authMethod,
             'auth_roles'  => $this->authRoles,
         ];
     }
-}
+}
diff --git a/tests/WAMP/DisclosePublisherTest.php b/tests/WAMP/DisclosePublisherTest.php
@@ -52,7 +52,7 @@ public function testSubscribe()
              * Subscribe to event
              */
             $session->subscribe('com.example.publish',
-              function ($args, $kwargs = null, $details, $publicationId = null) {
+              function ($args, $kwargs, $details, $publicationId = null) {
 
                   $this->_testArgs        = $args;
                   $this->_testPublisherId = $details->publisher;
@@ -96,4 +96,4 @@ function () use ($session) {
         $this->assertEquals("internalClient", $this->_testAuthMethod);
         $this->assertEquals('com.example.publish', $this->_testTopic);
     }
-} 
+}