cleanup code for new symfony versions

Author: dbu

CHANGELOG.md

@@ -4,10 +4,17 @@ Changelog
 5.x
 ===
 
+5.0.1
+-----
+
+* Cleanup PHP 8.1 language features.
+* Simplify configuration code.
+
 5.0.0
 -----
 
 * Drop support for Symfony < 6.4
+* Drop support for PHP < 8.1
 * The default framework configuration no longer enables validation attributes.
 * The PHPCR-ODM additional namespace is expected to use attributes rather than annotations.
 

composer.json

@@ -15,7 +15,9 @@
     },
     "require-dev": {
         "doctrine/doctrine-bundle": "^1.8 || ^2.0",
+        "doctrine/phpcr-odm": "^2.0",
         "doctrine/phpcr-bundle": "^3.0",
+        "jackalope/jackalope-doctrine-dbal": "^2.0",
         "symfony/console": "^6.4 || ^7.0",
         "symfony/dependency-injection": "^6.4 || ^7.0",
         "symfony/doctrine-bridge": "^6.4 || ^7.0",

src/DependencyInjection/Compiler/TestContainerPass.php

@@ -20,18 +20,16 @@
 class TestContainerPass implements CompilerPassInterface
 {
     /**
-     * An array of service id's which should be public in a test scenario.
-     *
-     * @var array
+     * @var string[] Service id's which should be public in a test scenario.
      */
-    private $services = [];
+    private array $services;
 
     public function __construct(array $services = [])
     {
         $this->services = $services;
     }
 
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         foreach ($container->getDefinitions() as $id => $definition) {
             if (\in_array($id, $this->services, true)) {

src/Functional/BaseTestCase.php

@@ -113,15 +113,12 @@ protected function getFrameworkBundleClient(): KernelBrowser
      *
      * @see self::getDbManager
      */
-    protected function db($type)
+    protected function db(string $type): PhpcrDecorator|PHPCR|ORM
     {
         return $this->getDbManager($type);
     }
 
-    /**
-     * @return ORM|PHPCR
-     */
-    protected function getDbManager(string $type)
+    protected function getDbManager(string $type): PhpcrDecorator|PHPCR|ORM
     {
         if (isset($this->dbManagers[$type])) {
             return $this->dbManagers[$type];

src/Functional/DbManager/PHPCR.php

@@ -23,7 +23,7 @@
 
 class PHPCR
 {
-    protected $container;
+    protected ContainerInterface $container;
 
     protected ?DocumentManager $om = null;
 

src/HttpKernel/TestKernel.php

@@ -32,17 +32,17 @@
  */
 abstract class TestKernel extends Kernel
 {
-    protected $bundleSets = [];
+    protected array $bundleSets = [];
 
-    protected $requiredBundles = [];
+    protected array $requiredBundles = [];
 
     /**
      * Register commonly needed bundle sets and then
      * after initializing the parent kernel, let the
      * concrete kernel configure itself using the abstracvt
      * configure() command.
      */
-    public function __construct($env, $debug)
+    public function __construct(string $env, bool $debug)
     {
         $defaultBundles = [
             FrameworkBundle::class,
@@ -72,23 +72,25 @@ public function __construct($env, $debug)
      *    $this->addBundle(new MyBundle);
      *    $this->addBundles(array(new Bundle1, new Bundle2));
      */
-    abstract protected function configure();
+    abstract protected function configure(): void;
 
     /**
      * Register a set of bundles with the given name.
      *
      * This method does not add the bundles to the kernel,
      * it just makes a set available.
      */
-    public function registerBundleSet($name, $bundles)
+    public function registerBundleSet(string $name, array $bundles): void
     {
         $this->bundleSets[$name] = $bundles;
     }
 
     /**
      * The bundles in the named sets will be added to the Kernel.
+     *
+     * @param string[] $names
      */
-    public function requireBundleSets(array $names)
+    public function requireBundleSets(array $names): void
     {
         foreach ($names as $name) {
             $this->requireBundleSet($name);
@@ -102,7 +104,7 @@ public function requireBundleSets(array $names)
      * This enables us to declare pre-defined bundle sets without
      * worrying if the bundle is actually present or not.
      */
-    public function requireBundleSet($name)
+    public function requireBundleSet(string $name): void
     {
         if (!isset($this->bundleSets[$name])) {
             throw new \InvalidArgumentException(sprintf(
@@ -127,7 +129,7 @@ public function requireBundleSet($name)
     /**
      * Add concrete bundles to the kernel.
      */
-    public function addBundles(array $bundles)
+    public function addBundles(array $bundles): void
     {
         foreach ($bundles as $bundle) {
             $this->addBundle($bundle);
@@ -137,7 +139,7 @@ public function addBundles(array $bundles)
     /**
      * Add a concrete bundle to the kernel.
      */
-    public function addBundle(BundleInterface $bundle)
+    public function addBundle(BundleInterface $bundle): void
     {
         $this->requiredBundles[] = $bundle;
     }
@@ -153,28 +155,22 @@ public function registerBundles(): iterable
     }
 
     /**
-     * Returns the KernelDir of the CHILD class,
+     * Returns the project directory of the CHILD class,
      * i.e. the concrete implementation in the bundles
      * src/ directory (or wherever).
      */
-    public function getKernelDir()
-    {
-        return $this->getProjectDir();
-    }
-
     public function getProjectDir(): string
     {
         $refl = new \ReflectionClass($this);
         $fname = $refl->getFileName();
-        $kernelDir = \dirname($fname);
 
-        return $kernelDir;
+        return \dirname($fname);
     }
 
     public function getCacheDir(): string
     {
         return implode('/', [
-            $this->getKernelDir(),
+            $this->getProjectDir(),
             'var',
             'cache',
         ]);
@@ -192,7 +188,7 @@ public function getLogDir(): string
     /**
      * Registers the bundles defined in config/bundles.php.
      */
-    protected function registerConfiguredBundles()
+    protected function registerConfiguredBundles(): void
     {
         $bundleFilePath = $this->getKernelDir().'/config/bundles.php';
         if (!file_exists($bundleFilePath)) {