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

Reviewed-on: #23

README.md

@@ -10,12 +10,15 @@ 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.
@@ -23,3 +26,6 @@ Jeweils im Format `Anmeldename, Passwort, Mailadresse`:
 - 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/404.php

@@ -4,6 +4,14 @@
 -->
 <main>
 
+    <?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "internal_error"): ?>
+        <p class="alert-message is-error">
+            Es ist ein interner Fehler beim Speichern aufgetreten. Bitte versuche es erneut.
+        </p>
+    <?php endif; ?>
+    <?php
+    unset($_SESSION["message"]);
+    ?>
     <h1>404 - Seite nicht vorhanden</h1>
     <p>
         Später im Projekt sollen über index.php?pfad= ... der Inhalt der index.php dynamisch gesetzt werden.

content/home.php

@@ -39,21 +39,21 @@ include_once 'php/controller/home-controller.php';
     <!-- Flexbox -->
      <div class="flexbox">
         <div class="container">
-            <a href="index.php?pfad=informatik" class="category-link">Informatik</a>
+            <a href="index.php?pfad=showCategory&category=informatik" class="category-link">Informatik</a>
             <div class="article-link">
                 <a href="index.php?pfad=showArticle&id=<?php echo $dummy3->getId()?>"><?php if(isset($dummy3)){echo $dummy3->getTitle();}else{echo "Fehler: Beitrag nicht gefunden!";} ?></a>
             </div>
         </div>
 
         <div class="container">
-            <a href="index.php?pfad=mathe" class="category-link">Mathe</a>
+            <a href="index.php?pfad=showCategory&category=mathe" class="category-link">Mathe</a>
             <div class="article-link">
                 <a href="index.php?pfad=showArticle&id=<?php echo $dummy1->getId()?>"><?php if(isset($dummy1)){echo $dummy1->getTitle();}else{echo "Fehler: Beitrag nicht gefunden!";} ?></a>
             </div>
         </div>
         
         <div class="container">
-            <a href="index.php?pfad=physik" class="category-link">Physik</a>
+            <a href="index.php?pfad=showCategory&category=physik" class="category-link">Physik</a>
             <div class="article-link">
                 <a href="index.php?pfad=showArticle&id=<?php echo $dummy2->getId()?>"><?php if(isset($dummy2)){echo $dummy2->getTitle();}else{echo "Fehler: Beitrag nicht gefunden!";} ?></a>
             </div>

content/profile.php

@@ -1,36 +1,84 @@
 <?php
 include_once 'php/controller/profile-controller.php';
-$user = $user ?? null;
-?>
 
-<!--
+$user = $user ?? null;
-    Content: Profil
+$isEditMode = (isset($_GET["edit"]) && $_GET["edit"] === "1") || !empty($error);
-    Inhalt: Das eigene Profil, wenn man angemeldet ist. Dort hat man die Möglichkeit seine Angaben zu ändern.
+?>
--->
 
 <main class="form-page">
     <div class="flexbox">
-        <!-- Linke Spalte: Profildaten -->
+
         <div class="container">
-            <form>
+
-                <label class="input-label">Name</label>
+            <?php if (!empty($error)): ?>
+                <p class="alert-message is-error">
+                    <?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($_POST["vorname"] ?? $user["vorname"] ?? ""); ?>"
-                       value="<?php echo htmlspecialchars($user["username"] ?? ""); ?>">
+                        <?php echo $isEditMode ? "" : "readonly"; ?>
+                       required>
+
+                <label class="input-label">Nachname</label>
+                <input type="text"
+                       name="nachname"
+                       class="login-input"
+                       value="<?php echo htmlspecialchars($_POST["nachname"] ?? $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($_POST["email"] ?? $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>
+
+                <?php if ($isEditMode): ?>
                     <input type="password"
+                           name="password"
                            class="login-input"
-                       readonly
+                           placeholder="Leer lassen, wenn es gleich bleiben soll">
-                       value="********">
+                <?php else: ?>
+                    <input type="password"
+                           name="password"
+                           class="login-input"
+                           value="********"
+                           readonly>
+                <?php endif; ?>
+
+                <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="button">
@@ -44,36 +92,53 @@ $user = $user ?? null;
             </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-date">
-                                <span class="article-category"><?php echo htmlspecialchars($userArticle->getCategory()); ?></span>
+                                    <?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 +146,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="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

@@ -1,6 +1,4 @@
 <?php
-include_once 'php/controller/register-controller.php';
-
 $error = $error ?? null;
 ?>
 
@@ -13,8 +11,8 @@ $error = $error ?? null;
 
         <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; ?>
@@ -26,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>
@@ -33,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>
@@ -40,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>

content/showCategory.php

@@ -13,16 +13,15 @@ include_once "php/controller/showCategory-controller.php";
                 <div class="s-res-item">
                     <div class="s-res-content">
                         <h2 class="s-res-item-title">
-                            <a href="index.php?pfad=showArticle&id=<?php echo $article['id']; ?>" class="s-res-link">
+                            <a href="index.php?pfad=showArticle&id=<?php echo $article->getId(); ?>" class="s-res-link">
-                                <?php echo htmlspecialchars($article['title']); ?>
+                                <?php echo htmlspecialchars($article->getTitle()); ?>
                             </a>
                         </h2>
-                        <p class="s-res-author">Von: <span class="s-res-author-name"><?php echo htmlspecialchars($article['author']); ?></span></p>
+                        <p class="s-res-author">Von: <span class="s-res-author-name"><?php echo htmlspecialchars($article->getAuthor()); ?></span></p>
                     </div>
                     <div class="s-res-arrow">&rarr;</div>
                 </div>
             <?php endforeach; ?>
-
         <?php
         else: ?>
         <p> Es sind noch keine Beiträge in dieser Kategorie enthalten.</p>

content/updateArticle.php

@@ -86,7 +86,7 @@ include_once 'php/controller/showArticle-controller.php';
     <aside class="editor-sidebar">
 
         <div class="sidebar-block">
-            <button type="submit" class="button">Änderungen speichern</button>
+            <button type="submit" id="editor-button" class="button">Änderungen speichern</button>
         </div>
 
         <div class="sidebar-block">

css/main.css

@@ -22,6 +22,10 @@ a, button, input, select, textarea, label, main{
     font-family: Arial, Helvetica, sans-serif;
 }
 
+h1 {
+    text-transform: uppercase;
+}
+
 .flexbox {
     display: flex;
     flex-direction: row;

datenschutzVSdatensicherheit.php

@@ -1,56 +0,0 @@
-<!--
-    Artikel: Datenschutz vs Datensicherheit
-    Funktion: Inhalt zum Fach Informatik
-    TODO: entfernen, wenn die Kategorie-Anzeige implementiert ist.
--->
-<!DOCTYPE html>
-<html lang="de">
-
-<head>
-    <meta charset="utf-8">
-    <meta name="description" content="EduForge">
-    <meta name="author" content="Niklas Ortmann">
-    <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <link rel="icon" type="image/x-icon" href="images/logos/logo_icon.ico">
-    <link rel="stylesheet" href="css/main.css">
-    <link rel="stylesheet" href="css/navbar.css">
-    <link rel="stylesheet" href="css/footer.css">
-    <title>EduForge</title>
-</head>
-
-<body>
-<?php
-include_once 'includes/navbar.php';
-?>
-    <main>
-
-        <h1>Datenschutz vs Datensicherheit</h1>
-
-        <p>
-            Datenschutz ist in unserer digital vernetzten Welt allgegenwärtig – ob in sozialen Netzwerken, beim
-            Online-Shopping oder dem Einsatz von Bonuskarten, bei der Auswertung von Konsumverhalten und selbstverständlich
-            auch im schulischen Kontext.
-        </p>
-        <h2>
-            Was ist Datenschutz?
-        </h2>
-        <p>
-            Datenschutz bedeutet, dass persönliche Daten nur gesammelt, gespeichert und verwendet werden dürfen, wenn
-            es wirklich nötig ist – und nur für klar festgelegte Zwecke. Es geht darum, die Privatsphäre von Menschen
-            zu schützen.
-        </p>
-        <h2>
-            Was ist Datenschutz?
-        </h2>
-        <p>
-            Datensicherheit umfasst alle technischen und organisatorischen Maßnahmen, die verhindern sollen, dass Daten
-            verloren gehen, beschädigt oder unbemerkt verändert werden. Sie schützt Informationen vor Missbrauch.
-        </p>
-    </main>
-
-<?php
-include_once 'includes/footer.php';
-?>
-</body>
-
-</html>

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/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

@@ -1,8 +1,11 @@
 <?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;
 
 if (!isset($_SESSION["user"])) {
     header("Location: index.php?pfad=login");
@@ -11,21 +14,78 @@ if (!isset($_SESSION["user"])) {
 
 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) {
+    $user = $dao->findUser($_SESSION["user_email"] ?? "");
+
+    if (!$user) {
         $_SESSION = [];
         session_destroy();
 
         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 (!userEmailValidator($newEmail)) {
+            $error = "Bitte gib eine gültige E-Mail-Adresse ein.";
+            $_GET["edit"] = "1";
+
+        } elseif (!userNameValidator($vorname)) {
+            $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";
+
+        } 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.";
+                $_GET["edit"] = "1";
+            }
+        }
+    }
+
+    $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 (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,40 +1,43 @@
 <?php
 
 require_once "php/model/LocalUserDAO.php";
+require_once "php/validator/user-validator.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"] ?? "";
+    $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 . " " . $nachname,
+            $dao->addUser($email, $vorname, $nachname, $password);
-            $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/controller/showCategory-controller.php

@@ -1,7 +1,7 @@
 <?php
-require_once 'model/LocalArticleManager.php';
+require_once 'php/model/Article.php';
-require_once 'model/ArticleManager.php';
+require_once 'php/model/ArticleManager.php';
-require_once 'validator/article-validator.php';
+require_once 'php/validator/article-validator.php';
 
 if (isset($_GET["category"]) && !empty($_GET["category"]) && articleCategoryValidator($_GET["category"])){
     $category = $_GET["category"];

php/model/ArticleManager.php

@@ -19,7 +19,7 @@ class ArticleManager extends LocalArticleManager
                 "Satz des Pythagoras",
                 "Der Satz des Pythagoras wurde von dem griechischen Philosophen Pythagoras von Samos formuliert und im dritten Jahrhundert vor Christus veröffentlicht. In der beigefügten Abbildung sehen wir ein rechtwinkliges Dreieck...",
                 "max.mustermann",
-                "Mathe",
+                "mathe",
                 "Dreiecke, Dreiecksseiten berechnen"
             );
         }
@@ -28,7 +28,7 @@ class ArticleManager extends LocalArticleManager
                 "Tunneleffekt",
                 "Der Tunneleffekt ist ein quantenmechanisches Phänomen, bei dem Teilchen...",
                 "max.mustermann",
-                "Physik",
+                "physik",
                 "Quantenphysik, Energie"
             );
         }
@@ -37,7 +37,7 @@ class ArticleManager extends LocalArticleManager
                 "Datenschutz vs Datensicherheit",
                 "Datenschutz ist in unserer digital vernetzten Welt allgegenwärtig...",
                 "max.mustermann",
-                "Informatik",
+                "informatik",
                 "Daten, DSGVO"
             );
         }
@@ -45,25 +45,4 @@ class ArticleManager extends LocalArticleManager
         return $articleManager;
     }
  
-    public function getArticlesByCategory($category)
-    {
-        $articles = $this->getAllArticles();
-        $filteredArticles = [];
-
-        foreach ($articles as $article) {
-            if (isset($article['category']) && $article['category'] == $category) {
-                $filteredArticles[] = new Article(
-                    intval($article['id']),
-                    $article['title'],
-                    $article['content'],
-                    $article['author'],
-                    $article['category'],
-                    $article['tags'],
-                    $article['creationDate']
-                );
-            }
-        }
-        return $filteredArticles;
-    }
- 
 }

php/model/LocalUserDAO.php

@@ -1,33 +1,78 @@
 <?php
+
 require_once "UserDAOInterface.php";
 
 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 [];
         }
 
         $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 : [];
     }
 
+    /**
+     * 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) {
-        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.");
+        }
+    }
+
+    /**
+     * Sucht einen Benutzer anhand seiner 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
+     */
     public function findUser($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 +80,93 @@ class LocalUserDAO implements UserDAOInterface {
         return null;
     }
 
-    public function addUser($email, $username, $password) {
+    /**
+     * 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) {
         $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);
     }
 
+    /**
+     * 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) {
+        $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;
+    }
+
+    /**
+     * 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) {
         $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,79 @@ 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 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 die E-Mail-Adresse bereits verwendet wird
+     * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder 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 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 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);
+}

pythagoras.php

@@ -1,53 +0,0 @@
-<!--
-    Artikel: Satz des Pythagoras
-    Funktion: Inhalt zum Fach Mathe
-    TODO: entfernen, wenn die Kategorie-Anzeige implementiert ist.
--->
-<!DOCTYPE html>
-<html lang="de">
-
-<head>
-    <meta charset="utf-8">
-    <meta name="description" content="EduForge">
-    <meta name="author" content="Niklas Ortmann">
-    <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <link rel="icon" type="image/x-icon" href="images/logos/logo_icon.ico">
-    <link rel="stylesheet" href="css/main.css">
-    <link rel="stylesheet" href="css/navbar.css">
-    <link rel="stylesheet" href="css/footer.css">
-    <title>EduForge</title>
-</head>
-
-<body>
-
-<?php
-include_once 'includes/navbar.php';
-?>
-    <main>
-
-        <h1>Satz des Pythagoras</h1>
-
-        <p>
-            Der Satz des Pythagoras wurde von dem griechischen Philosophen Pythagoras von Samos formuliert und im
-            dritten Jahrhundert vor Christus veröffentlicht.
-        </p>
-        <figure>
-          <img src="https://cdn8.picryl.com/photo/2016/05/14/pythagoras-e9560b-1024.jpg" alt="Trulli" style="width:10%">
-          <figcaption><a href="https://cdn8.picryl.com/photo/2016/05/14/pythagoras-e9560b-1024.jpg">Quelle</a></figcaption>
-        </figure>
-        <p>
-            In der beigefügten Abbildung sehen wir ein rechtwinkliges Dreieck, dessen drei Seiten die Längen a, b und c
-            besitzen. Auf jeder Seite ist ein Quadrat konstruiert. Das Quadrat auf der Seite a hat die Fläche a2, das
-            Quadrat auf der Seite b hat die Fläche b2 und das Quadrat auf der Seite c hat die Fläche c2. Der Satz des
-            Pythagoras besagt, dass die Summe der Flächen der beiden kleineren Quadrate gleich der Fläche des größten
-            Quadrats ist. Das bedeutet, dass a^2+b^2=c^2 (Satz des Pythagoras).
-        </p>
-    <a href="docs/Uebungsaufgaben_Pythagoras.pdf">Lernzettel-Download</a>
-    </main>
-
-<?php
-include_once 'includes/footer.php';
-?>
-</body>
-
-</html>

tunneleffekt.php

@@ -1,47 +0,0 @@
-<!--
-    Artikel: Tunneleffekt
-    Funktion: Inhalt zum Fach Physik
-    TODO: entfernen, wenn die Kategorie-Anzeige implementiert ist.
--->
-<!DOCTYPE html>
-<html lang="de">
-
-<head>
-    <meta charset="utf-8">
-    <meta name="description" content="EduForge">
-    <meta name="author" content="Niklas Ortmann">
-    <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <link rel="icon" type="image/x-icon" href="images/logos/logo_icon.ico">
-    <link rel="stylesheet" href="css/main.css">
-    <link rel="stylesheet" href="css/navbar.css">
-    <link rel="stylesheet" href="css/footer.css">
-    <title>EduForge</title>
-</head>
-
-<body>
-
-<?php
-include_once 'includes/navbar.php';
-?>
-    <main>
-
-        <h1>Tunneleffekt</h1>
-
-        <p>
-            Der Tunneleffekt ist ein quantenmechanisches Phänomen, bei dem Teilchen – vor allem Elektronen – eine
-            Energiebarriere überwinden können, obwohl sie nach den Regeln der klassischen Physik nicht genügend Energie
-            dafür besitzen. In der klassischen Vorstellung müsste ein Elektron entweder genügend Energie haben, um über
-            eine Barriere zu „springen“, oder es würde vollständig zurückgeworfen. In der Quantenphysik wird ein Elektron
-            jedoch nicht als punktförmiges Teilchen beschrieben, sondern als Wellenfunktion, die sich räumlich ausdehnt.
-            Dadurch besteht eine endliche Wahrscheinlichkeit, dass sich das Elektron auf der anderen Seite einer Barriere
-            befindet (Griffiths & Schroeter, 2018).
-        </p>
-
-    </main>
-
-<?php
-include_once 'includes/footer.php';
-?>
-</body>
-
-</html>