Add PHP 8.3 support (#7)

Author: alexander-schranz

.github/workflows/test-application.yaml

@@ -37,6 +37,12 @@ jobs:
                       env:
                           SYMFONY_DEPRECATIONS_HELPER: weak
 
+                    - php-version: '8.3'
+                      coverage: true
+                      dependency-versions: 'highest'
+                      env:
+                          SYMFONY_DEPRECATIONS_HELPER: weak
+
         services:
             mysql:
                 image: mysql:5.7

.gitignore

@@ -10,3 +10,5 @@ composer.lock
 
 /tests/Application/var
 /tests/Application/secrets
+
+phpstan.neon

composer.json

@@ -4,7 +4,7 @@
     "license": "proprietary",
     "type": "sulu-bundle",
     "require": {
-        "php": "8.0.* || 8.1.* || 8.2.*",
+        "php": "8.0.* || 8.1.* || 8.2.* || 8.3.*",
         "doctrine/dbal": "^3.3",
         "doctrine/doctrine-bundle": "^2.5",
         "doctrine/orm": "^2.11",
@@ -32,7 +32,7 @@
         "phpstan/phpstan-webmozart-assert": "^1.0",
         "phpunit/phpunit": "^9.5",
         "qossmic/deptrac-shim": "^0.24.0 || ^1.0",
-        "rector/rector": "^0.15.0",
+        "rector/rector": "^1.0",
         "schranz/test-generator": "^0.4",
         "symfony/browser-kit": "^5.4 || ^6.0 || ^7.0",
         "symfony/css-selector": "^5.4 || ^6.0 || ^7.0",

phpstan.neon phpstan.dist.neon

@@ -17,20 +17,25 @@
         __DIR__ . '/tests',
     ]);
 
+    $rectorConfig->phpstanConfigs([
+        __DIR__ . '/phpstan.dist.neon',
+        // rector does not load phpstan extension automatically so require them manually here:
+        __DIR__ . '/vendor/phpstan/phpstan-doctrine/extension.neon',
+        __DIR__ . '/vendor/phpstan/phpstan-symfony/extension.neon',
+    ]);
+
     $rectorConfig->skip([
         __DIR__ . '/tests/Application/var',
         __DIR__ . '/tests/Application/config',
     ]);
 
-    $rectorConfig->phpstanConfig(__DIR__ . '/phpstan.neon');
-
     // basic rules
     $rectorConfig->importNames();
     $rectorConfig->importShortClasses(false);
 
     $rectorConfig->sets([
         SetList::CODE_QUALITY,
-        LevelSetList::UP_TO_PHP_80,
+        // LevelSetList::UP_TO_PHP_80,
     ]);
 
     // symfony rules
@@ -39,7 +44,7 @@
     $rectorConfig->sets([
         SymfonySetList::SYMFONY_CODE_QUALITY,
         SymfonySetList::SYMFONY_CONSTRUCTOR_INJECTION,
-        SymfonyLevelSetList::UP_TO_SYMFONY_54,
+        // SymfonyLevelSetList::UP_TO_SYMFONY_54,
     ]);
 
     // doctrine rules
@@ -49,7 +54,7 @@
 
     // phpunit rules
     $rectorConfig->sets([
-        PHPUnitLevelSetList::UP_TO_PHPUNIT_90,
-        PHPUnitSetList::PHPUNIT_91,
+        // PHPUnitLevelSetList::UP_TO_PHPUNIT_90,
+        // PHPUnitSetList::PHPUNIT_91,
     ]);
 };

rector.php

@@ -22,7 +22,7 @@ public function handle(Envelope $envelope, StackInterface $stack): Envelope
         $envelope = $stack->next()->handle($envelope, $stack);
 
         // flush unit-of-work to the database after the root message was handled successfully
-        if (!empty($envelope->all(EnableFlushStamp::class))) {
+        if ([] !== $envelope->all(EnableFlushStamp::class)) {
             $this->container->get('doctrine.orm.entity_manager')->flush();
         }
 

src/Infrastructure/Symfony/Messenger/FlushMiddleware/DoctrineFlushMiddleware.php

@@ -56,7 +56,7 @@ private function createEnvelope(): Envelope
         return new Envelope(new \stdClass());
     }
 
-    private function createStack(callable $handler = null): StackMiddleware
+    private function createStack(?callable $handler = null): StackMiddleware
     {
         if (!$handler) {
             return new StackMiddleware([]);