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

Reviewed-on: #18

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,14 +4,6 @@
 -->
 <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

@@ -17,11 +17,6 @@ include_once 'php/controller/home-controller.php';
         Dein Beitrag wurde erfolgreich veröffentlicht!
     </p>
     <?php endif; ?>
-    <?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "invalid_category"): ?>
-        <p class="alert-message is-error">
-            Diese Kategorie ist nicht gültig.
-        </p>
-    <?php endif; ?>
     <?php
     unset($_SESSION["message"]);
     ?>
@@ -39,21 +34,21 @@ include_once 'php/controller/home-controller.php';
     <!-- Flexbox -->
      <div class="flexbox">
         <div class="container">
-            <a href="index.php?pfad=showCategory&category=informatik" class="category-link">Informatik</a>
+            <a href="index.php?pfad=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=showCategory&category=mathe" class="category-link">Mathe</a>
+            <a href="index.php?pfad=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=showCategory&category=physik" class="category-link">Physik</a>
+            <a href="index.php?pfad=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,76 @@
 <?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="button">
@@ -44,36 +84,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-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="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">
 

content/showCategory.php

@@ -1,31 +0,0 @@
-<?php
-include_once "php/controller/showCategory-controller.php";
-?>
-
-<main>
-
-    <h1><?php if (isset($category) && !empty($category)){ echo htmlspecialchars($category); } ?></h1>
-
-    <div class="s-res-list">
-        <?php
-        if (!empty($articles)): ?>
-            <?php foreach ($articles as $article): ?>
-                <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->getId(); ?>" class="s-res-link">
-                                <?php echo htmlspecialchars($article->getTitle()); ?>
-                            </a>
-                        </h2>
-                        <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>
-        <?php endif; ?>
-    </div>
-
-</main>

css/main.css

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

includes/navbar.php

@@ -39,50 +39,50 @@ Globales Menü, wird via PHP später in alle Seiten eingebunden
         <li>
             <label class="nav__mobile-label">Sprachen</label>
             <ul class="nav__mobile-submenu">
-                <li><a href="index.php?pfad=showCategory&category=deutsch">Deutsch</a></li>
-                <li><a href="index.php?pfad=showCategory&category=englisch">Englisch</a></li>
-                <li><a href="index.php?pfad=showCategory&category=franzoesisch">Französisch</a></li>
-                <li><a href="index.php?pfad=showCategory&category=latein">Latein</a></li>
-                <li><a href="index.php?pfad=showCategory&category=literatur">Literatur</a></li>
+                <li><a href="index.php?pfad=deutsch">Deutsch</a></li>
+                <li><a href="index.php?pfad=englisch">Englisch</a></li>
+                <li><a href="index.php?pfad=franzoesisch">Französisch</a></li>
+                <li><a href="index.php?pfad=latein">Latein</a></li>
+                <li><a href="index.php?pfad=literatur">Literatur</a></li>
             </ul>
         </li>
 
         <li>
             <label class="nav__mobile-label">MINT</label>
             <ul class="nav__mobile-submenu">
-                <li><a href="index.php?pfad=showCategory&category=mathe">Mathematik</a></li>
-                <li><a href="index.php?pfad=showCategory&category=biologie">Biologie</a></li>
-                <li><a href="index.php?pfad=showCategory&category=chemie">Chemie</a></li>
-                <li><a href="index.php?pfad=showCategory&category=physik">Physik</a></li>
-                <li><a href="index.php?pfad=showCategory&category=informatik">Informatik</a></li>
-                <li><a href="index.php?pfad=showCategory&category=astronomie">Astronomie</a></li>
+                <li><a href="index.php?pfad=mathe">Mathematik</a></li>
+                <li><a href="index.php?pfad=biologie">Biologie</a></li>
+                <li><a href="index.php?pfad=chemie">Chemie</a></li>
+                <li><a href="index.php?pfad=physik">Physik</a></li>
+                <li><a href="index.php?pfad=informatik">Informatik</a></li>
+                <li><a href="index.php?pfad=astronomie">Astronomie</a></li>
             </ul>
         </li>
 
         <li>
             <label class="nav__mobile-label">Gesellschaft & Werte</label>
             <ul class="nav__mobile-submenu">
-                <li><a href="index.php?pfad=showCategory&category=geschichte">Geschichte</a></li>
-                <li><a href="index.php?pfad=showCategory&category=erdkunde">Erdkunde</a></li>
-                <li><a href="index.php?pfad=showCategory&category=sozialkunde">Sozialkunde</a></li>
-                <li><a href="index.php?pfad=showCategory&category=wirtschaft">Wirtschaftskunde</a></li>
-                <li><a href="index.php?pfad=showCategory&category=religion">Religion</a></li>
-                <li><a href="index.php?pfad=showCategory&category=ethik">Ethikunterricht</a></li>
-                <li><a href="index.php?pfad=showCategory&category=philosophie">Philosophie</a></li>
-                <li><a href="index.php?pfad=showCategory&category=psychologie">Psychologie</a></li>
-                <li><a href="index.php?pfad=showCategory&category=kunst">Kunst</a></li>
-                <li><a href="index.php?pfad=showCategory&category=musik">Musik</a></li>
-                <li><a href="index.php?pfad=showCategory&category=theater">Theater</a></li>
+                <li><a href="index.php?pfad=geschichte">Geschichte</a></li>
+                <li><a href="index.php?pfad=erdkunde">Erdkunde</a></li>
+                <li><a href="index.php?pfad=sozialkunde">Sozialkunde</a></li>
+                <li><a href="index.php?pfad=wirtschaft">Wirtschaftskunde</a></li>
+                <li><a href="index.php?pfad=religion">Religion</a></li>
+                <li><a href="index.php?pfad=ethik">Ethikunterricht</a></li>
+                <li><a href="index.php?pfad=philosophie">Philosophie</a></li>
+                <li><a href="index.php?pfad=psychologie">Psychologie</a></li>
+                <li><a href="index.php?pfad=kunst">Kunst</a></li>
+                <li><a href="index.php?pfad=musik">Musik</a></li>
+                <li><a href="index.php?pfad=theater">Theater</a></li>
             </ul>
         </li>
 
         <li>
             <label class="nav__mobile-label">Technik & Praxis</label>
             <ul class="nav__mobile-submenu">
-                <li><a href="index.php?pfad=showCategory&category=technik">Technik</a></li>
-                <li><a href="index.php?pfad=showCategory&category=werken">Werken</a></li>
-                <li><a href="index.php?pfad=showCategory&category=hauswirtschaft">Hauswirtschaft</a></li>
-                <li><a href="index.php?pfad=showCategory&category=sport">Sport</a></li>
+                <li><a href="index.php?pfad=technik">Technik</a></li>
+                <li><a href="index.php?pfad=werken">Werken</a></li>
+                <li><a href="index.php?pfad=hauswirtschaft">Hauswirtschaft</a></li>
+                <li><a href="index.php?pfad=sport">Sport</a></li>
             </ul>
         </li>
     </ul>
@@ -92,50 +92,50 @@ Globales Menü, wird via PHP später in alle Seiten eingebunden
         <div class="nav__item nav__dropdown">
             <button class="nav__dropdown-toggle">Sprachen</button>
             <div class="nav__dropdown-menu">
-                <a href="index.php?pfad=showCategory&category=deutsch">Deutsch</a>
-                <a href="index.php?pfad=showCategory&category=englisch">Englisch</a>
-                <a href="index.php?pfad=showCategory&category=franzoesisch">Französisch</a>
-                <a href="index.php?pfad=showCategory&category=latein">Latein</a>
-                <a href="index.php?pfad=showCategory&category=literatur">Literatur</a>
+                <a href="index.php?pfad=deutsch">Deutsch</a>
+                <a href="index.php?pfad=englisch">Englisch</a>
+                <a href="index.php?pfad=franzoesisch">Französisch</a>
+                <a href="index.php?pfad=latein">Latein</a>
+                <a href="index.php?pfad=literatur">Literatur</a>
             </div>
         </div>
 
         <div class="nav__item nav__dropdown">
             <button class="nav__dropdown-toggle">MINT</button>
             <div class="nav__dropdown-menu">
-                <a href="index.php?pfad=showCategory&category=mathe">Mathematik</a>
-                <a href="index.php?pfad=showCategory&category=biologie">Biologie</a>
-                <a href="index.php?pfad=showCategory&category=chemie">Chemie</a>
-                <a href="index.php?pfad=showCategory&category=physik">Physik</a>
-                <a href="index.php?pfad=showCategory&category=informatik">Informatik</a>
-                <a href="index.php?pfad=showCategory&category=astronomie">Astronomie</a>
+                <a href="index.php?pfad=mathe">Mathematik</a>
+                <a href="index.php?pfad=biologie">Biologie</a>
+                <a href="index.php?pfad=chemie">Chemie</a>
+                <a href="index.php?pfad=physik">Physik</a>
+                <a href="index.php?pfad=informatik">Informatik</a>
+                <a href="index.php?pfad=astronomie">Astronomie</a>
             </div>
         </div>
 
         <div class="nav__item nav__dropdown">
             <button class="nav__dropdown-toggle">Gesellschaft & Werte</button>
             <div class="nav__dropdown-menu">
-                <a href="index.php?pfad=showCategory&category=geschichte">Geschichte</a>
-                <a href="index.php?pfad=showCategory&category=erdkunde">Erdkunde</a>
-                <a href="index.php?pfad=showCategory&category=sozialkunde">Sozialkunde</a>
-                <a href="index.php?pfad=showCategory&category=wirtschaft">Wirtschaftskunde</a>
-                <a href="index.php?pfad=showCategory&category=religion">Religion</a>
-                <a href="index.php?pfad=showCategory&category=ethik">Ethikunterricht</a>
-                <a href="index.php?pfad=showCategory&category=philosophie">Philosophie</a>
-                <a href="index.php?pfad=showCategory&category=psychologie">Psychologie</a>
-                <a href="index.php?pfad=showCategory&category=kunst">Kunst</a>
-                <a href="index.php?pfad=showCategory&category=musik">Musik</a>
-                <a href="index.php?pfad=showCategory&category=theater">Theater</a>
+                <a href="index.php?pfad=geschichte">Geschichte</a>
+                <a href="index.php?pfad=erdkunde">Erdkunde</a>
+                <a href="index.php?pfad=sozialkunde">Sozialkunde</a>
+                <a href="index.php?pfad=wirtschaft">Wirtschaftskunde</a>
+                <a href="index.php?pfad=religion">Religion</a>
+                <a href="index.php?pfad=ethik">Ethikunterricht</a>
+                <a href="index.php?pfad=philosophie">Philosophie</a>
+                <a href="index.php?pfad=psychologie">Psychologie</a>
+                <a href="index.php?pfad=kunst">Kunst</a>
+                <a href="index.php?pfad=musik">Musik</a>
+                <a href="index.php?pfad=theater">Theater</a>
             </div>
         </div>
 
         <div class="nav__item nav__dropdown">
             <button class="nav__dropdown-toggle">Technik & Praxis</button>
             <div class="nav__dropdown-menu">
-                <a href="index.php?pfad=showCategory&category=technik">Technik</a>
-                <a href="index.php?pfad=showCategory&category=werken">Werken</a>
-                <a href="index.php?pfad=showCategory&category=hauswirtschaft">Hauswirtschaft</a>
-                <a href="index.php?pfad=showCategory&category=sport">Sport</a>
+                <a href="index.php?pfad=technik">Technik</a>
+                <a href="index.php?pfad=werken">Werken</a>
+                <a href="index.php?pfad=hauswirtschaft">Hauswirtschaft</a>
+                <a href="index.php?pfad=sport">Sport</a>
             </div>
         </div>
     </div>

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");
@@ -27,4 +32,4 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
     } else {
         $error = "Login fehlgeschlagen. Bitte überprüfe deine Eingaben.";
     }
-}
+}

php/controller/profile-controller.php

@@ -4,28 +4,106 @@ 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();
 }
 
-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();
+/*
+ * 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);
 }
 
-if (!$user) {
-    $_SESSION = [];
-    session_destroy();
+/*
+ * 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;
+}
 
-    header("Location: index.php?pfad=login");
-    exit();
+/*
+ * 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"] ?? "");
+
+    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 (!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
         );
 
@@ -37,4 +34,4 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
         header("Location: index.php");
         exit();
     }
-}
+}

php/controller/showCategory-controller.php

@@ -1,21 +0,0 @@
-<?php
-require_once 'php/model/Article.php';
-require_once 'php/model/ArticleManager.php';
-require_once 'php/validator/article-validator.php';
-
-if (isset($_GET["category"]) && !empty($_GET["category"]) && articleCategoryValidator($_GET["category"])){
-    $category = $_GET["category"];
-    try {
-        $articleManager = ArticleManager::getInstance();
-        $articles = $articleManager->getArticlesByCategory($category);
-    } catch (Exception $e) {
-        $_SESSION["message"] = "internal_error";
-        include_once "content/404.php";
-        exit();
-    }
-}else{
-    $_SESSION["message"] = "invalid_category";
-    include_once "content/404.php";
-    exit();
-}
-?>

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"
             );
         }

php/model/ArticleManagerDAO.php

@@ -85,13 +85,6 @@ interface ArticleManagerDAO
      */
     public function search(string $keyword): array;
 
-    /**
-     * Gibt alle Beiträge einer gegebenen Kategorie aus.
-     * @param $category
-     * @return mixed
-     */
-    public function getArticlesByCategory($category);
-
 
 }
 ?>

php/model/LocalArticleManager.php

@@ -201,26 +201,5 @@ class LocalArticleManager implements ArticleManagerDAO {
     return $filteredArticles;
     }
 
-    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,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);
 }