From 3b0a598f82235f8f51b35df1ecacc341adeb6136 Mon Sep 17 00:00:00 2001
From: Jan-Hendrik Willms <tleilax+studip@gmail.com>
Date: Fri, 27 Sep 2024 09:39:59 +0000
Subject: [PATCH] remove bogus code from renaming tools, re #4633

Merge request studip/studip!3449
---
 app/controllers/course/contentmodules.php  | 2 +-
 app/views/course/contentmodules/rename.php | 5 ++---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/app/controllers/course/contentmodules.php b/app/controllers/course/contentmodules.php
index ef2272a8893..c3d2269a8b6 100644
--- a/app/controllers/course/contentmodules.php
+++ b/app/controllers/course/contentmodules.php
@@ -215,7 +215,7 @@ class Course_ContentmodulesController extends AuthenticatedController
         $this->module = PluginManager::getInstance()->getPluginById($module_id);
         $this->metadata = $this->module->getMetadata();
 
-        $this->original_name = $this->metadata['diplayname'] ?? $this->module->getMetadata()['displayname'];
+        $this->original_name = $this->metadata['displayname'] ?? $this->module->getPluginName();
 
         PageLayout::setTitle(_('Werkzeug umbenennen'));
         $this->tool = ToolActivation::find([$context->id, $module_id]);
diff --git a/app/views/course/contentmodules/rename.php b/app/views/course/contentmodules/rename.php
index f92ef7b7cb3..5396409a98e 100644
--- a/app/views/course/contentmodules/rename.php
+++ b/app/views/course/contentmodules/rename.php
@@ -3,7 +3,6 @@
  * @var Course_ContentmodulesController $controller
  * @var StudipModule $module
  * @var string $original_name
- * @var array $metadata
  * @var ToolActivation $tool
  */
 ?>
@@ -16,7 +15,7 @@
             <?= _('Neuer Name des Werkzeugs') ?>
             <input type="text"
                    name="displayname"
-                   value="<?= htmlReady($tool['metadata']['displayname'] ?? '') ?>"
+                   value="<?= htmlReady($tool->metadata['displayname'] ?? '') ?>"
                    placeholder="<?= htmlReady($original_name) ?>">
         </label>
 
@@ -26,7 +25,7 @@
     </fieldset>
     <div data-dialog-button>
         <?= \Studip\Button::create(_('Speichern'))?>
-        <? if (!empty($tool['metadata']['displayname'])) : ?>
+        <? if (!empty($tool->metadata['displayname'])) : ?>
             <?= \Studip\Button::create(_('Namen löschen'), 'delete') ?>
         <? endif ?>
     </div>
-- 
GitLab