From 422b4b1914e9ac58263ce05b019921d876edd262 Mon Sep 17 00:00:00 2001
From: Jan-Hendrik Willms <tleilax+studip@gmail.com>
Date: Fri, 17 Jun 2022 13:17:46 +0000
Subject: [PATCH] fix redirect after editing configuration entries, fixes #1182

Closes #1182

Merge request studip/studip!702
---
 app/controllers/admin/configuration.php | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/app/controllers/admin/configuration.php b/app/controllers/admin/configuration.php
index f136ad4d26a..c61446a0042 100644
--- a/app/controllers/admin/configuration.php
+++ b/app/controllers/admin/configuration.php
@@ -46,10 +46,8 @@ class Admin_ConfigurationController extends AuthenticatedController
 
     /**
      * Maintenance view for the configuration parameters
-     *
-     * @param mixed $section Open section
      */
-    public function configuration_action($open_section = null)
+    public function configuration_action()
     {
         PageLayout::setTitle(_('Verwaltung von Systemkonfigurationen'));
 
@@ -67,7 +65,7 @@ class Admin_ConfigurationController extends AuthenticatedController
 
         // set variables for view
         $this->only_section = $section;
-        $this->open_section = $open_section ?? $section;
+        $this->open_section = Request::get('open_section', $section);
         $this->needle = $needle;
         $this->sections = ConfigurationModel::getConfig($section, $needle);
 
@@ -103,7 +101,7 @@ class Admin_ConfigurationController extends AuthenticatedController
                     htmlReady($field)
                 ));
 
-                $this->relocate("admin/configuration/configuration/{$section}#field-{$field}");
+                $this->relocate($this->action_url("configuration#field-{$field}", ['open_section' => $section]));
                 return;
             }
         }
-- 
GitLab