From 319a73ba74ac2d71dc3cc1d76537d6a6368a0acd Mon Sep 17 00:00:00 2001
From: Jan-Hendrik Willms <tleilax+studip@gmail.com>
Date: Fri, 11 Jul 2025 11:12:47 +0200
Subject: fix typos and rename DUMMY_TEACHER_ID to DUMMY_LECTURER_ID, fixes
 #5716

---
 app/views/login/activate_email.php                 |  6 ++--
 app/views/lti/_tool_form_fields.php                |  6 ++--
 ...0.52_rename_dummy_teacher_to_dummy_lecturer.php | 34 ++++++++++++++++++++++
 lib/classes/UserManagement.php                     | 23 ++++++++-------
 4 files changed, 53 insertions(+), 16 deletions(-)
 create mode 100644 db/migrations/6.0.52_rename_dummy_teacher_to_dummy_lecturer.php

diff --git a/app/views/login/activate_email.php b/app/views/login/activate_email.php
index 366990a..84a978e 100644
--- a/app/views/login/activate_email.php
+++ b/app/views/login/activate_email.php
@@ -2,12 +2,12 @@
     <form action="<?= URLHelper::getLink() ?>" method="post" class="default">
         <fieldset>
         <legend>
-            <?= _('Sie haben Ihre E-Mail-Adresse geändert.
-        Um diese frei zu schalten müssen Sie den Ihnen an Ihre neue Adresse zugeschickten Aktivierungs Schlüssel im unten stehenden Eingabefeld eintragen.') ?>
+            <?= _('Sie haben Ihre E-Mail-Adresse geändert.') ?>
+            <?= _('Um diese freizuschalten, tragen Sie im folgenden Eingabefeld den AKtivierungsschlüssel ein, der an Ihre neue Adresse geschickt wurde.') ?>
         </legend>
         <?= CSRFProtection::tokenTag() ?>
             <label>
-                <?=_('Aktivierungs Schlüssel')?>
+                <?=_('Aktivierungsschlüssel')?>
                 <input type="text" name="key">
             </label>
             <input name="uid" type="hidden" value="<?= htmlReady(Request::option('uid')) ?>">
diff --git a/app/views/lti/_tool_form_fields.php b/app/views/lti/_tool_form_fields.php
index d1dccec..51e4932 100644
--- a/app/views/lti/_tool_form_fields.php
+++ b/app/views/lti/_tool_form_fields.php
@@ -19,14 +19,14 @@
     <label>
         <?= _('Datenschutzhinweise') ?>
         <textarea name="data_protection_notes" class="wysiwyg"
-                  placeholder="<?= _('Bitte machen Sie Angaben zu dem angebundenen Werkzeug, soweit sie ihnen bekannt sind. Wie ist der Name, wer bietet es an, wozu wird es eingesetzt und welche Daten werden übertragen? (Beispiel: „Tool XY wird zur Durchführung von Sprachtests genutzt und Testergebnisse und ggf. Noten gespeichert. Zur Anmeldung werden Name und Nutzerkennung übertragen.“)') ?>"><?= wysiwygReady($tool->data_protection_notes) ?></textarea>
+                  placeholder="<?= _('Bitte machen Sie Angaben zu dem angebundenen Werkzeug, soweit sie ihnen bekannt sind. Wie ist der Name, wer bietet es an, wozu wird es eingesetzt und welche Daten werden übertragen? (Beispiel: „Tool XY wird zur Durchführung von Sprachtests genutzt und Testergebnisse und ggf. Noten werden gespeichert. Zur Anmeldung werden Name und Nutzerkennung übertragen.“)') ?>"><?= wysiwygReady($tool->data_protection_notes) ?></textarea>
     </label>
     <label>
-        <?= _('URL zu den Nutzungsbedingungen des LTI-Tools (falls verfügbar)') ?>
+        <?= _('Link zu den Nutzungsbedingungen des LTI-Tools (falls verfügbar)') ?>
         <input type="url" name="terms_of_use_url" value="<?= htmlReady($tool->terms_of_use_url) ?>">
     </label>
     <label>
-        <?= _('URL zur Datenschutzerklärung des LTI-Tools (falls verfügbar)') ?>
+        <?= _('Link zur Datenschutzerklärung des LTI-Tools (falls verfügbar)') ?>
         <input type="url" name="privacy_policy_url" value="<?= htmlReady($tool->privacy_policy_url) ?>">
     </label>
 </fieldset>
diff --git a/db/migrations/6.0.52_rename_dummy_teacher_to_dummy_lecturer.php b/db/migrations/6.0.52_rename_dummy_teacher_to_dummy_lecturer.php
new file mode 100644
index 0000000..15b2b64
--- /dev/null
+++ b/db/migrations/6.0.52_rename_dummy_teacher_to_dummy_lecturer.php
@@ -0,0 +1,34 @@
+<?php
+final class RenameDummyTeacherToDummyLecturer extends Migration
+{
+    public function description()
+    {
+        return 'Renames configuration DUMMY_TEACHER_ID to DUMMY_LECTURER_ID';
+    }
+
+    public function up()
+    {
+        $query = "UPDATE `config`
+                  SET `field` = 'DUMMY_LECTURER_ID'
+                  WHERE `field` = 'DUMMY_TEACHER_ID'";
+        DBManager::get()->exec($query);
+
+        $query = "UPDATE `config_values`
+                  SET `field` = 'DUMMY_LECTURER_ID'
+                  WHERE `field` = 'DUMMY_TEACHER_ID'";
+        DBManager::get()->exec($query);
+    }
+
+    protected function down()
+    {
+        $query = "UPDATE `config_values`
+                  SET `field` = 'DUMMY_TEACHER_ID'
+                  WHERE `field` = 'DUMMY_LECTURER_ID'";
+        DBManager::get()->exec($query);
+
+        $query = "UPDATE `config`
+                  SET `field` = 'DUMMY_TEACHER_ID'
+                  WHERE `field` = 'DUMMY_LECTURER_ID'";
+        DBManager::get()->exec($query);
+    }
+}
diff --git a/lib/classes/UserManagement.php b/lib/classes/UserManagement.php
index 255bba1..1170acc 100644
--- a/lib/classes/UserManagement.php
+++ b/lib/classes/UserManagement.php
@@ -689,14 +689,14 @@ class UserManagement
         // new users alawys receive a link to generate a password
         if ($new) {
             $prefix = ltrim(Config::get()->MAIL_SUBJECT_PREFIX . ' ');
-            $subject = $prefix . _('Es wurde ein Zugang für sie erstellt - Setzen sie ein Passwort');
+            $subject = $prefix . _('Es wurde ein Zugang für Sie erstellt - setzen Sie ein Passwort');
 
             $mailbody = sprintf(
                 _("Dies ist eine Bestätigungsmail des Stud.IP-Systems\n"
                     ."(Studienbegleitender Internetsupport von Präsenzlehre)\n- %1\$s -\n\n"
-                    ."Es wurde für sie ein Zugang zum System erstellt, Ihr Nutzername lautet:\n\n"
+                    ."Es wurde für Sie ein Zugang zum System erstellt, Ihr Nutzername lautet:\n\n"
                     ."%2\$s\n\n"
-                    ."Um den Zugang nutzen zu können, müssen sie ein Passwort setzen.\n"
+                    ."Um den Zugang nutzen zu können, müssen Sie ein Passwort setzen.\n"
                     ."Öffnen Sie dafür bitte folgenden Link\n\n"
                     ."%3\$s\n\n"
                     ."in Ihrem Browser.\n\n"
@@ -727,7 +727,7 @@ class UserManagement
                     . "um das Passwort zurückzusetzen.\n"
                     . "Dies ist aber für den mit dieser Mail \n"
                     . "verknüpften Account so nicht möglich.\n\n"
-                    . "Wenden sie sich bitte stattdessen an\n%s"
+                    . "Wenden Sie sich bitte stattdessen an\n%s"
                 ),
                 Config::get()->UNI_NAME_CLEAN,
                 $GLOBALS['UNI_CONTACT']
@@ -813,10 +813,13 @@ class UserManagement
         }
 
         // Check if dummy teacher exists
-        if (!Config::get()->DUMMY_TEACHER_ID || !User::find(Config::get()->DUMMY_TEACHER_ID)) {
+        if (
+            !Config::get()->getValue('DUMMY_LECTURER_ID')
+            || !User::find(Config::get()->getValue('DUMMY_LECTURER_ID'))
+        ) {
             $this->msg .= 'error§' . sprintf(
-                _('Dummy-Dozent (id: %s) nicht gefunden. Bitte DUMMY_TEACHER_ID in Konfiguration setzen.'),
-                Config::get()->DUMMY_TEACHER_ID) . '§';
+                _('Dummy-Dozent (id: %s) nicht gefunden. Bitte DUMMY_LECTURER_ID in Konfiguration setzen.'),
+                Config::get()->getValue('DUMMY_LECTURER_ID')) . '§';
             return false;
         }
 
@@ -828,7 +831,7 @@ class UserManagement
             HAVING COUNT(DISTINCT user_id) = 1 AND user_id = ?",
             [
                 studygroup_sem_types(),
-                Config::get()->DUMMY_TEACHER_ID,
+                Config::get()->getValue('DUMMY_LECTURER_ID'),
                 $this->user->id
             ]
         );
@@ -848,7 +851,7 @@ class UserManagement
             HAVING COUNT(DISTINCT user_id) = 1 AND user_id = ?",
             [
                 studygroup_sem_types(),
-                Config::get()->DUMMY_TEACHER_ID,
+                Config::get()->getValue('DUMMY_LECTURER_ID'),
                 $this->user->id
             ]
         );
@@ -859,7 +862,7 @@ class UserManagement
             ) . '§';
             CourseMember::insertCourseMember(
                 $member->course->id,
-                Config::get()->DUMMY_TEACHER_ID,
+                Config::get()->getValue('DUMMY_LECTURER_ID'),
                 'dozent'
             );
 
-- 
cgit v1.0