From cc5f46c00e9deb75f89bd9a7a4999e5af1831843 Mon Sep 17 00:00:00 2001
From: Caroline Schulte <Caroline.schulte@uni-oldenburg.de>
Date: Tue, 30 Jun 2026 20:25:36 +0200
Subject: [PATCH] =?UTF-8?q?Review=C3=A4nderungen?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 content/confirm-password.php                  | 61 ++++-------------
 content/confirm-register.php                  | 65 ++++---------------
 content/login.php                             |  2 +
 data/mails/.gitkeep.php                       |  1 -
 data/pending-password/.gitkeep.php            |  1 -
 data/pending/.gitkeep.php                     |  1 -
 includes/alertMessages.php                    | 11 ++++
 index.php                                     | 10 ++-
 .../confirm-password-controller.php           | 65 +++++++++++++++++++
 .../confirm-register-controller.php           | 64 ++++++++++++++++++
 10 files changed, 177 insertions(+), 104 deletions(-)
 delete mode 100644 data/mails/.gitkeep.php
 delete mode 100644 data/pending-password/.gitkeep.php
 delete mode 100644 data/pending/.gitkeep.php
 create mode 100644 php/controller/confirm-password-controller.php
 create mode 100644 php/controller/confirm-register-controller.php

diff --git a/content/confirm-password.php b/content/confirm-password.php
index 67ecccd..bd51e41 100644
--- a/content/confirm-password.php
+++ b/content/confirm-password.php
@@ -1,55 +1,20 @@
 <?php
+/** @var string $title */
+/** @var string $message */
+/** @var string $link */
+/** @var bool $isSuccess */
 
-require_once "php/model/UserManager.php";
-
-$token = basename($_GET["token"] ?? "");
-$file = "data/pending-password/" . $token . ".json";
-
-$title = "Passwort zurücksetzen";
-$message = "";
-$link = "";
-$isSuccess = false;
-
-if (!file_exists($file)) {
-    $message = "Der Bestätigungslink ist ungültig oder bereits abgelaufen.";
-} else {
-    $data = json_decode(file_get_contents($file), true);
-
-    if ($data === null || empty($data["email"]) || empty($data["password"])) {
-        $message = "Die Daten zur Passwortänderung konnten nicht gelesen werden.";
-    } else {
-        try {
-            $dao = UserManager::getInstance();
-            $user = $dao->findUser($data["email"]);
-
-            if ($user !== null) {
-                $dao->updateUser(
-                    $user["email"],
-                    $user["email"],
-                    $user["vorname"],
-                    $user["nachname"],
-                    $data["password"]
-                );
-
-                unlink($file);
-
-                $title = "Passwort geändert";
-                $message = "Ihr Passwort wurde erfolgreich geändert. Sie können sich jetzt anmelden.";
-                $link = '<a class="button confirm-button" href="index.php?pfad=login">Zum Login</a>';
-                $isSuccess = true;
-            } else {
-                unlink($file);
-
-                $message = "Der Benutzer konnte nicht gefunden werden.";
-            }
-
-        } catch (Exception $e) {
-            $message = "Das Passwort konnte nicht geändert werden.";
-        }
-    }
-}
+$title = $title ?? "Passwort zurücksetzen";
+$message = $message ?? "Der Bestätigungslink konnte nicht verarbeitet werden.";
+$link = $link ?? "";
+$isSuccess = $isSuccess ?? false;
 ?>
 
+<!--
+    Ansicht: Passwortbestätigung
+    Funktion: Zeigt das Ergebnis der Passwortänderung an.
+-->
+
 <main class="login-page">
     <div class="login-container">
 
diff --git a/content/confirm-register.php b/content/confirm-register.php
index 9b074ab..43fd73b 100644
--- a/content/confirm-register.php
+++ b/content/confirm-register.php
@@ -1,58 +1,19 @@
 <?php
+/** @var string $title */
+/** @var string $message */
+/** @var string $link */
+/** @var bool $isSuccess */
 
-require_once "php/model/UserManager.php";
-
-$token = basename($_GET["token"] ?? "");
-$file = "data/pending/" . $token . ".json";
-
-$title = "Registrierung";
-$message = "";
-$link = "";
-$isSuccess = false;
-
-if (!file_exists($file)) {
-    $message = "Der Registrierungslink ist ungültig oder bereits abgelaufen.";
-} else {
-
-    $data = json_decode(file_get_contents($file), true);
-
-    if ($data === null) {
-        $message = "Die Registrierungsdaten konnten nicht gelesen werden.";
-    } elseif (
-        empty($data["email"]) ||
-        empty($data["vorname"]) ||
-        empty($data["nachname"]) ||
-        empty($data["password"])
-    ) {
-        $message = "Die Registrierungsdaten sind unvollständig.";
-    } else {
-
-        try {
-            $dao = UserManager::getInstance();
-
-            if ($dao->findUser($data["email"]) === null) {
-                $dao->addUser(
-                    $data["email"],
-                    $data["vorname"],
-                    $data["nachname"],
-                    $data["password"]
-                );
-            }
-
-            unlink($file);
-
-            $title = "Registrierung erfolgreich";
-            $message = "Ihre Registrierung wurde erfolgreich abgeschlossen. Sie können sich jetzt anmelden.";
-            $link = '<a class="button confirm-button" href="index.php?pfad=login">Zum Login</a>';
-            $isSuccess = true;
-
-        } catch (Exception $e) {
-            $message = "Die Registrierung konnte nicht abgeschlossen werden.";
-        }
-    }
-}
+$title = $title ?? "Registrierung";
+$message = $message ?? "Der Registrierungslink konnte nicht verarbeitet werden.";
+$link = $link ?? "";
+$isSuccess = $isSuccess ?? false;
 ?>
 
+<!--
+    Ansicht: Registrierungsbestätigung
+    Funktion: Zeigt das Ergebnis der Registrierungsbestätigung an.
+-->
 <main class="login-page">
     <div class="login-container">
 
@@ -65,4 +26,4 @@ if (!file_exists($file)) {
         <?php echo $link; ?>
 
     </div>
-</main>
+</main>
\ No newline at end of file
diff --git a/content/login.php b/content/login.php
index c7db755..8a47590 100644
--- a/content/login.php
+++ b/content/login.php
@@ -11,6 +11,8 @@ $error = $error ?? null;
 
         <h1>Bitte anmelden</h1>
 
+        <?php include_once "includes/alertMessages.php"; ?>
+
         <?php if (!empty($error)): ?>
             <p class="alert-message is-error">
                 <?php echo htmlspecialchars($error); ?>
diff --git a/data/mails/.gitkeep.php b/data/mails/.gitkeep.php
deleted file mode 100644
index b3d9bbc..0000000
--- a/data/mails/.gitkeep.php
+++ /dev/null
@@ -1 +0,0 @@
-<?php
diff --git a/data/pending-password/.gitkeep.php b/data/pending-password/.gitkeep.php
deleted file mode 100644
index b3d9bbc..0000000
--- a/data/pending-password/.gitkeep.php
+++ /dev/null
@@ -1 +0,0 @@
-<?php
diff --git a/data/pending/.gitkeep.php b/data/pending/.gitkeep.php
deleted file mode 100644
index b3d9bbc..0000000
--- a/data/pending/.gitkeep.php
+++ /dev/null
@@ -1 +0,0 @@
-<?php
diff --git a/includes/alertMessages.php b/includes/alertMessages.php
index e6112da..3f848ed 100644
--- a/includes/alertMessages.php
+++ b/includes/alertMessages.php
@@ -78,6 +78,17 @@
         Das Bild konnte nicht hochgeladen werden. Bitte versuche es erneut oder verwende ein anderes Bildformat.
     </p>
 <?php endif; ?>
+<?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "registration_confirmed"): ?>
+    <p class="alert-message is-success">
+        Ihre Registrierung wurde erfolgreich abgeschlossen. Sie können sich jetzt anmelden.
+    </p>
+<?php endif; ?>
+
+<?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "password_changed"): ?>
+    <p class="alert-message is-success">
+        Ihr Passwort wurde erfolgreich geändert. Sie können sich jetzt anmelden.
+    </p>
+<?php endif; ?>
 <?php
 unset($_SESSION["message"]);
 ?>
diff --git a/index.php b/index.php
index f06eeb4..0651795 100644
--- a/index.php
+++ b/index.php
@@ -18,7 +18,15 @@ if ($pfad === "register") {
     include_once "php/controller/register-controller.php";
 }
 if ($pfad === "password-forgotten") {
-    include_once "php/controller/password-forgotten-controller.php";
+include_once "php/controller/password-forgotten-controller.php";
+}
+
+if ($pfad === "confirm-register") {
+    include_once "php/controller/confirm-register-controller.php";
+}
+
+if ($pfad === "confirm-password") {
+    include_once "php/controller/confirm-password-controller.php";
 }
 
 if ($pfad === "logout") {
diff --git a/php/controller/confirm-password-controller.php b/php/controller/confirm-password-controller.php
new file mode 100644
index 0000000..0f72e99
--- /dev/null
+++ b/php/controller/confirm-password-controller.php
@@ -0,0 +1,65 @@
+<?php
+
+require_once "php/model/UserManager.php";
+
+/*
+    Verarbeitung der Passwortbestätigung
+    Funktion: Liest die zwischengespeicherten Passwortdaten aus,
+    aktualisiert das Passwort des Benutzers und gibt das Ergebnis
+    für die View zurück.
+*/
+
+$title = "Passwort zurücksetzen";
+$message = "";
+$link = "";
+$isSuccess = false;
+
+$token = basename($_GET["token"] ?? "");
+$file = "data/pending-password/" . $token . ".json";
+
+if (empty($token) || !file_exists($file)) {
+
+    $message = "Der Bestätigungslink ist ungültig oder bereits abgelaufen.";
+
+} else {
+
+    $data = json_decode(file_get_contents($file), true);
+
+    if ($data === null || empty($data["email"]) || empty($data["password"])) {
+
+        $message = "Die Daten zur Passwortänderung konnten nicht gelesen werden.";
+
+    } else {
+
+        try {
+
+            $dao = UserManager::getInstance();
+            $user = $dao->findUser($data["email"]);
+
+            if ($user !== null) {
+
+                $dao->updateUser(
+                    $user["email"],
+                    $user["email"],
+                    $user["vorname"],
+                    $user["nachname"],
+                    $data["password"]
+                );
+
+                $_SESSION["message"] = "password_changed";
+                header("Location: index.php?pfad=login");
+                exit();
+
+            } else {
+
+                $message = "Der Benutzer konnte nicht gefunden werden.";
+            }
+
+            unlink($file);
+
+        } catch (Exception $e) {
+
+            $message = "Das Passwort konnte nicht geändert werden.";
+        }
+    }
+}
diff --git a/php/controller/confirm-register-controller.php b/php/controller/confirm-register-controller.php
new file mode 100644
index 0000000..de47639
--- /dev/null
+++ b/php/controller/confirm-register-controller.php
@@ -0,0 +1,64 @@
+<?php
+
+require_once "php/model/UserManager.php";
+
+/*
+    Verarbeitung der Registrierungsbestätigung
+    Funktion: Liest die zwischengespeicherten Registrierungsdaten aus,
+    legt den Benutzer an und gibt das Ergebnis für die View zurück.
+*/
+
+$title = "Registrierung";
+$message = "";
+$link = "";
+$isSuccess = false;
+
+$token = basename($_GET["token"] ?? "");
+$file = "data/pending/" . $token . ".json";
+
+if (empty($token) || !file_exists($file)) {
+
+    $message = "Der Registrierungslink ist ungültig oder bereits abgelaufen.";
+
+} else {
+
+    $data = json_decode(file_get_contents($file), true);
+
+    if (
+        $data === null ||
+        empty($data["email"]) ||
+        empty($data["vorname"]) ||
+        empty($data["nachname"]) ||
+        empty($data["password"])
+    ) {
+
+        $message = "Die Registrierungsdaten konnten nicht gelesen werden.";
+
+    } else {
+
+        try {
+
+            $dao = UserManager::getInstance();
+
+            if ($dao->findUser($data["email"]) === null) {
+
+                $dao->addUser(
+                    $data["email"],
+                    $data["vorname"],
+                    $data["nachname"],
+                    $data["password"]
+                );
+            }
+
+            unlink($file);
+
+            $_SESSION["message"] = "registration_confirmed";
+            header("Location: index.php?pfad=login");
+            exit();
+
+        } catch (Exception $e) {
+
+            $message = "Die Registrierung konnte nicht abgeschlossen werden.";
+        }
+    }
+}
\ No newline at end of file