From 8f6a427babce4474c95e1ef2424a89d5f5f12ddf Mon Sep 17 00:00:00 2001
From: Elmar Ludwig <elmar.ludwig@uni-osnabrueck.de>
Date: Thu, 23 Oct 2025 12:57:12 +0200
Subject: always update priority (to force consistency), fixes #5968

Closes #5968

Merge request studip/studip!4573
---
 app/controllers/settings/statusgruppen.php | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/app/controllers/settings/statusgruppen.php b/app/controllers/settings/statusgruppen.php
index 789d0ca..9427986 100644
--- a/app/controllers/settings/statusgruppen.php
+++ b/app/controllers/settings/statusgruppen.php
@@ -284,19 +284,20 @@ class Settings_StatusgruppenController extends Settings_SettingsController
                 $changed                                = true;
             }
 
-            if ($changed) {
-                $query = "UPDATE user_inst
-                          SET priority = ?
-                          WHERE user_id = ? AND Institut_id = ?";
-                $statement = DBManager::get()->prepare($query);
+            $query = "UPDATE user_inst
+                      SET priority = ?
+                      WHERE user_id = ? AND Institut_id = ?";
+            $statement = DBManager::get()->prepare($query);
 
-                foreach ($priorities as $id => $priority) {
-                    $statement->execute([
-                        $priority,
-                        $this->user->user_id,
-                        $id,
-                    ]);
-                }
+            foreach ($priorities as $id => $priority) {
+                $statement->execute([
+                    $priority,
+                    $this->user->user_id,
+                    $id,
+                ]);
+            }
+
+            if ($changed) {
                 PageLayout::postSuccess(_('Reihenfolge wurde geändert'));
             }
         }
-- 
cgit v1.0