update column lengths

Author: tatevikg1

config/config_test.yml

@@ -13,13 +13,9 @@ doctrine:
     dbal:
         driver: 'pdo_mysql'
         host: '%database_host%'
-        path: '%database_path%'
         port: '%database_port%'
         dbname: 'phplist'
         user: '%database_user%'
         password: '%database_password%'
         charset: UTF8
-#    orm:
-#        entity_managers:
-#            default:
-#                report_fields_where_declared: true
+

src/Domain/Analytics/Model/LinkTrackForward.php

@@ -21,7 +21,9 @@ class LinkTrackForward implements DomainModel, Identity
     #[ORM\GeneratedValue]
     private ?int $id = null;
 
-    #[ORM\Column(type: 'string', length: 2083, nullable: true)]
+    // Defined as string(255) due to MySQL limitation (actual max URL length is 2083):
+    // TEXT can't be indexed without a prefix, which Doctrine doesn't support.
+    #[ORM\Column(type: 'string', length: 255, nullable: true)]
     private ?string $url = null;
 
     #[ORM\Column(name: 'urlhash', type: 'string', length: 32, nullable: true)]

src/Domain/Configuration/Model/I18n.php

@@ -18,8 +18,10 @@ class I18n implements DomainModel
     #[ORM\Column(type: 'string', length: 10)]
     private string $lan;
 
+    // Defined as string with length due to MySQL limitation:
+    // TEXT columns can't be indexed without a prefix length, which Doctrine doesn't support.
     #[ORM\Id]
-    #[ORM\Column(type: 'text')]
+    #[ORM\Column(type: 'string', length: 255)]
     private string $original;
 
     #[ORM\Column(type: 'text')]

src/Domain/Configuration/Model/UrlCache.php

@@ -20,7 +20,9 @@ class UrlCache implements DomainModel, Identity
     #[ORM\GeneratedValue]
     private ?int $id = null;
 
-    #[ORM\Column(name: 'url', type: 'string', length: 2083)]
+    // Defined as string(255) due to MySQL limitation (actual max URL length is 2083):
+    // TEXT can't be indexed without a prefix, which Doctrine doesn't support.
+    #[ORM\Column(name: 'url', type: 'string', length: 255)]
     private string $url;
 
     #[ORM\Column(name: 'lastmodified', type: 'integer', nullable: true)]

src/TestingSupport/Traits/DatabaseTestTrait.php

@@ -83,25 +83,17 @@ protected function loadSchema(): void
         $schemaTool = new SchemaTool($this->entityManager);
         $metadata = $this->entityManager->getMetadataFactory()->getAllMetadata();
 
-        // Create all tables at once to handle dependencies properly
         try {
             $schemaTool->createSchema($metadata);
         } catch (ToolsException $e) {
-            // If creating all tables at once fails, try to create them one by one
-            // This is a fallback mechanism
             $connection = $this->entityManager->getConnection();
             $schemaManager = $connection->createSchemaManager();
 
             foreach ($metadata as $classMetadata) {
                 $tableName = $classMetadata->getTableName();
 
                 if (!$schemaManager->tablesExist([$tableName])) {
-                    try {
-                        $schemaTool->createSchema([$classMetadata]);
-                    } catch (ToolsException $e) {
-                        // Log the error but continue with other tables
-                        echo $e->getMessage() . PHP_EOL;
-                    }
+                    $schemaTool->createSchema([$classMetadata]);
                 }
             }
         }

tests/Integration/Domain/Identity/Repository/AdministratorRepositoryTest.php

@@ -43,7 +43,7 @@ protected function tearDown(): void
     public function testFindReadsModelFromDatabase(): void
     {
         /** @var Administrator $actual */
-        $actual = $this->repository->find(1);
+        $actual = $this->repository->findOneBy(['email' => '[email protected]']);
 
         $this->assertNotNull($actual);
         $this->assertFalse($actual->isDisabled());