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

Reviewed-on: #23

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

@@ -17,6 +17,11 @@ 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"]);
     ?>
@@ -34,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

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

content/register.php

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

content/showCategory.php

@@ -0,0 +1,31 @@
+<?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>

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>

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=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>
+                <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>
             </ul>
         </li>
 
         <li>
             <label class="nav__mobile-label">MINT</label>
             <ul class="nav__mobile-submenu">
-                <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>
+                <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>
             </ul>
         </li>
 
         <li>
             <label class="nav__mobile-label">Gesellschaft & Werte</label>
             <ul class="nav__mobile-submenu">
-                <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>
+                <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>
             </ul>
         </li>
 
         <li>
             <label class="nav__mobile-label">Technik & Praxis</label>
             <ul class="nav__mobile-submenu">
-                <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>
+                <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>
             </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=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>
+                <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>
             </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=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>
+                <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>
             </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=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>
+                <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>
             </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=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>
+                <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>
             </div>
         </div>
     </div>

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/' . $_GET["pfad"] . '.php')) {
-            include_once 'content/' . $_GET["pfad"] . '.php';
-        } else {
-            include_once 'content/404.php';
-        }
+    if (file_exists('content/' . $pfad . '.php')) {
+        include_once 'content/' . $pfad . '.php';
     } else {
-        include_once 'content/home.php';
+        include_once 'content/404.php';
     }
 
     include_once 'includes/footer.php';
     ?>
 
-
     </body>
     </html>
 

php/controller/profile-controller.php

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

php/controller/register-controller.php

@@ -1,37 +1,43 @@
 <?php
 
 require_once "php/model/LocalUserDAO.php";
+require_once "php/validator/user-validator.php";
 
-$dao = new LocalUserDAO();
 $error = null;
 
 if ($_SERVER["REQUEST_METHOD"] === "POST") {
 
-    $email = $_POST["email"] ?? "";
-    $vorname = $_POST["vorname"] ?? "";
-    $nachname = $_POST["nachname"] ?? "";
-    $password = password_hash(
-        $_POST["password"] ?? "",
-        PASSWORD_DEFAULT
-    );
-
-    if ($dao->findUser($email)) {
-
-        $error = "Diese E-Mail-Adresse ist bereits registriert.";
+    $email = trim($_POST["email"] ?? "");
+    $vorname = trim($_POST["vorname"] ?? "");
+    $nachname = trim($_POST["nachname"] ?? "");
+    $plainPassword = $_POST["password"] ?? "";
 
+    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(
-            $email,
-            $vorname,
-            $nachname,
-            $password
-        );
+            $password = password_hash($plainPassword, PASSWORD_DEFAULT);
 
-        $_SESSION["user"] = $vorname . " " . $nachname;
-        $_SESSION["user_email"] = $email;
+            $dao->addUser($email, $vorname, $nachname, $password);
 
-        header("Location: index.php");
-        exit();
+            $_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

@@ -0,0 +1,21 @@
+<?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,6 +85,13 @@ 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,5 +201,26 @@ 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

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

php/model/UserDAOInterface.php

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

php/validator/user-validator.php

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

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>