fix for Contao 5.6

Author: bennyborn

composer.json

@@ -11,7 +11,7 @@
     ],
     "require": {
         "contao/core-bundle": "^4.13 || ^5.0",
-        "doctrine/dbal": "^3.3",
+        "doctrine/dbal": "^3.6 || ^4.3",
         "psr/log": "^1.1 || 2.0 || ^3.0",
         "symfony/config": "^5.4 || ^6.4 || ^7.0",
         "symfony/console": "^5.4 || ^6.4 || ^7.0",

src/EventListener/DataContainer/ModuleListener.php

@@ -138,21 +138,20 @@ public function getTags( DataContainer $dc ): array {
         $result = null;
         if( $ptable ) {
 
-            $result = $this->connection
-                ->prepare(
-                    "SELECT DISTINCT tag.id, tag.tag
-                    FROM $tTag AS tag
-                    JOIN $tRel AS rel ON (rel.tag_id=tag.id AND rel.ptable=:ptable AND rel.field=:field)
-                    ORDER BY tag.tag ASC")
-                ->executeQuery(['ptable'=>$ptable, 'field'=>'tags'])
-            ;
+            $result = $this->connection->executeQuery(
+                "SELECT DISTINCT tag.id, tag.tag
+                FROM $tTag AS tag
+                JOIN $tRel AS rel ON (rel.tag_id=tag.id AND rel.ptable=:ptable AND rel.field=:field)
+                ORDER BY tag.tag ASC"
+            ,   ['ptable'=>$ptable, 'field'=>'tags']
+            );
         }
 
         $tags = [];
 
         if( $result && $result->rowCount() ) {
 
-            $rows = $result->fetchAll();
+            $rows = $result->fetchAllAssociative();
 
             foreach( $rows as $row ) {
                 $tags[$row['id']] = $row['tag'];

src/EventListener/DataContainer/TagsListener.php

@@ -170,10 +170,10 @@ public function saveTags( $varValue, DataContainer $dc ): ?string {
         $tRel = TagsRelModel::getTable();
 
         // remove all tag relations for this element
-        $this->connection
-            ->prepare("DELETE FROM $tRel WHERE pid=? AND ptable=? AND field=?")
-            ->executeStatement([$dc->activeRecord->id, $dc->table, $dc->field])
-        ;
+        $this->connection->executeStatement(
+            "DELETE FROM $tRel WHERE pid=? AND ptable=? AND field=?"
+        ,   [$dc->activeRecord->id, $dc->table, $dc->field]
+        );
 
         if( !empty($varValue) ) {
 
@@ -182,10 +182,10 @@ public function saveTags( $varValue, DataContainer $dc ): ?string {
             // add tag relations for this element
             foreach( $tags as $i => $id ) {
 
-                $this->connection
-                    ->prepare("INSERT INTO $tRel (tag_id, pid, ptable, field) VALUES (?,?,?,?)")
-                    ->executeStatement([$id, $dc->activeRecord->id, $dc->table, $dc->field])
-                ;
+                $this->connection->executeStatement(
+                    "INSERT INTO $tRel (tag_id, pid, ptable, field) VALUES (?,?,?,?)"
+                ,   [$id, $dc->activeRecord->id, $dc->table, $dc->field]
+                );
 
                 // explicitly cast the id into a string, otherwise the filter options in the backend won't work
                 $tags[$i] = (string)$id;
@@ -267,21 +267,21 @@ public function mergeTagSelectButton( $buttons, DataContainer $dc ): array {
 
                             if( $rel['tag_id'] != $newId ) {
 
-                                $this->connection
-                                    ->prepare("INSERT INTO $tRel (tag_id, pid, ptable, field) VALUES (?,?,?,?)")
-                                    ->executeStatement([$newId, $rel['pid'], $rel['ptable'], $rel['field']])
-                                ;
+                                $this->connection->executeStatement(
+                                    "INSERT INTO $tRel (tag_id, pid, ptable, field) VALUES (?,?,?,?)"
+                                ,   [$newId, $rel['pid'], $rel['ptable'], $rel['field']]
+                                );
                             }
                         }
                     }
 
                     // delete rel for other tags
                     foreach( $rowsProcessed as $hash => $rel ) {
 
-                        $this->connection
-                            ->prepare("DELETE FROM $tRel WHERE tag_id!=? AND pid=? AND ptable=? AND field=?")
-                            ->executeStatement([$newId, $rel['pid'], $rel['ptable'], $rel['field']])
-                        ;
+                        $this->connection->executeStatement(
+                            "DELETE FROM $tRel WHERE tag_id!=? AND pid=? AND ptable=? AND field=?"
+                        ,   [$newId, $rel['pid'], $rel['ptable'], $rel['field']]
+                        );
                     }
 
                     // delete tag for other tags

src/Migration/Version007Update.php

@@ -6,7 +6,7 @@
  * @author    Benny Born <benny.born@numero2.de>
  * @author    Michael Bösherz <michael.boesherz@numero2.de>
  * @license   LGPL-3.0-or-later
- * @copyright Copyright (c) 2023, numero2 - Agentur für digitales Marketing GbR
+ * @copyright Copyright (c) 2025, numero2 - Agentur für digitales Marketing GbR
  */
 
 
@@ -40,14 +40,14 @@ public function __construct( ContaoFramework $framework, Connection $connection
 
         $this->framework = $framework;
         $this->framework->initialize();
-        
+
         $this->connection = $connection;
     }
 
 
     public function shouldRun(): bool {
 
-        $schemaManager = $this->connection->getSchemaManager();
+        $schemaManager = $this->connection->createSchemaManager();
 
         $t = TagsModel::getTable();
 
@@ -71,7 +71,7 @@ public function shouldRun(): bool {
                             if( !array_key_exists($dca,$this->fields) ) {
                                 $this->fields[$dca] = [];
                             }
-                            
+
                             if( in_array($field,$this->fields[$dca]) === false ) {
                                 $this->fields[$dca][] = $field;
                             }
@@ -94,7 +94,7 @@ public function shouldRun(): bool {
                     if( in_array($field, $columns) ) {
 
                         $res = $this->connection->prepare("SELECT 1 FROM $dca WHERE $field IS NOT NULL AND $field NOT LIKE '%:\"%'; ")->executeQuery();
-    
+
                         // return as soon as we found our first value in the wrong format
                         if( $res && $res->rowCount() ) {
                             return true;
@@ -123,15 +123,14 @@ public function run(): MigrationResult {
                         $value = StringUtil::deserialize($row[$field]);
 
                         if( !empty($value) ) {
-                            
+
                             // cast each id explicitly into a string
                             $value = array_map(fn($v): string => (string)$v, $value);
 
                             $row[$field] = serialize($value);
 
                             // update the row
-                            $updateStmt = $this->connection->prepare("UPDATE $dca SET $field = :$field WHERE id = :id");
-                            $updateStmt->execute($row);
+                            $updateStmt = $this->connection->executeStatement("UPDATE $dca SET $field = :$field WHERE id = :id", $row);
                         }
                     }
                 }