Fix psalm issues

amphp · Aug 20, 2023 · bac8c10 · bac8c10
1 parent c0745b9
commit bac8c10
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/psalm.xml b/psalm.xml
@@ -27,5 +27,11 @@
                 <directory name="src" />
             </errorLevel>
         </MissingClosureReturnType>
+
+        <StringIncrement>
+            <errorLevel type="suppress">
+                <directory name="src" />
+            </errorLevel>
+        </StringIncrement>
     </issueHandlers>
 </psalm>
diff --git a/src/Base64UrlSessionIdGenerator.php b/src/Base64UrlSessionIdGenerator.php
@@ -13,11 +13,12 @@ final class Base64UrlSessionIdGenerator implements SessionIdGenerator
 
     public function __construct(int $length = 48)
     {
-        if ($length < 0 || $length % 4 !== 0) {
+        if ($length <= 0 || $length % 4 !== 0) {
             throw new \Error('Invalid length (' . $length . '), must be divisible by four');
         }
 
         // divisible by four to not waste chars with "=" and simplify regexp.
+        /** @psalm-suppress PropertyTypeCoercion */
         $this->bytes = (int) ($length / 4 * 3);
         $this->regex = '/^[A-Za-z0-9_\-]{' . $length . '}$/';