add migration that acutally does what 5.4.6 should have done, fixes #5129

Closes #5129

Merge request studip/studip!3936

2 files changed, 21 insertions, 3 deletions

diff --git a/db/migrations/5.4.19_fix_tree_changes.php b/db/migrations/5.4.19_fix_tree_changes.php
new file mode 100644
index 0000000..b4631ed
--- /dev/null
+++ b/db/migrations/5.4.19_fix_tree_changes.php
@@ -0,0 +1,20 @@
+<?php
+final class FixTreeChanges extends Migration
+{
+    public function description()
+    {
+        return 'Actually removes the config entries from 5.4.6 that was faulty at first';
+    }
+
+    protected function up()
+    {
+        $query = "DELETE `config`, `config_values`
+                  FROM `config`
+                  LEFT JOIN `config_values` USING (`field`)
+                  WHERE `field` IN (
+                    'RANGE_TREE_ADMIN_PERM',
+                    'SEM_TREE_ADMIN_PERM'
+                  )";
+        DBManager::get()->exec($query);
+    }
+}
diff --git a/db/migrations/5.4.6_tree_changes.php b/db/migrations/5.4.6_tree_changes.php
index 03a0ea3..aaef5b5 100644
--- a/db/migrations/5.4.6_tree_changes.php
+++ b/db/migrations/5.4.6_tree_changes.php
@@ -1,8 +1,6 @@
-<?
-
+<?php
 final class TreeChanges extends Migration
 {
-
     const FIELDS = [
         'RANGE_TREE_ADMIN_PERM',
         'SEM_TREE_ADMIN_PERM'