From 05e87fd5920b77fbd00a301accc0933ce4f4d7c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BCrgen=20Mummert?= <juergen@MacBookPro.fritz.box>
Date: Sun, 21 Dec 2025 20:25:28 +0100
Subject: [PATCH] Finalize tl_search migration with proper shouldRun check

---
 src/Migration/ExtendTlSearchMigration.php | 42 +++++++++++++++++------
 1 file changed, 32 insertions(+), 10 deletions(-)

diff --git a/src/Migration/ExtendTlSearchMigration.php b/src/Migration/ExtendTlSearchMigration.php
index 592d45b..2506ccd 100644
--- a/src/Migration/ExtendTlSearchMigration.php
+++ b/src/Migration/ExtendTlSearchMigration.php
@@ -8,30 +8,52 @@ use Doctrine\DBAL\Connection;
 
 class ExtendTlSearchMigration implements MigrationInterface
 {
-    public function __construct(private readonly Connection $connection)
-    {
+    public function __construct(
+        private readonly Connection $connection
+    ) {
     }
 
+    /**
+     * Eindeutiger Name der Migration
+     */
     public function getName(): string
     {
         return 'mummert_media_extend_tl_search';
     }
 
+    /**
+     * Läuft nur, solange mindestens eine der Spalten fehlt
+     */
     public function shouldRun(): bool
     {
-        // Wir lassen MySQL entscheiden – kein Schema-Precheck
-        return true;
+        $existingColumns = (int) $this->connection->fetchOne(
+            <<<SQL
+SELECT COUNT(*)
+FROM INFORMATION_SCHEMA.COLUMNS
+WHERE TABLE_SCHEMA = DATABASE()
+  AND TABLE_NAME = 'tl_search'
+  AND COLUMN_NAME IN ('keywords', 'priority', 'imagepath', 'startDate')
+SQL
+        );
+
+        // Wir erwarten genau 4 Spalten
+        return $existingColumns < 4;
     }
 
+    /**
+     * Fügt die Spalten hinzu (idempotent)
+     */
     public function run(): MigrationResult
     {
-        $this->connection->executeStatement(<<<SQL
+        $this->connection->executeStatement(
+            <<<SQL
 ALTER TABLE tl_search
-    ADD COLUMN IF NOT EXISTS keywords varchar(255) NOT NULL DEFAULT '',
-    ADD COLUMN IF NOT EXISTS priority int(1) NOT NULL DEFAULT 2,
-    ADD COLUMN IF NOT EXISTS imagepath varchar(512) NOT NULL DEFAULT '',
-    ADD COLUMN IF NOT EXISTS startDate bigint(20) NOT NULL DEFAULT 0
-SQL);
+    ADD COLUMN IF NOT EXISTS keywords   varchar(255) NOT NULL DEFAULT '',
+    ADD COLUMN IF NOT EXISTS priority   int(1)       NOT NULL DEFAULT 2,
+    ADD COLUMN IF NOT EXISTS imagepath  varchar(512) NOT NULL DEFAULT '',
+    ADD COLUMN IF NOT EXISTS startDate  bigint(20)   NOT NULL DEFAULT 0
+SQL
+        );
 
         return new MigrationResult(
             true,