Merge pull request 'Vor und Nachname separat speichern sowie Profilbearbeitung' (#18) from Profilbearbeitung into dev

Reviewed-on: #18

README.md

@@ -1,13 +1,31 @@
-Team Tag-Buchstabe: DI-14-C-Schulte-Eilers-Ortmann
-Namen der Studierenden:
-- Name 1 Caroline Schulte
-- Name 2 Jacques Eilers
-- Name 3 Niklas Ortmann
-Abgabe zu Aufgabenblatt 1
+# Webprogrammierung: Gruppe DI-14-C
 
-Hinweise:
-index.html wurde zu einer index.php umbenannt.
-webhook.php bitte ignorieren. Diese dient nur als zukünftige Automatisierung des Repo-Pulls (s. https://docs.gitea.com/usage/webhooks)
+## Beteiligte
+- Caroline Schulte
+- Jacques Eilers
+- Niklas Ortmann
 
-Beim Verwenden der Navbar-Links bitte Folgendes beachten: nur die Kategorien Physik, Mathe, Informatik führen exemplarisch auf eine Beispiel-Seite.
-Die anderen Links sind erstmal Platzhalter, dessen dynamischer Inhalt später mit PHP in die index.php eingefügt wird. Alle anderen Links führen auf eine 404-Seite.
+## Login-Informationen für Dummy-User
+Jeweils im Format `Anmeldename, Passwort, Mailadresse`:
+- `max.mustermann, test123, mustermann@web.de`
+
+## Weitere Voraussetzungen zur Nutzung
+- Per Klick auf das Logo gelangt man auf die Home-Seite.
+
+## Ausgelassene Teilaufgaben
+
+## Bekannte Fehler und Mängel
+- Der Beitragseditor fügt beim Laden von Inhalten leere Zeichen ein.
+- Bitte auf die gesetzten TODO's achten. Wenn Inhalte fehlen, sind sie i.d.R. als TODO kommentiert.
+- Die Navbar hat noch keine onHover-Effekte und erschwert etwas die Bedienung.
+- Suchleiste in der Nav ist in der mobilen Ansicht etwas schmal. Ggf. ein alternatives (kleineres) Logo implementieren.
+
+## Besonderheiten des Projektes
+- Es wurde ein einfacher Beitrags-Editor erstellt. Mit diesem können Beiträge erstellt oder bearbeitet werden.
+  Es handelt es sich um eine einfache Version. Später sollen z.B. Bilder und die Positionierung der Elemente folgen.
+- Es sind drei Dummy-Beiträge für den Nutzer max.mustermann hinterlegt.
+- Beim Verwenden der Navbar-Links bitte Folgendes beachten: nur die Kategorien Physik, Mathe, Informatik führen exemplarisch auf eine Beispiel-Seite.
+  Die anderen Links sind erstmal Platzhalter und führen auf eine 404-Seite.
+- Die Such-Seite umfasst eine Suchfunktion (aber noch nicht nach Tags) und Sortierfunktion. Jedoch fehlt noch eine Filterfunktion (z.B. nur Mathe anzeigen). 
+  Wenn die Filterfunktion implementiert ist, dann können die statischen Seiten "Informatik", "Mathe", "Physik" entfernt und dynamisch auf der Suchseite 
+  nach gefiltertem Fach angezeigt werden. (Vorläufige showCategory.php implementiert)

content/createArticle.php

@@ -62,7 +62,7 @@ if (!isset($_SESSION["user"])) {
     <aside class="editor-sidebar">
 
         <div class="sidebar-block">
-            <button type="submit" class="btn-publish">Veröffentlichen</button>
+            <button type="submit" id="editor-button" class="button">Veröffentlichen</button>
         </div>
 
         <div class="sidebar-block">

content/login.php

@@ -37,7 +37,7 @@ $error = $error ?? null;
             <button type="submit"
                     value="anmelden"
                     name="loginSubmit"
-                    class="login-button">
+                    class="button">
                 anmelden
             </button>
 

content/profile.php

@@ -1,79 +1,136 @@
 <?php
 include_once 'php/controller/profile-controller.php';
-$user = $user ?? null;
-?>
 
-<!--
-    Content: Profil
-    Inhalt: Das eigene Profil, wenn man angemeldet ist. Dort hat man die Möglichkeit seine Angaben zu ändern.
--->
+$user = $user ?? null;
+$isEditMode = isset($_GET["edit"]) && $_GET["edit"] === "1";
+?>
 
 <main class="form-page">
     <div class="flexbox">
-        <!-- Linke Spalte: Profildaten -->
+
         <div class="container">
-            <form>
-                <label class="input-label">Name</label>
+
+            <?php if (isset($error) && $error): ?>
+                <p style="color:red;">
+                    <?php echo htmlspecialchars($error); ?>
+                </p>
+            <?php endif; ?>
+
+            <form method="post" action="index.php?pfad=profile">
+
+                <label class="input-label">Vorname</label>
                 <input type="text"
+                       name="vorname"
                        class="login-input"
-                       readonly
-                       value="<?php echo htmlspecialchars($user["username"] ?? ""); ?>">
+                       value="<?php echo htmlspecialchars($user["vorname"] ?? ""); ?>"
+                        <?php echo $isEditMode ? "" : "readonly"; ?>
+                       required>
+
+                <label class="input-label">Nachname</label>
+                <input type="text"
+                       name="nachname"
+                       class="login-input"
+                       value="<?php echo htmlspecialchars($user["nachname"] ?? ""); ?>"
+                        <?php echo $isEditMode ? "" : "readonly"; ?>
+                       required>
 
                 <label class="input-label">Email-Adresse</label>
                 <input type="email"
+                       name="email"
                        class="login-input"
-                       readonly
-                       value="<?php echo htmlspecialchars($user["email"] ?? ""); ?>">
+                       value="<?php echo htmlspecialchars($user["email"] ?? ""); ?>"
+                        <?php echo $isEditMode ? "" : "readonly"; ?>
+                       required>
 
-                <label class="input-label">Passwort</label>
+                <label class="input-label">
+                    <?php echo $isEditMode ? "Neues Passwort" : "Passwort"; ?>
+                </label>
                 <input type="password"
+                       name="password"
                        class="login-input"
-                       readonly
-                       value="********">
+                       placeholder="<?php echo $isEditMode ? "Leer lassen, wenn es gleich bleiben soll" : "********"; ?>"
+                        <?php echo $isEditMode ? "" : "readonly"; ?>>
+
+                <br><br>
+
+                <?php if ($isEditMode): ?>
+                    <button type="submit"
+                            name="saveProfile"
+                            class="button">
+                        Speichern
+                    </button>
+
+                    <a href="index.php?pfad=profile" class="button">
+                        Abbrechen
+                    </a>
+                <?php else: ?>
+                    <a href="index.php?pfad=profile&edit=1" class="button">
+                        Bearbeiten
+                    </a>
+                <?php endif; ?>
+
             </form>
+
             <br>
 
-            <a href="index.php?pfad=deleteAccount" class="login-button">
+            <a href="index.php?pfad=deleteAccount" class="button">
                 Account löschen
             </a>
 
             <br><br>
 
-            <a href="index.php?pfad=logout" class="login-button">
+            <a href="index.php?pfad=logout" class="button">
                 Abmelden
             </a>
         </div>
 
-        <!-- Rechte Spalte: Eigene Beiträge -->
         <div class="container">
             <h2 class="section-title">Meine Beiträge</h2>
+
             <div class="articles-list">
                 <?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "internal_error"): ?>
+
                     <p class="alert-message is-error">
                         Es ist ein interner Fehler aufgetreten. Bitte versuche es erneut.
                     </p>
+
                 <?php elseif (isset($userArticles) && count($userArticles) > 0): ?>
+
                     <?php foreach ($userArticles as $userArticle): ?>
-                        <!-- Ein einzelner Artikel-Eintrag -->
                         <div class="article-item">
                             <div class="article-meta">
-                                <span class="article-date"><?php echo htmlspecialchars($userArticle->getCreationDate()); ?></span>
-                                <span class="article-category"><?php echo htmlspecialchars($userArticle->getCategory()); ?></span>
+                                <span class="article-date">
+                                    <?php echo htmlspecialchars($userArticle->getCreationDate()); ?>
+                                </span>
+
+                                <span class="article-category">
+                                    <?php echo htmlspecialchars($userArticle->getCategory()); ?>
+                                </span>
                             </div>
-                            <h3 class="article-title"><?php echo htmlspecialchars($userArticle->getTitle()); ?></h3>
+
+                            <h3 class="article-title">
+                                <?php echo htmlspecialchars($userArticle->getTitle()); ?>
+                            </h3>
+
                             <?php
                             $tags = $userArticle->getTags();
-                            if (isset($tags) && !empty($tags)): ?>
+                            if (isset($tags) && !empty($tags)):
+                                ?>
                                 <div class="article-view-bottom-section">
                                     <div class="article-view-tags-label">Tags:</div>
+
                                     <div class="article-view-tags-list">
                                         <?php
                                         $tagArray = is_array($tags) ? $tags : explode(',', $tags);
+
                                         foreach ($tagArray as $tag):
                                             $trimmedTag = trim($tag);
+
                                             if (!empty($trimmedTag)):
                                                 ?>
-                                                <span class="article-view-tag-item"><?php echo htmlspecialchars($trimmedTag); ?></span>
+                                                <span class="article-view-tag-item">
+                                                    <?php echo htmlspecialchars($trimmedTag); ?>
+                                                </span>
                                             <?php
                                             endif;
                                         endforeach;
@@ -81,18 +138,27 @@ $user = $user ?? null;
                                     </div>
                                 </div>
                             <?php endif; ?>
-                            <a href="index.php?pfad=updateArticle&id=<?php echo $userArticle->getID(); ?>" class="edit-link-button">Bearbeiten</a>
+
+                            <a href="index.php?pfad=updateArticle&id=<?php echo $userArticle->getID(); ?>"
+                               class="edit-link-button">
+                                Bearbeiten
+                            </a>
                         </div>
                     <?php endforeach; ?>
+
                 <?php else: ?>
+
                     <p>Du hast noch keine Beiträge erstellt.</p>
-                    <button type="button" class="login-button" onclick="window.location.href='index.php?pfad=createArticle';">
+
+                    <button type="button"
+                            class="button"
+                            onclick="window.location.href='index.php?pfad=createArticle';">
                         Beitrag erstellen!
                     </button>
+
                 <?php endif; ?>
-                <?php
-                unset($_SESSION["message"]);
-                ?>
+
+                <?php unset($_SESSION["message"]); ?>
             </div>
         </div>
 

content/register.php

@@ -4,10 +4,6 @@ include_once 'php/controller/register-controller.php';
 $error = $error ?? null;
 ?>
 
-<!--
-    Form: Registrierung
-    Funktion: Erstellung neuer Benutzerkonten
--->
 <main class="login-page">
     <div class="login-container">
 
@@ -52,7 +48,7 @@ $error = $error ?? null;
             <button type="submit"
                     value="register"
                     name="registerSubmit"
-                    class="login-button">
+                    class="button">
                 kostenlos registrieren
             </button>
 

content/updateArticle.php

@@ -53,6 +53,11 @@ include_once 'php/controller/showArticle-controller.php';
                 Bei der Validierung deiner Daten ist ein Fehler aufgetreten. Bitte versuche es erneut.
             </p>
         <?php endif; ?>
+        <?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "unauthorized_access"): ?>
+            <p class="alert-message is-error">
+                Sie sind nicht berechtigt, diesen Beitrag zu bearbeiten.
+            </p>
+        <?php endif; ?>
         <?php
         unset($_SESSION["message"]);
         ?>
@@ -81,7 +86,7 @@ include_once 'php/controller/showArticle-controller.php';
     <aside class="editor-sidebar">
 
         <div class="sidebar-block">
-            <button type="submit" class="btn-publish">Änderungen speichern</button>
+            <button type="submit" class="button">Änderungen speichern</button>
         </div>
 
         <div class="sidebar-block">

css/createArticle.css

@@ -86,20 +86,8 @@
 }
 
 /* Button-Zeugs */
-.article-editor-scope .btn-publish {
-    background-color: #007cba;
-    color: white;
-    border: none;
-    padding: 12px;
-    border-radius: 4px;
-    font-weight: 600;
-    cursor: pointer;
-    font-size: 1rem;
-    width: 100%;
-}
-
-.article-editor-scope .btn-publish:hover {
-    background-color: #006ba1;
+.button#editor-button {
+    padding: 18px 12px;
 }
 
 /* Responsive Anpassungen unter 760px (für z.B. Smartphones) */

css/main.css

@@ -155,7 +155,7 @@ a, button, input, select, textarea, label, main{
     color: #1f2937;
 }
 
-.login-button {
+.button {
     width: 100%;
     padding: 12px;
     background-color: #2563eb;
@@ -168,7 +168,7 @@ a, button, input, select, textarea, label, main{
     transition: background-color 0.2s, transform 0.2s, box-shadow 0.2s;
 }
 
-.login-button:hover {
+.button:hover {
     background-color: #1d4ed8;
     transform: translateY(-2px);
     box-shadow: 0 4px 10px rgba(0,0,0,0.15);

datenschutzVSdatensicherheit.php

@@ -1,6 +1,7 @@
 <!--
     Artikel: Datenschutz vs Datensicherheit
     Funktion: Inhalt zum Fach Informatik
+    TODO: entfernen, wenn die Kategorie-Anzeige implementiert ist.
 -->
 <!DOCTYPE html>
 <html lang="de">

path.php

@@ -1,4 +0,0 @@
-<?php
-// mit $_SERVER['DOCUMENT_ROOT'] später umbauen?
-$abs_path = __DIR__;
-?>

php/controller/login-controller.php

@@ -18,7 +18,12 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
 
     if ($user && password_verify($password, $user["password"])) {
 
-        $_SESSION["user"] = $user["username"];
+        if (isset($user["vorname"]) && isset($user["nachname"])) {
+            $_SESSION["user"] = $user["vorname"] . " " . $user["nachname"];
+        } else {
+            $_SESSION["user"] = $user["username"] ?? "";
+        }
+
         $_SESSION["user_email"] = $user["email"];
 
         header("Location: index.php");

php/controller/profile-controller.php

@@ -4,23 +4,45 @@ require_once "php/model/LocalUserDAO.php";
 require_once 'php/model/Article.php';
 require_once 'php/model/ArticleManager.php';
 
+$error = null;
+
 if (!isset($_SESSION["user"])) {
     header("Location: index.php?pfad=login");
     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();
+
     $user = $dao->findUser($_SESSION["user_email"] ?? "");
-    $articleManager = ArticleManager::getInstance();
-    $userArticles = $articleManager->getArticlesByAuthor($_SESSION["user_email"]);
-    if(!isset($userArticles)) {
-        $_SESSION["message"] = "user_has_no_articles";
-    }
-} catch (Exception $e) {
-    $_SESSION["message"] = "internal_error";
-    exit();
-}
 
     if (!$user) {
         $_SESSION = [];
@@ -29,3 +51,59 @@ if (!$user) {
         header("Location: index.php?pfad=login");
         exit();
     }
+
+    if ($_SERVER["REQUEST_METHOD"] === "POST" && isset($_POST["saveProfile"])) {
+        $oldEmail = $_SESSION["user_email"];
+
+        $newEmail = trim($_POST["email"] ?? "");
+        $vorname = trim($_POST["vorname"] ?? "");
+        $nachname = trim($_POST["nachname"] ?? "");
+        $password = $_POST["password"] ?? "";
+
+        if (!isValidName($vorname)) {
+            $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)) {
+            $error = "Das Passwort muss mindestens 8 Zeichen lang sein.";
+        } else {
+            $existingUser = $dao->findUser($newEmail);
+
+            if ($existingUser && $newEmail !== $oldEmail) {
+                $error = "Diese E-Mail-Adresse wird bereits verwendet.";
+            } else {
+                $updated = $dao->updateUser(
+                    $oldEmail,
+                    $newEmail,
+                    $vorname,
+                    $nachname,
+                    $password
+                );
+
+                if ($updated) {
+                    $_SESSION["user"] = $vorname . " " . $nachname;
+                    $_SESSION["user_email"] = $newEmail;
+
+                    header("Location: index.php?pfad=profile");
+                    exit();
+                } else {
+                    $error = "Die Daten konnten nicht gespeichert werden.";
+                }
+            }
+        }
+    }
+
+    $user = $dao->findUser($_SESSION["user_email"] ?? "");
+
+    $articleManager = ArticleManager::getInstance();
+    $userArticles = $articleManager->getArticlesByAuthor($_SESSION["user_email"]);
+
+    if (!isset($userArticles)) {
+        $_SESSION["message"] = "user_has_no_articles";
+    }
+
+} catch (Exception $e) {
+    $_SESSION["message"] = "internal_error";
+}

php/controller/register-controller.php

@@ -5,10 +5,6 @@ require_once "php/model/LocalUserDAO.php";
 $dao = new LocalUserDAO();
 $error = null;
 
-/*
-    Verarbeitung des Registrierungs-Formulars
-    Funktion: Erstellt neuen Benutzer und speichert ihn im DAO + Session
-*/
 if ($_SERVER["REQUEST_METHOD"] === "POST") {
 
     $email = $_POST["email"] ?? "";
@@ -27,7 +23,8 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
 
         $dao->addUser(
             $email,
-            $vorname . " " . $nachname,
+            $vorname,
+            $nachname,
             $password
         );
 

php/controller/updateArticle-controller.php

@@ -1,5 +1,5 @@
 <?php
-session_start();
+SESSION_START();
 require_once '../model/LocalArticleManager.php';
 require_once '../model/ArticleManager.php';
 require_once '../model/Article.php';
@@ -8,7 +8,7 @@ require_once '../validator/article-validator.php';
 if ($_SERVER["REQUEST_METHOD"] === "POST") {
     $_SESSION["old_title"]    = $_POST["title"] ?? '';
     $_SESSION["old_content"]  = $_POST["content"] ?? '';
-    $_SESSION["old_category"] = $_POST["category"] ?? '';
+    $_SESSION["old_category"] = $_POST["category"] ?? ''; // TODO: die Kategorie im Dropdown setzen, wenn der Editor erneut geöffnet wird.
     $_SESSION["old_tags"]     = $_POST["tags"] ?? '';
 
     if (isset($_GET["id"]) && !empty($_GET["id"])) {
@@ -26,7 +26,7 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
     }else{
         $title = $_POST["title"];
         $content = $_POST["content"];
-        $author = "max.mustermann"; // TODO: später aus Session den angemeldeten Nutzer beziehen.
+        $author = $_SESSION["user_email"];
         $category = $_POST["category"];
         $tags = $_POST['tags'] ?? '';
 
@@ -82,7 +82,7 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
             $article->setTags($cleanedTags);
             $articleManager->updateArticle($id ,$article, $author);
         } catch (Exception $e){
-            $_SESSION["message"] = "internal_error";
+            $_SESSION["message"] = $e->getMessage();
             header("location: ../../index.php?pfad=updateArticle");
             exit();
         }

php/model/ArticleManagerDAO.php

@@ -1,5 +1,10 @@
 <?php
 require_once "Article.php";
+
+class NotFoundException extends Exception {}
+class UnauthorizedAccessException extends Exception {}
+class InternalServerErrorException extends Exception {}
+
 /**
  * Die Klasse beinhaltet alle Methoden für die Operation mit den Artikel-Daten.
  *
@@ -16,7 +21,7 @@ interface ArticleManagerDAO
      * @param $tags string optionale Schlagworte für eine bessere Suche
      *
      * Mögliche Exceptions:
-     * TODO Fehlerbeschreibung hinzufügen & tags hinzufügen
+     * TODO: Exceptions implementieren.
      */
     public function addArticle($title, $content, $author, $category, $tags);
 
@@ -29,7 +34,10 @@ interface ArticleManagerDAO
      * @param $author
      * @return void
      *
-     * TODO: Fehlerbeschreibung hinzufügen
+     * @throws InternalServerErrorException
+     * @throws NotFoundException
+     * @throws UnauthorizedAccessException
+     * /
      */
     public function updateArticle($id, $article, $author);
 
@@ -38,7 +46,7 @@ interface ArticleManagerDAO
      * @param $id
      * @return void
      *
-     * TODO: Fehlerbeschreibung hinzufügen
+     * TODO: Exceptions implementieren.
      */
     public function deleteArticle($id);
 
@@ -48,7 +56,7 @@ interface ArticleManagerDAO
      *
      * @return Article
      * Mögliche Exceptions:
-     * TODO Fehlerbeschreibung hinzufügen
+     * TODO: Exceptions implementieren.
      */
     public function getArticle($id);
 
@@ -56,7 +64,7 @@ interface ArticleManagerDAO
      * Alle Beiträge aufrufen.
      *
      * Mögliche Exceptions:
-     * TODO Fehlerbeschreibung hinzufügen
+     * TODO: Exceptions implementieren.
      */
     public function getAllArticles();
 
@@ -64,6 +72,7 @@ interface ArticleManagerDAO
      * Gibt alle Beiträge eines Nutzer mit einer gegebenen ID aus.
      * @param $author
      * @return Article[]
+     * TODO: Exceptions implementieren.
      */
     public function getArticlesByAuthor($author);
 

php/model/LocalArticleManager.php

@@ -36,6 +36,7 @@ class LocalArticleManager implements ArticleManagerDAO {
      * Speichert alle Artikel/Beiträge in der Datei.
      * @param $articles
      * @return void
+     * TODO: Exceptions implementieren.
      */
     public function saveArticle($articles)
     {
@@ -65,14 +66,12 @@ class LocalArticleManager implements ArticleManagerDAO {
     public function updateArticle($id, $article, $author)
     {
         if (empty($article)) {
-            // TODO: Implement Exception.
-            return;
+            throw new InternalServerErrorException("internal_error");
         }
 
         // Berechtigungsprüfung:
         if ($article->getAuthor() !== $author) {
-            // TODO: Implement Exception.
-            return;
+            throw new UnauthorizedAccessException("unauthorized_access");
         }
 
         // Beitrag aktualisieren:
@@ -92,15 +91,14 @@ class LocalArticleManager implements ArticleManagerDAO {
                 ];
                 $updated = true;
                 break;
-            }else{
-                // TODO: Implement Exception.
-                return;
             }
         }
 
         // Nur speichern, wenn Beitrag geändert wurde:
         if ($updated) {
             $this->saveArticle($articles);
+        } else {
+            throw new NotFoundException("missing_id");
         }
     }
 

php/model/LocalUserDAO.php

@@ -1,4 +1,5 @@
 <?php
+
 require_once "UserDAOInterface.php";
 
 class LocalUserDAO implements UserDAOInterface {
@@ -11,23 +12,65 @@ class LocalUserDAO implements UserDAOInterface {
         }
 
         $json = file_get_contents($this->file);
+
+        if ($json === false) {
+            throw new RuntimeException("Benutzerdaten konnten nicht gelesen werden.");
+        }
+
         $users = json_decode($json, true);
 
+        if ($users === null && json_last_error() !== JSON_ERROR_NONE) {
+            throw new RuntimeException("Benutzerdaten sind fehlerhaft.");
+        }
+
         return is_array($users) ? $users : [];
     }
 
     private function saveUsers($users) {
-        file_put_contents(
+        $result = file_put_contents(
             $this->file,
             json_encode($users, JSON_PRETTY_PRINT)
         );
+
+        if ($result === false) {
+            throw new RuntimeException("Benutzerdaten konnten nicht gespeichert werden.");
+        }
+    }
+
+    private function validateEmail($email) {
+        if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
+            throw new InvalidArgumentException("Ungültige E-Mail-Adresse.");
+        }
+    }
+
+    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) {
-            if ($user["email"] === $email) {
+            if (isset($user["email"]) && $user["email"] === $email) {
+
+                if (!isset($user["vorname"]) && isset($user["username"])) {
+                    $nameParts = explode(" ", $user["username"], 2);
+                    $user["vorname"] = $nameParts[0] ?? "";
+                    $user["nachname"] = $nameParts[1] ?? "";
+                }
+
                 return $user;
             }
         }
@@ -35,23 +78,75 @@ class LocalUserDAO implements UserDAOInterface {
         return null;
     }
 
-    public function addUser($email, $username, $password) {
+    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) {
+            if (isset($user["email"]) && $user["email"] === $email) {
+                throw new InvalidArgumentException("Diese E-Mail-Adresse wird bereits verwendet.");
+            }
+        }
+
         $users[] = [
             "email" => $email,
-            "username" => $username,
+            "vorname" => $vorname,
+            "nachname" => $nachname,
             "password" => $password
         ];
 
         $this->saveUsers($users);
     }
 
+    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) {
+            if (
+                isset($user["email"])
+                && $user["email"] === $newEmail
+                && $newEmail !== $oldEmail
+            ) {
+                throw new InvalidArgumentException("Diese E-Mail-Adresse wird bereits verwendet.");
+            }
+        }
+
+        foreach ($users as $i => $user) {
+            if (isset($user["email"]) && $user["email"] === $oldEmail) {
+                $users[$i]["email"] = $newEmail;
+                $users[$i]["vorname"] = $vorname;
+                $users[$i]["nachname"] = $nachname;
+
+                unset($users[$i]["username"]);
+
+                if (!empty($password)) {
+                    $users[$i]["password"] = password_hash($password, PASSWORD_DEFAULT);
+                }
+
+                $this->saveUsers($users);
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     public function deleteUser($email) {
+        $this->validateEmail($email);
+
         $users = $this->loadUsers();
 
         foreach ($users as $i => $user) {
-            if ($user["email"] === $email) {
+            if (isset($user["email"]) && $user["email"] === $email) {
                 unset($users[$i]);
                 $users = array_values($users);
                 $this->saveUsers($users);

php/model/UserDAOInterface.php

@@ -11,26 +11,85 @@ interface UserDAOInterface {
     /**
      * Sucht einen Benutzer anhand seiner E-Mail-Adresse.
      *
+     * Funktion:
+     * Liefert die gespeicherten Benutzerdaten zu einer E-Mail-Adresse.
+     *
+     * Eingabe:
      * @param string $email E-Mail-Adresse des gesuchten Benutzers
+     *
+     * Ausgabe:
      * @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);
 
     /**
      * Fügt einen neuen Benutzer hinzu.
      *
+     * Funktion:
+     * Erstellt einen neuen Benutzereintrag und speichert ihn
+     * in der jeweiligen Datenquelle.
+     *
+     * Eingabe:
      * @param string $email E-Mail-Adresse des Benutzers
-     * @param string $username Benutzername des Benutzers
+     * @param string $vorname Vorname des Benutzers
+     * @param string $nachname Nachname des Benutzers
      * @param string $password Passwort des Benutzers
+     *
+     * Ausgabe:
      * @return void
+     *
+     * Mögliche Fehler:
+     * @throws InvalidArgumentException wenn Eingabedaten ungültig sind
+     * oder die E-Mail-Adresse bereits verwendet wird
+     * @throws RuntimeException wenn die Benutzerdaten nicht gespeichert werden können
      */
-    public function addUser($email, $username, $password);
+    public function addUser($email, $vorname, $nachname, $password);
+
+    /**
+     * Aktualisiert einen bestehenden Benutzer.
+     *
+     * Funktion:
+     * Ändert die E-Mail-Adresse, den Vornamen, den Nachnamen
+     * und optional das Passwort eines bestehenden Benutzers.
+     *
+     * Eingabe:
+     * @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
+     *
+     * Ausgabe:
+     * @return bool true, wenn der Benutzer aktualisiert wurde, sonst false
+     *
+     * Mögliche Fehler:
+     * @throws InvalidArgumentException wenn Eingabedaten ungültig sind
+     * oder 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);
 
     /**
      * Löscht einen Benutzer anhand seiner E-Mail-Adresse.
      *
+     * Funktion:
+     * Entfernt einen vorhandenen Benutzer aus der Datenquelle.
+     *
+     * Eingabe:
      * @param string $email E-Mail-Adresse des zu löschenden Benutzers
+     *
+     * Ausgabe:
      * @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
      */
     public function deleteUser($email);
 }

pythagoras.php

@@ -1,6 +1,7 @@
 <!--
     Artikel: Satz des Pythagoras
     Funktion: Inhalt zum Fach Mathe
+    TODO: entfernen, wenn die Kategorie-Anzeige implementiert ist.
 -->
 <!DOCTYPE html>
 <html lang="de">

tunneleffekt.php

@@ -1,6 +1,7 @@
 <!--
     Artikel: Tunneleffekt
     Funktion: Inhalt zum Fach Physik
+    TODO: entfernen, wenn die Kategorie-Anzeige implementiert ist.
 -->
 <!DOCTYPE html>
 <html lang="de">