From a894ec048b4d81cd1565ebed3cabb855697d8dd8 Mon Sep 17 00:00:00 2001
From: Jan-Hendrik Willms <tleilax+studip@gmail.com>
Date: Mon, 17 Feb 2025 07:06:50 +0000
Subject: add migration that acutally does what 5.4.6 should have done, fixes
 #5129

Closes #5129

Merge request studip/studip!3936
---
 db/migrations/5.4.19_fix_tree_changes.php | 20 ++++++++++++++++++++
 db/migrations/5.4.6_tree_changes.php      |  4 +---
 2 files changed, 21 insertions(+), 3 deletions(-)
 create mode 100644 db/migrations/5.4.19_fix_tree_changes.php

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'
-- 
cgit v1.0