Merge pull request 'Abgabe03: Profilbearbeitung: Reviewnacharbeitung' (#23) from dev into Abgabe03

Reviewed-on: #23

content/profile.php

@@ -2,7 +2,7 @@
 include_once 'php/controller/profile-controller.php';
 
 $user = $user ?? null;
-$isEditMode = isset($_GET["edit"]) && $_GET["edit"] === "1";
+$isEditMode = (isset($_GET["edit"]) && $_GET["edit"] === "1") || !empty($error);
 ?>
 
 <main class="form-page">
@@ -10,8 +10,8 @@ $isEditMode = isset($_GET["edit"]) && $_GET["edit"] === "1";
 
         <div class="container">
 
-            <?php if (isset($error) && $error): ?>
+            <?php if (!empty($error)): ?>
-                <p style="color:red;">
+                <p class="alert-message is-error">
                     <?php echo htmlspecialchars($error); ?>
                 </p>
             <?php endif; ?>
@@ -22,7 +22,7 @@ $isEditMode = isset($_GET["edit"]) && $_GET["edit"] === "1";
                 <input type="text"
                        name="vorname"
                        class="login-input"
-                       value="<?php echo htmlspecialchars($user["vorname"] ?? ""); ?>"
+                       value="<?php echo htmlspecialchars($_POST["vorname"] ?? $user["vorname"] ?? ""); ?>"
                         <?php echo $isEditMode ? "" : "readonly"; ?>
                        required>
 
@@ -30,7 +30,7 @@ $isEditMode = isset($_GET["edit"]) && $_GET["edit"] === "1";
                 <input type="text"
                        name="nachname"
                        class="login-input"
-                       value="<?php echo htmlspecialchars($user["nachname"] ?? ""); ?>"
+                       value="<?php echo htmlspecialchars($_POST["nachname"] ?? $user["nachname"] ?? ""); ?>"
                         <?php echo $isEditMode ? "" : "readonly"; ?>
                        required>
 
@@ -38,18 +38,26 @@ $isEditMode = isset($_GET["edit"]) && $_GET["edit"] === "1";
                 <input type="email"
                        name="email"
                        class="login-input"
-                       value="<?php echo htmlspecialchars($user["email"] ?? ""); ?>"
+                       value="<?php echo htmlspecialchars($_POST["email"] ?? $user["email"] ?? ""); ?>"
                         <?php echo $isEditMode ? "" : "readonly"; ?>
                        required>
 
                 <label class="input-label">
                     <?php echo $isEditMode ? "Neues Passwort" : "Passwort"; ?>
                 </label>
+
+                <?php if ($isEditMode): ?>
                     <input type="password"
                            name="password"
                            class="login-input"
-                       placeholder="<?php echo $isEditMode ? "Leer lassen, wenn es gleich bleiben soll" : "********"; ?>"
+                           placeholder="Leer lassen, wenn es gleich bleiben soll">
-                        <?php echo $isEditMode ? "" : "readonly"; ?>>
+                <?php else: ?>
+                    <input type="password"
+                           name="password"
+                           class="login-input"
+                           value="********"
+                           readonly>
+                <?php endif; ?>
 
                 <br><br>
 

content/register.php

@@ -1,16 +1,18 @@
 <?php
-include_once 'php/controller/register-controller.php';
-
 $error = $error ?? null;
 ?>
 
+<!--
+    Form: Registrierung
+    Funktion: Erstellung neuer Benutzerkonten
+-->
 <main class="login-page">
     <div class="login-container">
 
         <h1>Jetzt Registrieren!</h1>
 
-        <?php if ($error): ?>
+        <?php if (!empty($error)): ?>
-            <p style="color:red;">
+            <p class="alert-message is-error" style="color:red;">
                 <?php echo htmlspecialchars($error); ?>
             </p>
         <?php endif; ?>
@@ -22,6 +24,7 @@ $error = $error ?? null;
                    name="email"
                    class="login-input"
                    placeholder="mustermann@web.de"
+                   value="<?php echo htmlspecialchars($_POST["email"] ?? ""); ?>"
                    required>
 
             <p class="input-label">Vorname:</p>
@@ -29,6 +32,7 @@ $error = $error ?? null;
                    name="vorname"
                    class="login-input"
                    placeholder="Max"
+                   value="<?php echo htmlspecialchars($_POST["vorname"] ?? ""); ?>"
                    required>
 
             <p class="input-label">Nachname:</p>
@@ -36,6 +40,7 @@ $error = $error ?? null;
                    name="nachname"
                    class="login-input"
                    placeholder="Mustermann"
+                   value="<?php echo htmlspecialchars($_POST["nachname"] ?? ""); ?>"
                    required>
 
             <p class="input-label">Passwort:</p>

index.php

@@ -1,4 +1,3 @@
-
 <?php
 session_start();
 ob_start();
@@ -52,8 +51,8 @@ if ($pfad === "deleteAccount") {
         <title>EduForge</title>
     </head>
 
+    <body>
 
-<body>
     <?php
     include_once 'includes/navbar.php';
 
@@ -61,20 +60,15 @@ if ($pfad === "deleteAccount") {
         Dynamischer Inhalt:
         Je nach pfad-Parameter wird die passende Datei aus content geladen.
     */
-    if (isset($_GET["pfad"])) {
+    if (file_exists('content/' . $pfad . '.php')) {
-        if (file_exists('content/' . $_GET["pfad"] . '.php')) {
+        include_once 'content/' . $pfad . '.php';
-            include_once 'content/' . $_GET["pfad"] . '.php';
     } else {
         include_once 'content/404.php';
     }
-    } else {
-        include_once 'content/home.php';
-    }
 
     include_once 'includes/footer.php';
     ?>
 
-
     </body>
     </html>
 

php/controller/profile-controller.php

@@ -1,8 +1,9 @@
 <?php
 
 require_once "php/model/LocalUserDAO.php";
-require_once 'php/model/Article.php';
+require_once "php/model/Article.php";
-require_once 'php/model/ArticleManager.php';
+require_once "php/model/ArticleManager.php";
+require_once "php/validator/user-validator.php";
 
 $error = null;
 
@@ -11,34 +12,6 @@ if (!isset($_SESSION["user"])) {
     exit();
 }
 
-/*
- * Prüft Vor- und Nachnamen.
- * Erlaubt sind Buchstaben, Umlaute, Leerzeichen und Bindestriche.
- */
-function isValidName($name): bool {
-    return preg_match("/^[a-zA-ZäöüÄÖÜß -]{2,50}$/", $name);
-}
-
-/*
- * Prüft, ob die E-Mail-Adresse ein gültiges Format hat.
- */
-function isValidEmailAddress($email): bool {
-    return filter_var($email, FILTER_VALIDATE_EMAIL) !== false
-        && strlen($email) <= 100;
-}
-
-/*
- * Prüft das neue Passwort.
- * Leeres Passwort ist erlaubt, wenn der Nutzer es nicht ändern möchte.
- */
-function isValidProfilePassword($password): bool {
-    if ($password === "") {
-        return true;
-    }
-
-    return strlen($password) >= 8 && strlen($password) <= 72;
-}
-
 try {
     $dao = new LocalUserDAO();
 
@@ -60,19 +33,22 @@ try {
         $nachname = trim($_POST["nachname"] ?? "");
         $password = $_POST["password"] ?? "";
 
-        if (!isValidName($vorname)) {
+        if (!userEmailValidator($newEmail)) {
-            $error = "Der Vorname darf nur Buchstaben, Leerzeichen und Bindestriche enthalten und muss 2 bis 50 Zeichen lang sein.";
-        } elseif (!isValidName($nachname)) {
-            $error = "Der Nachname darf nur Buchstaben, Leerzeichen und Bindestriche enthalten und muss 2 bis 50 Zeichen lang sein.";
-        } elseif (!isValidEmailAddress($newEmail)) {
             $error = "Bitte gib eine gültige E-Mail-Adresse ein.";
-        } elseif (!isValidProfilePassword($password)) {
+            $_GET["edit"] = "1";
-            $error = "Das Passwort muss mindestens 8 Zeichen lang sein.";
+
-        } else {
+        } elseif (!userNameValidator($vorname)) {
-            $existingUser = $dao->findUser($newEmail);
+            $error = "Der Vorname muss 2 bis 50 Zeichen lang sein und darf nur Buchstaben, Leerzeichen und Bindestriche enthalten.";
+            $_GET["edit"] = "1";
+
+        } elseif (!userNameValidator($nachname)) {
+            $error = "Der Nachname muss 2 bis 50 Zeichen lang sein und darf nur Buchstaben, Leerzeichen und Bindestriche enthalten.";
+            $_GET["edit"] = "1";
+
+        } elseif (!userOptionalPasswordValidator($password)) {
+            $error = "Das Passwort muss 8 bis 72 Zeichen lang sein.";
+            $_GET["edit"] = "1";
 
-            if ($existingUser && $newEmail !== $oldEmail) {
-                $error = "Diese E-Mail-Adresse wird bereits verwendet.";
         } else {
             $updated = $dao->updateUser(
                 $oldEmail,
@@ -90,7 +66,7 @@ try {
                 exit();
             } else {
                 $error = "Die Daten konnten nicht gespeichert werden.";
-                }
+                $_GET["edit"] = "1";
             }
         }
     }
@@ -104,6 +80,12 @@ try {
         $_SESSION["message"] = "user_has_no_articles";
     }
 
+} catch (InvalidArgumentException $e) {
+    $error = $e->getMessage();
+    $_GET["edit"] = "1";
+
 } catch (Exception $e) {
+    $error = "Es ist ein interner Fehler aufgetreten. Bitte versuche es erneut.";
     $_SESSION["message"] = "internal_error";
+    $_GET["edit"] = "1";
 }

php/controller/register-controller.php

@@ -1,37 +1,43 @@
 <?php
 
 require_once "php/model/LocalUserDAO.php";
+require_once "php/validator/user-validator.php";
 
-$dao = new LocalUserDAO();
 $error = null;
 
 if ($_SERVER["REQUEST_METHOD"] === "POST") {
 
-    $email = $_POST["email"] ?? "";
+    $email = trim($_POST["email"] ?? "");
-    $vorname = $_POST["vorname"] ?? "";
+    $vorname = trim($_POST["vorname"] ?? "");
-    $nachname = $_POST["nachname"] ?? "";
+    $nachname = trim($_POST["nachname"] ?? "");
-    $password = password_hash(
+    $plainPassword = $_POST["password"] ?? "";
-        $_POST["password"] ?? "",
-        PASSWORD_DEFAULT
-    );
-
-    if ($dao->findUser($email)) {
-
-        $error = "Diese E-Mail-Adresse ist bereits registriert.";
 
+    if (!userEmailValidator($email)) {
+        $error = "Bitte gib eine gültige E-Mail-Adresse ein.";
+    } elseif (!userNameValidator($vorname)) {
+        $error = "Der Vorname muss 2 bis 50 Zeichen lang sein und darf nur Buchstaben, Umlaute, Leerzeichen und Bindestriche enthalten.";
+    } elseif (!userNameValidator($nachname)) {
+        $error = "Der Nachname muss 2 bis 50 Zeichen lang sein und darf nur Buchstaben, Umlaute, Leerzeichen und Bindestriche enthalten.";
+    } elseif (!userPasswordValidator($plainPassword)) {
+        $error = "Das Passwort muss 8 bis 72 Zeichen lang sein.";
     } else {
+        try {
+            $dao = new LocalUserDAO();
 
-        $dao->addUser(
+            $password = password_hash($plainPassword, PASSWORD_DEFAULT);
-            $email,
+
-            $vorname,
+            $dao->addUser($email, $vorname, $nachname, $password);
-            $nachname,
-            $password
-        );
 
             $_SESSION["user"] = $vorname . " " . $nachname;
             $_SESSION["user_email"] = $email;
 
             header("Location: index.php");
             exit();
+
+        } catch (InvalidArgumentException $e) {
+            $error = $e->getMessage();
+        } catch (Exception $e) {
+            $error = "Die Registrierung konnte nicht gespeichert werden.";
+        }
     }
 }

php/model/LocalUserDAO.php

@@ -6,6 +6,13 @@ class LocalUserDAO implements UserDAOInterface {
 
     private string $file = "data/users.json";
 
+    /**
+     * Lädt alle Benutzer aus der JSON-Datei.
+     *
+     * @return array Liste aller Benutzer
+     * @throws RuntimeException wenn die Datei nicht gelesen werden kann
+     * oder fehlerhafte JSON-Daten enthält
+     */
     private function loadUsers() {
         if (!file_exists($this->file)) {
             return [];
@@ -26,40 +33,35 @@ class LocalUserDAO implements UserDAOInterface {
         return is_array($users) ? $users : [];
     }
 
+    /**
+     * Speichert alle Benutzer in die JSON-Datei.
+     *
+     * @param array $users Liste aller Benutzer
+     * @return void
+     * @throws RuntimeException wenn die Daten nicht gespeichert werden können
+     */
     private function saveUsers($users) {
-        $result = file_put_contents(
+        $json = json_encode($users, JSON_PRETTY_PRINT);
-            $this->file,
+
-            json_encode($users, JSON_PRETTY_PRINT)
+        if ($json === false) {
-        );
+            throw new RuntimeException("Benutzerdaten konnten nicht umgewandelt werden.");
+        }
+
+        $result = file_put_contents($this->file, $json);
 
         if ($result === false) {
             throw new RuntimeException("Benutzerdaten konnten nicht gespeichert werden.");
         }
     }
 
-    private function validateEmail($email) {
+    /**
-        if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
+     * Sucht einen Benutzer anhand seiner E-Mail-Adresse.
-            throw new InvalidArgumentException("Ungültige E-Mail-Adresse.");
+     *
-        }
+     * @param string $email E-Mail-Adresse des gesuchten Benutzers
-    }
+     * @return array|null Benutzerdaten oder null, wenn kein Benutzer gefunden wurde
-
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen werden können
-    private function validateName($name, $fieldName) {
+     */
-        if (!preg_match("/^[a-zA-ZäöüÄÖÜß -]{2,50}$/", $name)) {
-            throw new InvalidArgumentException(
-                $fieldName . " darf nur Buchstaben, Leerzeichen und Bindestriche enthalten."
-            );
-        }
-    }
-
-    private function validatePassword($password) {
-        if (empty($password)) {
-            throw new InvalidArgumentException("Passwort darf nicht leer sein.");
-        }
-    }
-
     public function findUser($email) {
-        $this->validateEmail($email);
-
         $users = $this->loadUsers();
 
         foreach ($users as $user) {
@@ -78,12 +80,18 @@ class LocalUserDAO implements UserDAOInterface {
         return null;
     }
 
+    /**
+     * Fügt einen neuen Benutzer hinzu.
+     *
+     * @param string $email E-Mail-Adresse
+     * @param string $vorname Vorname
+     * @param string $nachname Nachname
+     * @param string $password Passwort-Hash
+     * @return void
+     * @throws InvalidArgumentException wenn die E-Mail-Adresse bereits verwendet wird
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder gespeichert werden können
+     */
     public function addUser($email, $vorname, $nachname, $password) {
-        $this->validateEmail($email);
-        $this->validateName($vorname, "Vorname");
-        $this->validateName($nachname, "Nachname");
-        $this->validatePassword($password);
-
         $users = $this->loadUsers();
 
         foreach ($users as $user) {
@@ -102,12 +110,19 @@ class LocalUserDAO implements UserDAOInterface {
         $this->saveUsers($users);
     }
 
+    /**
+     * Aktualisiert einen bestehenden Benutzer.
+     *
+     * @param string $oldEmail Alte E-Mail-Adresse
+     * @param string $newEmail Neue E-Mail-Adresse
+     * @param string $vorname Neuer Vorname
+     * @param string $nachname Neuer Nachname
+     * @param string|null $password Neues Passwort oder null
+     * @return bool true, wenn der Benutzer aktualisiert wurde, sonst false
+     * @throws InvalidArgumentException wenn die neue E-Mail-Adresse bereits verwendet wird
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder gespeichert werden können
+     */
     public function updateUser($oldEmail, $newEmail, $vorname, $nachname, $password = null) {
-        $this->validateEmail($oldEmail);
-        $this->validateEmail($newEmail);
-        $this->validateName($vorname, "Vorname");
-        $this->validateName($nachname, "Nachname");
-
         $users = $this->loadUsers();
 
         foreach ($users as $user) {
@@ -140,9 +155,14 @@ class LocalUserDAO implements UserDAOInterface {
         return false;
     }
 
+    /**
+     * Löscht einen Benutzer anhand seiner E-Mail-Adresse.
+     *
+     * @param string $email E-Mail-Adresse des zu löschenden Benutzers
+     * @return bool true, wenn der Benutzer gelöscht wurde, sonst false
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder gespeichert werden können
+     */
     public function deleteUser($email) {
-        $this->validateEmail($email);
-
         $users = $this->loadUsers();
 
         foreach ($users as $i => $user) {

php/model/UserDAOInterface.php

@@ -21,7 +21,6 @@ interface UserDAOInterface {
      * @return array|null Benutzerdaten als Array oder null
      *
      * Mögliche Fehler:
-     * @throws InvalidArgumentException wenn die E-Mail-Adresse ungültig ist
      * @throws RuntimeException wenn die Benutzerdaten nicht gelesen werden können
      */
     public function findUser($email);
@@ -43,9 +42,8 @@ interface UserDAOInterface {
      * @return void
      *
      * Mögliche Fehler:
-     * @throws InvalidArgumentException wenn Eingabedaten ungültig sind
+     * @throws InvalidArgumentException wenn die E-Mail-Adresse bereits verwendet wird
-     * oder die E-Mail-Adresse bereits verwendet wird
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder gespeichert werden können
-     * @throws RuntimeException wenn die Benutzerdaten nicht gespeichert werden können
      */
     public function addUser($email, $vorname, $nachname, $password);
 
@@ -67,10 +65,8 @@ interface UserDAOInterface {
      * @return bool true, wenn der Benutzer aktualisiert wurde, sonst false
      *
      * Mögliche Fehler:
-     * @throws InvalidArgumentException wenn Eingabedaten ungültig sind
+     * @throws InvalidArgumentException wenn die neue E-Mail-Adresse bereits verwendet wird
-     * oder die neue E-Mail-Adresse bereits verwendet wird
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder gespeichert werden können
-     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen
-     * oder gespeichert werden können
      */
     public function updateUser($oldEmail, $newEmail, $vorname, $nachname, $password = null);
 
@@ -87,9 +83,7 @@ interface UserDAOInterface {
      * @return bool true, wenn der Benutzer gelöscht wurde, sonst false
      *
      * Mögliche Fehler:
-     * @throws InvalidArgumentException wenn die E-Mail-Adresse ungültig ist
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder gespeichert werden können
-     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen
-     * oder gespeichert werden können
      */
     public function deleteUser($email);
 }

php/validator/user-validator.php

@@ -0,0 +1,33 @@
+<?php
+
+function userNameValidator($name)
+{
+    $name = trim($name);
+    $namePattern = '/^[a-zA-ZäöüÄÖÜß\s-]{2,50}$/u';
+
+    return preg_match($namePattern, $name) === 1;
+}
+
+function userEmailValidator($email)
+{
+    $email = trim($email);
+
+    return filter_var($email, FILTER_VALIDATE_EMAIL) !== false
+        && mb_strlen($email) <= 100;
+}
+
+function userPasswordValidator($password)
+{
+    $zeichenAnzahl = mb_strlen($password);
+
+    return $zeichenAnzahl >= 8 && $zeichenAnzahl <= 72;
+}
+
+function userOptionalPasswordValidator($password)
+{
+    if (!isset($password) || $password === '') {
+        return true;
+    }
+
+    return userPasswordValidator($password);
+}