Merge pull request #342 from kenjis/fix-checkQueryReturn

refactor: fix incorrect types

Author: kenjis

src/Authentication/Authenticators/AccessTokens.php

@@ -44,7 +44,7 @@ public function attempt(array $credentials): Result
         $request = service('request');
 
         $ipAddress = $request->getIPAddress();
-        $userAgent = $request->getUserAgent();
+        $userAgent = (string) $request->getUserAgent();
 
         $result = $this->check($credentials);
 

src/Authentication/Authenticators/Session.php

@@ -98,7 +98,7 @@ public function attempt(array $credentials): Result
         $request = service('request');
 
         $ipAddress = $request->getIPAddress();
-        $userAgent = $request->getUserAgent();
+        $userAgent = (string) $request->getUserAgent();
 
         $result = $this->check($credentials);
 

src/Controllers/MagicLinkController.php

@@ -178,7 +178,7 @@ private function recordLoginAttempt(
             $identifier,
             $success,
             $this->request->getIPAddress(),
-            $this->request->getUserAgent(),
+            (string) $this->request->getUserAgent(),
             $userId
         );
     }

src/Models/CheckQueryReturnTrait.php

@@ -10,7 +10,10 @@ trait CheckQueryReturnTrait
 {
     private ?bool $currentDBDebug = null;
 
-    private function checkQueryReturn(bool $return): void
+    /**
+     * @param bool|int|string $return insert() returns insert ID.
+     */
+    private function checkQueryReturn($return): void
     {
         $this->restoreDBDebug();