Merge pull request #63 from Rareloop/chore/php-85

AliceKLWilliams · web-flow · commit b44b11afb9c8 · 2026-02-11T15:06:08.000Z
Upgrade to PHP 8.5
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -7,7 +7,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php_version: [8.1, 8.2, 8.3, 8.4]
+        php_version: [8.3, 8.4, 8.5]
         composer_flags: ['', '--prefer-lowest']
 
     steps:
diff --git a/composer.json b/composer.json
@@ -3,19 +3,19 @@
     "description": "A powerful MVC framework for the modern WordPress developer. Write better, more expressive and easier to maintain code",
     "license": "MIT",
     "require": {
-        "php": ">=8.1",
+        "php": ">=8.3",
         "php-di/php-di": "^7.1.1",
-        "rareloop/router": "^6.0.3",
+        "rareloop/router": "^6.1.0",
         "psr/container": "^2.0.2",
         "psr/http-message": "^2",
         "psr/http-server-middleware": "^1.0.2",
         "timber/timber": "^2.3.3",
         "monolog/monolog": "^3.9",
         "illuminate/collections": "^10.49",
         "statamic/stringy": "^3.1.3",
-        "laminas/laminas-diactoros": "^3.6.0",
+        "laminas/laminas-diactoros": "^3.8.0",
         "rareloop/psr7-server-request-extension": "^2.2.0",
-        "spatie/macroable": "^1.0.1",
+        "spatie/macroable": "^2.1.0",
         "mindplay/middleman": "^4.0.4",
         "psr/log": "^2.0.0",
         "symfony/var-dumper": "^6.4.26",
diff --git a/phpunit.xml b/phpunit.xml
@@ -1,22 +1,13 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit bootstrap="vendor/autoload.php"
-         backupGlobals="false"
-         backupStaticAttributes="false"
-         colors="true"
-         verbose="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false"
-         stopOnFailure="false">
-    <testsuites>
-        <testsuite name="Rareloop Router Test Suite">
-            <directory>tests</directory>
-        </testsuite>
-    </testsuites>
-    <filter>
-        <whitelist>
-            <directory suffix=".php">src/</directory>
-        </whitelist>
-    </filter>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" bootstrap="vendor/autoload.php" backupGlobals="false" backupStaticAttributes="false" colors="true" verbose="true" convertErrorsToExceptions="true" convertNoticesToExceptions="true" convertWarningsToExceptions="true" processIsolation="false" stopOnFailure="false" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd">
+  <coverage>
+    <include>
+      <directory suffix=".php">src/</directory>
+    </include>
+  </coverage>
+  <testsuites>
+    <testsuite name="Rareloop Router Test Suite">
+      <directory>tests</directory>
+    </testsuite>
+  </testsuites>
 </phpunit>
diff --git a/src/Application.php b/src/Application.php
@@ -180,10 +180,10 @@ public function register($provider)
 
     public function getProvider($provider)
     {
-        $providerClass = is_string($provider) ? $provider : get_class($provider);
+        $providerClass = is_string($provider) ? $provider : $provider::class;
 
         return (new Collection($this->loadedProviders))->first(function ($provider) use ($providerClass) {
-            return get_class($provider) === $providerClass;
+            return $provider::class === $providerClass;
         });
     }
 
diff --git a/src/Dcrypt/Hash.php b/src/Dcrypt/Hash.php
@@ -70,7 +70,7 @@ private static function build(string $input, string $password, int $cost, ?strin
      */
     private static function cost(int $cost): int
     {
-        return $cost % \pow(2, 32);
+        return $cost % 2 ** 32;
     }
 
     /**
diff --git a/src/Dcrypt/OpensslBridge.php b/src/Dcrypt/OpensslBridge.php
@@ -162,11 +162,7 @@ private static function mode(): string
 
         $cipher = \strtolower(static::CIPHER);
 
-        if (isset($legacy[$cipher])) {
-            return $legacy[$cipher];
-        }
-
-        return $cipher;
+        return $legacy[$cipher] ?? $cipher;
     }
 
     /**
diff --git a/src/Dcrypt/Spritz.php b/src/Dcrypt/Spritz.php
@@ -36,6 +36,7 @@ class Spritz extends Rc4
      * @param string $key Key to use for encryption
      * @return string
      */
+    #[\Override]
     public static function crypt(string $str, string $key): string
     {
         $s = self::initializeState($key);
diff --git a/src/Encrypter.php b/src/Encrypter.php
@@ -7,11 +7,8 @@
 
 class Encrypter implements EncrypterContract
 {
-    protected $key;
-
-    public function __construct($key)
+    public function __construct(protected $key)
     {
-        $this->key = $key;
     }
 
     public function encrypt($data)
diff --git a/src/Exceptions/Handler.php b/src/Exceptions/Handler.php
@@ -53,6 +53,6 @@ public function render(ServerRequestInterface $request, Exception $e): ResponseI
 
     protected function shouldNotReport(Exception $e)
     {
-        return in_array(get_class($e), $this->dontReport);
+        return in_array($e::class, $this->dontReport);
     }
 }
diff --git a/src/Http/Lumberjack.php b/src/Http/Lumberjack.php
@@ -13,8 +13,6 @@
 
 class Lumberjack
 {
-    private $app;
-
     protected $bootstrappers = [
         LoadConfiguration::class,
         RegisterExceptionHandler::class,
@@ -25,9 +23,8 @@ class Lumberjack
         RegisterRequestHandler::class,
     ];
 
-    public function __construct(Application $app)
+    public function __construct(private Application $app)
     {
-        $this->app = $app;
     }
 
     public function bootstrap()
diff --git a/src/Http/Router.php b/src/Http/Router.php
@@ -17,6 +17,7 @@ class Router extends RareRouter
      * @param  callable|string $callback
      * @return \Rareloop\Router\Route
      */
+    #[\Override]
     public function map(array $verbs, string $uri, $callback): Route
     {
         if ($this->isControllerString($callback)) {
@@ -34,7 +35,7 @@ public function map(array $verbs, string $uri, $callback): Route
      */
     private function isControllerString($callback) : bool
     {
-        return is_string($callback) && strpos($callback, '@') !== false;
+        return is_string($callback) && str_contains($callback, '@');
     }
 
     /**
diff --git a/src/Page.php b/src/Page.php
@@ -11,6 +11,7 @@ class Page extends Post
      *
      * @return string
      */
+    #[\Override]
     public static function getPostType()
     {
         return 'page';
diff --git a/src/Post.php b/src/Post.php
@@ -26,6 +26,7 @@ public function __construct(mixed $wpPost = null, $preventTimberInit = false)
         }
     }
 
+    #[\Override]
     public function __call($name, $arguments)
     {
         if (static::hasMacro($name)) {
@@ -46,7 +47,7 @@ public static function __callStatic($name, $arguments)
             return call_user_func_array([$builder, $name], $arguments);
         }
 
-        trigger_error('Call to undefined method ' . __CLASS__ . '::' . $name . '()', E_USER_ERROR);
+        trigger_error('Call to undefined method ' . self::class . '::' . $name . '()', E_USER_ERROR);
     }
 
     /**
diff --git a/src/Providers/SessionServiceProvider.php b/src/Providers/SessionServiceProvider.php
@@ -44,11 +44,13 @@ public function boot()
                 setcookie(
                     $this->session->getName(),
                     $this->session->getId(),
-                    time() + ($cookieOptions['lifetime'] * 60),
-                    $cookieOptions['path'],
-                    $cookieOptions['domain'],
-                    $cookieOptions['secure'],
-                    $cookieOptions['httpOnly']
+                    [
+                        'expires' => time() + ($cookieOptions['lifetime'] * 60),
+                        'path' => $cookieOptions['path'],
+                        'domain' => $cookieOptions['domain'],
+                        'secure' => $cookieOptions['secure'],
+                        'httponly' => $cookieOptions['httpOnly']
+                    ]
                 );
 
                 $cookieSet = true;
diff --git a/src/ScopedQueryBuilder.php b/src/ScopedQueryBuilder.php
@@ -12,14 +12,10 @@
 
 class ScopedQueryBuilder
 {
-    protected $postClass;
-
     protected $queryBuilder;
 
-    public function __construct($postClass)
+    public function __construct(protected $postClass)
     {
-        $this->postClass = $postClass;
-
         $this->queryBuilder = Helpers::app(QueryBuilderContract::class);
 
         $this->queryBuilder
diff --git a/src/Session/EncryptedStore.php b/src/Session/EncryptedStore.php
@@ -25,11 +25,13 @@ public function __construct(
         parent::__construct($name, $handler, $id);
     }
 
+    #[\Override]
     protected function prepareForStorage($data)
     {
         return $this->encrypter->encrypt($data);
     }
 
+    #[\Override]
     protected function prepareForUnserialize($data)
     {
         if ($data === '') {
diff --git a/src/Session/FileSessionHandler.php b/src/Session/FileSessionHandler.php
@@ -8,13 +8,8 @@
 
 class FileSessionHandler implements SessionHandlerInterface
 {
-    protected $path;
-    protected $prefix;
-
-    public function __construct($path, $prefix = 'lumberjack_session_')
+    public function __construct(protected $path, protected $prefix = 'lumberjack_session_')
     {
-        $this->path = $path;
-        $this->prefix = $prefix;
     }
 
     #[\ReturnTypeWillChange]
diff --git a/tests/Unit/ApplicationTest.php b/tests/Unit/ApplicationTest.php
@@ -552,21 +552,15 @@ public function calling_detectWhenRequestHasNotBeenHandled_adds_actions()
 
 class BootstrapperBootstrapTester
 {
-    public $callback;
-
-    public function __construct($callback)
+    public function __construct(public $callback)
     {
-        $this->callback = $callback;
     }
 }
 
 abstract class TestBootstrapperBase
 {
-    private BootstrapperBootstrapTester $tester;
-
-    public function __construct(BootstrapperBootstrapTester $tester)
+    public function __construct(private BootstrapperBootstrapTester $tester)
     {
-        $this->tester = $tester;
     }
 
     public function bootstrap(Application $app)
@@ -654,13 +648,9 @@ public function __construct(TestInterface $test)
 class RequiresAdditionalConstructorParams
 {
     public $param;
-    public $param1;
-    public $param2;
 
-    public function __construct(TestInterface $test, $param1, $param2)
+    public function __construct(TestInterface $test, public $param1, public $param2)
     {
         $this->param = $test;
-        $this->param1 = $param1;
-        $this->param2 = $param2;
     }
 }
diff --git a/tests/Unit/Dcrypt/HashTest.php b/tests/Unit/Dcrypt/HashTest.php
@@ -41,8 +41,8 @@ public function testHmacAlgoFailure()
 
     public function testFail()
     {
-        $input = str_repeat('A', rand(0, 10000));
-        $key = str_repeat('A', rand(10, 100));
+        $input = str_repeat('A', random_int(0, 10000));
+        $key = str_repeat('A', random_int(10, 100));
         $cost = 1;
 
         $output = Hash::make($input, $key, $cost);
diff --git a/tests/Unit/Dcrypt/StrTest.php b/tests/Unit/Dcrypt/StrTest.php
@@ -9,11 +9,11 @@ class StrTest extends \PHPUnit\Framework\TestCase
     public function testEquals()
     {
         // Test with hash_equals
-        $this->assertTrue(Str::equal('2222', '2222', true));
-        $this->assertFalse(Str::equal('2222', '3333', true));
+        $this->assertTrue(Str::equal('2222', '2222'));
+        $this->assertFalse(Str::equal('2222', '3333'));
 
         // Test without hash_equals
-        $this->assertTrue(Str::equal('2222', '2222', false));
-        $this->assertFalse(Str::equal('2222', '3333', false));
+        $this->assertTrue(Str::equal('2222', '2222'));
+        $this->assertFalse(Str::equal('2222', '3333'));
     }
 }
diff --git a/tests/Unit/Dcrypt/TestSupport.php b/tests/Unit/Dcrypt/TestSupport.php
@@ -16,7 +16,7 @@ public static function swaprandbyte($inp)
         // @codeCoverageIgnoreStart
         $len = strlen($inp);
         $inp = str_split($inp);
-        $off = rand(0, $len - 1);
+        $off = random_int(0, $len - 1);
         $byte = $inp[$off];
         $rbyte = \random_bytes(1);
         if ($byte === $rbyte) {
diff --git a/tests/Unit/HelpersTest.php b/tests/Unit/HelpersTest.php
@@ -375,12 +375,7 @@ class TestExceptionHandler extends Handler
 
 class RequiresConstructorParams
 {
-    public $param1;
-    public $param2;
-
-    public function __construct($param1, $param2)
+    public function __construct(public $param1, public $param2)
     {
-        $this->param1 = $param1;
-        $this->param2 = $param2;
     }
 }
diff --git a/tests/Unit/Http/MiddlewareAliasStoreTest.php b/tests/Unit/Http/MiddlewareAliasStoreTest.php
@@ -108,12 +108,7 @@ class MASTestClass
 
 class MASTestClassWithConstructorParams
 {
-    public $param1;
-    public $param2;
-
-    public function __construct($param1, $param2)
+    public function __construct(public $param1, public $param2)
     {
-        $this->param1 = $param1;
-        $this->param2 = $param2;
     }
 }
diff --git a/tests/Unit/PostQueryBuilderTest.php b/tests/Unit/PostQueryBuilderTest.php
@@ -83,6 +83,7 @@ public static function setCreateBuilderResponse($builder)
         static::$injectedBuilder = $builder;
     }
 
+    #[\Override]
     public static function builder(): ScopedQueryBuilder
     {
         return static::$injectedBuilder;
diff --git a/tests/Unit/PostTest.php b/tests/Unit/PostTest.php
@@ -19,6 +19,7 @@
  */
 class PostTest extends TestCase
 {
+    public $dummyData;
     use BrainMonkeyPHPUnitIntegration;
 
     /** @test */
@@ -239,6 +240,7 @@ class PostWithPrivateData extends Post
 
 class RegisterablePostType extends Post
 {
+    #[\Override]
     public static function getPostType(): string
     {
         return 'registerable_post_type';
@@ -289,6 +291,7 @@ protected static function getPostTypeConfig(): array
 
 class UnregisterablePostTypeWithoutConfig extends Post
 {
+    #[\Override]
     public static function getPostType(): string
     {
         return 'post_type';
diff --git a/tests/Unit/Providers/CustomPostTypesServiceProviderTest.php b/tests/Unit/Providers/CustomPostTypesServiceProviderTest.php
diff --git a/tests/Unit/Providers/RouterServiceProviderTest.php b/tests/Unit/Providers/RouterServiceProviderTest.php
diff --git a/tests/Unit/Providers/WordPressControllersServiceProviderTest.php b/tests/Unit/Providers/WordPressControllersServiceProviderTest.php
diff --git a/tests/Unit/QueryBuilderTest.php b/tests/Unit/QueryBuilderTest.php
diff --git a/tests/Unit/ScopedQueryBuilderTest.php b/tests/Unit/ScopedQueryBuilderTest.php
diff --git a/tests/Unit/Session/FileSessionHandlerTest.php b/tests/Unit/Session/FileSessionHandlerTest.php
diff --git a/tests/Unit/Session/SessionManagerTest.php b/tests/Unit/Session/SessionManagerTest.php

Original file line number	Diff line number	Diff line change
`@@ -180,10 +180,10 @@ public function register($provider)`
`180`	`180`
`181`	`181`	`public function getProvider($provider)`
`182`	`182`	`{`
`183`		`- $providerClass = is_string($provider) ? $provider : get_class($provider);`
	`183`	`+ $providerClass = is_string($provider) ? $provider : $provider::class;`
`184`	`184`
`185`	`185`	`return (new Collection($this->loadedProviders))->first(function ($provider) use ($providerClass) {`
`186`		`- return get_class($provider) === $providerClass;`
	`186`	`+ return $provider::class === $providerClass;`
`187`	`187`	`});`
`188`	`188`	`}`
`189`	`189`
Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ private static function build(string $input, string $password, int $cost, ?strin`
`70`	`70`	`*/`
`71`	`71`	`private static function cost(int $cost): int`
`72`	`72`	`{`
`73`		`- return $cost % \pow(2, 32);`
	`73`	`+ return $cost % 2 ** 32;`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ class Spritz extends Rc4`
`36`	`36`	`* @param string $key Key to use for encryption`
`37`	`37`	`* @return string`
`38`	`38`	`*/`
	`39`	`+ #[\Override]`
`39`	`40`	`public static function crypt(string $str, string $key): string`
`40`	`41`	`{`
`41`	`42`	`$s = self::initializeState($key);`
Original file line number	Diff line number	Diff line change
`@@ -7,11 +7,8 @@`
`7`	`7`
`8`	`8`	`class Encrypter implements EncrypterContract`
`9`	`9`	`{`
`10`		`- protected $key;`
`11`		`-`
`12`		`- public function __construct($key)`
	`10`	`+ public function __construct(protected $key)`
`13`	`11`	`{`
`14`		`- $this->key = $key;`
`15`	`12`	`}`
`16`	`13`
`17`	`14`	`public function encrypt($data)`
Original file line number	Diff line number	Diff line change
`@@ -53,6 +53,6 @@ public function render(ServerRequestInterface $request, Exception $e): ResponseI`
`53`	`53`
`54`	`54`	`protected function shouldNotReport(Exception $e)`
`55`	`55`	`{`
`56`		`- return in_array(get_class($e), $this->dontReport);`
	`56`	`+ return in_array($e::class, $this->dontReport);`
`57`	`57`	`}`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,6 @@`
`13`	`13`
`14`	`14`	`class Lumberjack`
`15`	`15`	`{`
`16`		`- private $app;`
`17`		`-`
`18`	`16`	`protected $bootstrappers = [`
`19`	`17`	`LoadConfiguration::class,`
`20`	`18`	`RegisterExceptionHandler::class,`
`@@ -25,9 +23,8 @@ class Lumberjack`
`25`	`23`	`RegisterRequestHandler::class,`
`26`	`24`	`];`
`27`	`25`
`28`		`- public function __construct(Application $app)`
	`26`	`+ public function __construct(private Application $app)`
`29`	`27`	`{`
`30`		`- $this->app = $app;`
`31`	`28`	`}`
`32`	`29`
`33`	`30`	`public function bootstrap()`