Fix Doctrine deprecations (#4608)

.github/workflows/testing.yaml

@@ -47,7 +47,7 @@ jobs:
                 run: echo "composer_cache_directory=$(composer config cache-dir)" >> $GITHUB_ENV
 
             -   name: Cache Composer dependencies
-                uses: actions/cache@v3
+                uses: actions/cache@v4
                 with:
                     path: "${{ env.composer_cache_directory }}"
                     key: ${{ runner.os }}-${{ matrix.php }}-${{ hashFiles('**/composer.lock') }}

config/packages/doctrine.yaml

@@ -17,7 +17,7 @@ doctrine:
                 charset: utf8mb4
                 default_table_options:
                     charset: utf8mb4
-                    collate: utf8mb4_unicode_ci
+                    collation: utf8mb4_unicode_ci
                 schema_manager_factory: doctrine.dbal.default_schema_manager_factory
 
         types:

config/packages/doctrine_migrations.yaml

@@ -5,3 +5,4 @@ doctrine_migrations:
             table_name: 'migration_versions'
     migrations_paths:
         'DoctrineMigrations': '%kernel.project_dir%/migrations'
+    custom_template: '%kernel.project_dir%/migrations/MigrationTemplate.txt'

migrations/MigrationTemplate.txt

@@ -0,0 +1,34 @@
+<?php
+
+/*
+ * This file is part of the Kimai time-tracking app.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace DoctrineMigrations;
+
+use App\Doctrine\AbstractMigration;
+use Doctrine\DBAL\Schema\Schema;
+
+/**
+ * @version 2.x
+ */
+final class <className> extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return '';
+    }
+
+    public function up(Schema $schema): void
+    {
+<up>
+    }
+
+    public function down(Schema $schema): void
+    {
+<down>
+    }<override>
+}

migrations/Version20230327143628.php

@@ -51,9 +51,4 @@ final class Version20230327143628 extends AbstractMigration
         $workingTimes->removeForeignKey('FK_F95E49334EA3CB3D');
         $schema->dropTable('kimai2_working_times');
     }
-
-    public function isTransactional(): bool
-    {
-        return false;
-    }
 }

migrations/Version20230606125948.php

@@ -35,9 +35,4 @@ final class Version20230606125948 extends AbstractMigration
         $tags = $schema->getTable('kimai2_tags');
         $tags->dropColumn('visible');
     }
-
-    public function isTransactional(): bool
-    {
-        return false;
-    }
 }

migrations/Version20230819090536.php

@@ -11,8 +11,8 @@ declare(strict_types=1);
 
 namespace DoctrineMigrations;
 
+use App\Doctrine\AbstractMigration;
 use Doctrine\DBAL\Schema\Schema;
-use Doctrine\Migrations\AbstractMigration;
 
 /**
  * @version 2.0.31
@@ -40,9 +40,4 @@ final class Version20230819090536 extends AbstractMigration
         $table->dropIndex('IDX_B9AC5BCE19E9AC5F');
         $table->dropColumn('supervisor_id');
     }
-
-    public function isTransactional(): bool
-    {
-        return false;
-    }
 }

migrations/Version20231130000719.php

@@ -11,8 +11,8 @@ declare(strict_types=1);
 
 namespace DoctrineMigrations;
 
+use App\Doctrine\AbstractMigration;
 use Doctrine\DBAL\Schema\Schema;
-use Doctrine\Migrations\AbstractMigration;
 
 /**
  * @version 2.5.0
@@ -34,10 +34,6 @@ final class Version20231130000719 extends AbstractMigration
 
     public function down(Schema $schema): void
     {
-    }
-
-    public function isTransactional(): bool
-    {
-        return false;
+        $this->preventEmptyMigrationWarning();
     }
 }

src/Doctrine/AbstractMigration.php

@@ -30,7 +30,6 @@ abstract class AbstractMigration extends BaseAbstractMigration
     }
 
     /**
-     * @param Schema $schema
      * @throws Exception
      */
     public function preUp(Schema $schema): void
@@ -39,7 +38,6 @@ abstract class AbstractMigration extends BaseAbstractMigration
     }
 
     /**
-     * @param Schema $schema
      * @throws Exception
      */
     public function preDown(Schema $schema): void
@@ -52,7 +50,7 @@ abstract class AbstractMigration extends BaseAbstractMigration
      *
      * @throws Exception
      */
-    protected function abortIfPlatformNotSupported(): void
+    private function abortIfPlatformNotSupported(): void
     {
         $platform = $this->connection->getDatabasePlatform();
         if (!($platform instanceof MySQLPlatform)) {

src/Doctrine/DsnParserFactory.php

@@ -0,0 +1,64 @@
+<?php
+
+/*
+ * This file is part of the Kimai time-tracking app.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace App\Doctrine;
+
+use Doctrine\DBAL\Platforms\AbstractPlatform;
+use Doctrine\DBAL\Tools\DsnParser;
+
+final class DsnParserFactory
+{
+    /**
+     * Copied here from Doctrine v3 DriverManager, as they are going to be removed in Doctrine 4.
+     *
+     * @var array<string, string>
+     */
+    private static array $driverSchemeAliases = [
+        'db2' => 'ibm_db2',
+        'mssql' => 'pdo_sqlsrv',
+        'mysql' => 'pdo_mysql',
+        'mysql2' => 'pdo_mysql', // Amazon RDS, for some weird reason
+        'postgres' => 'pdo_pgsql',
+        'postgresql' => 'pdo_pgsql',
+        'pgsql' => 'pdo_pgsql',
+        'sqlite' => 'pdo_sqlite',
+        'sqlite3' => 'pdo_sqlite',
+    ];
+
+    public function create(): DsnParser
+    {
+        return new DsnParser(self::$driverSchemeAliases);
+    }
+
+    /**
+     * @return array<string, array<mixed>|bool|AbstractPlatform|int|string>
+     */
+    public function parse(
+        #[\SensitiveParameter]
+        string $dsn
+    ): array
+    {
+        // see https://github.com/doctrine/dbal/pull/5843
+
+        $options = $this->create()->parse($dsn);
+
+        $options = array_merge(
+            $options,
+            [
+                'charset' => 'utf8mb4',
+                'defaultTableOptions' => [
+                    'charset' => 'utf8mb4',
+                    'collation' => 'utf8mb4_unicode_ci',
+                ]
+            ]
+        );
+
+        return $options;
+    }
+}