Update createArticle.php

.idea/dataSources.local.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project version="4">
-  <component name="dataSourceStorageLocal" created-in="IU-253.32098.101">
+  <component name="dataSourceStorageLocal" created-in="IU-261.25134.95">
     <data-source name="articles" uuid="315cb5c9-2b0f-435b-b602-59823b160908">
       <database-info product="SQLite" version="3.51.1" jdbc-version="4.2" driver-name="SQLite JDBC" driver-version="3.51.1.0" dbms="SQLITE" exact-version="3.51.1" exact-driver-version="3.51">
         <identifier-quote-string>&quot;</identifier-quote-string>

content/createArticle.php

@@ -27,6 +27,8 @@ $blocks = array_values($blocks); // sequentielle Indizes 0..n-1 sicherstellen
 -->
 <form method="post" action="php/controller/createArticle-controller.php" id="editor-form" enctype="multipart/form-data" class="article-editor-scope.editor-container article-editor-scope editor-container">
 
+    <?php csrf_field(); ?>
+
     <main class="editor-main">
         <?php include_once "includes/alertMessages.php"?>
 

content/profile.php

@@ -18,6 +18,8 @@ $isEditMode = (isset($_GET["edit"]) && $_GET["edit"] === "1") || !empty($error);
 
             <form method="post" action="index.php?pfad=profile">
 
+                <?php csrf_field(); ?>
+
                 <label class="input-label">Vorname</label>
                 <input type="text"
                        name="vorname"
@@ -81,12 +83,11 @@ $isEditMode = (isset($_GET["edit"]) && $_GET["edit"] === "1") || !empty($error);
 
             <br>
 
-            <form action="php/controller/deleteAccount-controller.php"
+            <form action="php/controller/deleteAccount-controller.php" method="POST">
-                  method="POST"
+                <?php csrf_field(); ?>
-                  class="confirm-delete-account">
-
                 <button type="submit"
-                        class="button">
+                        class="button"
+                        onclick="return confirm('Möchtest du deinen Account und alle deine Beiträge wirklich unwiderruflich löschen?');">
                     Account löschen
                 </button>
             </form>
@@ -156,16 +157,15 @@ $isEditMode = (isset($_GET["edit"]) && $_GET["edit"] === "1") || !empty($error);
                                 Bearbeiten
                             </a>
 
-                            <form action="php/controller/deleteArticle-controller.php"
+                            <form action="php/controller/deleteArticle-controller.php" method="POST">
-                                  method="POST"
+                                <?php csrf_field(); ?>
-                                  class="confirm-delete-article">
-
                                 <input type="hidden"
                                        name="id"
                                        value="<?php echo htmlspecialchars($userArticle->getID()); ?>">
 
                                 <button type="submit"
-                                        class="button">
+                                        class="button"
+                                        onclick="return confirm('Möchtest du diesen Artikel wirklich löschen?');">
                                     Löschen
                                 </button>
                             </form>
@@ -176,9 +176,11 @@ $isEditMode = (isset($_GET["edit"]) && $_GET["edit"] === "1") || !empty($error);
 
                     <p>Du hast noch keine Beiträge erstellt.</p>
 
-                    <a href="index.php?pfad=createArticle" class="button">
+                    <button type="button"
+                            class="button"
+                            onclick="window.location.href='index.php?pfad=createArticle';">
                         Beitrag erstellen!
-                    </a>
+                    </button>
 
                 <?php endif; ?>
             </div>

content/search-results.php

@@ -124,9 +124,7 @@ $categories = [
             </div>
 
             <noscript>
-                <button type="submit" class="nav__search-button">
+                <button type="submit" class="nav__search-button">Filter anwenden</button>
-                    Filter anwenden
-                </button>
             </noscript>
         </form>
 
@@ -195,34 +193,17 @@ $categories = [
             </div>
 
             <nav class="s-res-page-navigation" aria-label="Seitennavigation">
-
+                <!-- No-JS-Fallback: -->
                 <noscript>
-                    <?php echo renderNoJsPagination(
+                    <?php echo renderNoJsPagination($currentPage, $totalPages, $query, $currentSort, $currentCategory, $limit); ?>
-                            $currentPage,
-                            $totalPages,
-                            $query,
-                            $currentSort,
-                            $currentCategory,
-                            $limit
-                    ); ?>
                 </noscript>
 
-                <div id="js-page-navigation" style="display: none;">
+                <!-- JS-Version: wird per search-results.js befüllt/eingeblendet: -->
-                    <button type="button"
+                <div id="js-page-navigation" style="display:none;">
-                            class="s-res-page-btn"
+                    <button type="button" class="s-res-page-btn" id="prev-page-btn">«</button>
-                            id="prev-page-btn">
-                        «
-                    </button>
-
                     <span id="dynamic-page-numbers"></span>
-
+                    <button type="button" class="s-res-page-btn" id="next-page-btn">»</button>
-                    <button type="button"
-                            class="s-res-page-btn"
-                            id="next-page-btn">
-                        »
-                    </button>
                 </div>
-
             </nav>
 
         </div>

content/showArticle.php

@@ -45,26 +45,31 @@ if ($replyAuthor === null) {
     <!-- Metadaten & Titel -->
     <div class="article-view-top-section">
 
-            <div class="category-and-likes-row">
+        <div class="category-and-likes-row">
-                <?php if (isset($category) && !empty($category)): ?>
+            <?php if (isset($category) && !empty($category)): ?>
-                    <span class="article-view-category"><?php echo htmlspecialchars($category); ?></span>
+                <span class="article-view-category"><?php echo htmlspecialchars($category); ?></span>
-                <?php endif; ?>
+            <?php endif; ?>
 
-                <!-- Like-Anzeige und dynamischer Like-Button  -->
+            <!-- Like-Anzeige und dynamischer Like-Button  -->
-                <?php if (isset($articleObj) && $articleObj !== null): ?>
+            <?php if (isset($articleObj) && $articleObj !== null): ?>
-                    <div class="article-view-likes">
+                <div class="article-view-likes">
-                        <span>❤️ <span class="like-count"><?php echo $articleObj->getLikeCount(); ?></span></span>
+                    <span>❤️ <span class="like-count"><?php echo $articleObj->getLikeCount(); ?></span></span>
 
-                        <?php if (isset($_SESSION["user_email"])): ?>
+                    <?php if (isset($_SESSION["user_email"])): ?>
-                            <a href="php/controller/like-controller.php?id=<?php echo $articleObj->getId(); ?>" class="like-toggle-btn">
+                        <form method="post"
+                              action="php/controller/like-controller.php?id=<?php echo $articleObj->getId(); ?>"
+                              class="like-toggle-form">
+                            <?php csrf_field(); ?>
+                            <button type="submit" class="like-toggle-btn">
                                 <?php echo $articleObj->hasLiked($_SESSION["user_email"]) ? '👎 Gefällt mir nicht mehr' : '👍 Gefällt mir'; ?>
-                            </a>
+                            </button>
-                        <?php else: ?>
+                        </form>
-                            <span class="login-hint">(Anmelden zum Liken)</span>
+                    <?php else: ?>
-                        <?php endif; ?>
+                        <span class="login-hint">(Anmelden zum Liken)</span>
-                    </div>
+                    <?php endif; ?>
-                <?php endif; ?>
+                </div>
-            </div>
+            <?php endif; ?>
+        </div>
 
         <h1 class="article-view-title">
             <?php if (isset($title)) { echo htmlspecialchars($title); } ?>
@@ -202,6 +207,8 @@ if ($replyAuthor === null) {
                                           action="index.php?pfad=updateComment"
                                           class="edit-comment-form">
 
+                                        <?php csrf_field(); ?>
+
                                         <input type="hidden"
                                                name="comment_id"
                                                value="<?php echo htmlspecialchars(
@@ -237,7 +244,9 @@ if ($replyAuthor === null) {
 
                                 <form method="post"
                                       action="index.php?pfad=deleteComment"
-                                      class="delete-comment-form confirm-delete-comment">
+                                      class="delete-comment-form">
+
+                                    <?php csrf_field(); ?>
 
                                     <input type="hidden"
                                            name="comment_id"
@@ -252,7 +261,8 @@ if ($replyAuthor === null) {
                                            ); ?>">
 
                                     <button type="submit"
-                                            class="delete-comment-button">
+                                            class="delete-comment-button"
+                                            onclick="return confirm('Möchtest du diesen Kommentar wirklich löschen?');">
                                         Kommentar löschen
                                     </button>
                                 </form>
@@ -333,6 +343,8 @@ if ($replyAuthor === null) {
                                                           action="index.php?pfad=updateComment"
                                                           class="edit-comment-form">
 
+                                                        <?php csrf_field(); ?>
+
                                                         <input type="hidden"
                                                                name="comment_id"
                                                                value="<?php echo htmlspecialchars(
@@ -368,7 +380,9 @@ if ($replyAuthor === null) {
 
                                                 <form method="post"
                                                       action="index.php?pfad=deleteComment"
-                                                      class="delete-comment-form confirm-delete-comment">
+                                                      class="delete-comment-form">
+
+                                                    <?php csrf_field(); ?>
 
                                                     <input type="hidden"
                                                            name="comment_id"
@@ -383,8 +397,9 @@ if ($replyAuthor === null) {
                                                            ); ?>">
 
                                                     <button type="submit"
-                                                            class="delete-comment-button">
+                                                            class="delete-comment-button"
-                                                        Antwort löschen
+                                                            onclick="return confirm('Möchtest du diesen Kommentar wirklich löschen?');">
+                                                        Kommentar löschen
                                                     </button>
                                                 </form>
 
@@ -410,6 +425,8 @@ if ($replyAuthor === null) {
                   method="post"
                   action="php/ajax/add-comment.php">
 
+                <?php csrf_field(); ?>
+
                 <input type="hidden"
                        name="article_id"
                        value="<?php echo htmlspecialchars(
@@ -463,4 +480,4 @@ if ($replyAuthor === null) {
             </div>
         <?php endif; ?>
     </section>
-</main>
+</main>

content/updateArticle.php

@@ -27,6 +27,8 @@ $blocks = array_values($blocks);
 -->
 <form method="post" action="php/controller/updateArticle-controller.php?id=<?php if(isset($id) && !empty($id)){echo htmlspecialchars($id);}else{$_SESSION["message"] = "missing_id";} ?>" id="editor-form" enctype="multipart/form-data" class="article-editor-scope.editor-container article-editor-scope editor-container">
 
+    <?php csrf_field(); ?>
+
     <main class="editor-main">
         <?php include_once "includes/alertMessages.php"?>
 

css/navbar.css

@@ -250,7 +250,7 @@ CSS für die navbar
         padding: 2rem 1rem;
         box-shadow: 2px 0 10px rgba(0,0,0,0.5);
         overflow-y: auto;
-        
+
         /* Genug Abstand oben rechts, damit Links nicht hinter dem X liegen */
         padding: 4rem 1.5rem 2rem 1.5rem;
     }
@@ -285,6 +285,32 @@ CSS für die navbar
         border-bottom: 1px solid #333d43;
     }
 
+    .nav__logout-form {
+        display: contents;
+        margin: 0;
+    }
+
+    .nav__logout-form .nav__button {
+        width: 100%;
+        height: 100%;
+    }
+
+    .nav__mobile-logout-button {
+        color: #fff;
+        text-decoration: none;
+        font-size: 1.2rem;
+        font-weight: 600;
+        display: block;
+        width: 100%;
+        text-align: left;
+        padding: 0.5rem 1rem;
+        border: none;
+        border-bottom: 1px solid #333d43;
+        background: none;
+        cursor: pointer;
+        font-family: inherit;
+    }
+
     .nav__mobile-submenu {
         display: block;
         list-style: none;

includes/alertMessages.php

@@ -89,6 +89,12 @@
         Es ist ein Datenbankfehler aufgetreten. Bitte versuche es erneut.
     </p>
 <?php endif; ?>
+<?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "invalid_csrf_token"): ?>
+    <p class="alert-message is-error">
+        Deine Sitzung ist abgelaufen oder die Anfrage konnte nicht überprüft werden.
+        Bitte lade die Seite neu und versuche es erneut.
+    </p>
+<?php endif; ?>
 <?php
 unset($_SESSION["message"]);
-?>
+?>

includes/csrf.php

@@ -0,0 +1,88 @@
+<?php
+/**
+ * CSRF-Schutz nach dem Synchronizer-Token-Pattern.
+ *
+ * Pro Session wird ein einziges, zufälliges Token erzeugt,
+ * das in jedem Formular als verstecktes Feld mitgeschickt und bei jeder
+ * zustandsändernden Anfrage serverseitig mit dem Session-Token verglichen
+ * wird.
+ *
+ * @author Niklas Ortmann
+ */
+
+if (session_status() === PHP_SESSION_NONE) {
+    session_start();
+}
+
+/**
+ * Gibt das aktuelle CSRF-Token der Session zurück.
+ *
+ * Existiert noch kein Token, wird eines erzeugt und in der Session
+ * gespeichert.
+ *
+ * @return string Das CSRF-Token
+ */
+function csrf_token(): string
+{
+    if (empty($_SESSION["csrf_token"]) || !is_string($_SESSION["csrf_token"])) {
+        $_SESSION["csrf_token"] = bin2hex(random_bytes(32));
+    }
+
+    return $_SESSION["csrf_token"];
+}
+
+/**
+ * Gibt ein verstecktes Formularfeld mit dem aktuellen CSRF-Token aus.
+ *
+ * Wird in jedem Formular benötigt, das eine zustandsändernde
+ * Aktion auslöst.
+ *
+ * @return void
+ */
+function csrf_field(): void
+{
+    echo '<input type="hidden" name="csrf_token" value="'
+        . htmlspecialchars(csrf_token())
+        . '">';
+}
+
+/**
+ * Prüft, ob das per POST gesendete CSRF-Token zum Session-Token passt.
+ *
+ * Der Vergleich erfolgt zeitkonstant über hash_equals(), um
+ * Timing-Angriffe auf den Vergleich selbst auszuschließen.
+ *
+ * @return bool true, wenn das Token gültig ist
+ */
+function csrf_verify(): bool
+{
+    $sentToken = $_POST["csrf_token"] ?? "";
+    $sessionToken = $_SESSION["csrf_token"] ?? "";
+
+    if (!is_string($sentToken) || $sentToken === "" || $sessionToken === "") {
+        return false;
+    }
+
+    return hash_equals($sessionToken, $sentToken);
+}
+
+/**
+ * Bricht die Anfrage ab und leitet mit einer Fehlermeldung um,
+ * wenn das mitgesendete CSRF-Token ungültig oder nicht vorhanden ist.
+ *
+ * Muss am Anfang jeder zustandsändernden POST-Aktion aufgerufen werden,
+ * bevor irgendeine Änderung an Daten vorgenommen wird.
+ *
+ * @param string $redirectTo Ziel-URL, zu der bei ungültigem Token
+ *                           weitergeleitet wird
+ * @return void
+ */
+function csrf_require_valid(string $redirectTo = "index.php"): void
+{
+    if (!csrf_verify()) {
+        http_response_code(403);
+        $_SESSION["message"] = "invalid_csrf_token";
+        header("Location: " . $redirectTo);
+        exit();
+    }
+}

includes/navbar.php

@@ -179,4 +179,4 @@ Globales Menü, wird via PHP später in alle Seiten eingebunden
         <?php endif; ?>
 
     </div>
-</nav>
+</nav>

index.php

@@ -2,6 +2,7 @@
 if (session_status() === PHP_SESSION_NONE) {
     session_start();
 }
+include_once "includes/csrf.php";
 include_once "php/controller/index-controller.php";
 ?>
 <!DOCTYPE html>
@@ -24,10 +25,9 @@ include_once "php/controller/index-controller.php";
     <link rel="stylesheet" href="css/showCategory.css">
 
         <script src="js/comments.js" defer></script>
-        <script src="js/profile.js" defer></script>
         <script src="js/editor.js" async></script>
+        <script src="js/search-results.js" async></script>
         <script src="js/showCategory.js" async></script>
-        <script src="js/search.js" async></script>
 
 
     <title>EduForge</title>
@@ -36,6 +36,7 @@ include_once "php/controller/index-controller.php";
 
 <?php
 include_once 'includes/navbar.php';
+include_once "includes/alertMessages.php";
 
 // Dynamischer Inhalt
 if (isset($pfad) && $pfad !== "404" && file_exists('content/' . $pfad . '.php')) {

js/comments.js

@@ -11,26 +11,6 @@ document.addEventListener("DOMContentLoaded", function () {
     const parentCommentInput = document.getElementById("parent-comment-id");
     const replyInfo = document.getElementById("reply-info");
 
-    /**
-     * Fragt vor dem Löschen eines Kommentars nach einer Bestätigung.
-     *
-     * Der Listener funktioniert auch für Kommentare,
-     * die später per AJAX eingefügt werden.
-     */
-    document.addEventListener("submit", function (event) {
-        if (!event.target.classList.contains("delete-comment-form")) {
-            return;
-        }
-
-        const confirmed = confirm(
-            "Möchtest du diesen Kommentar wirklich löschen?"
-        );
-
-        if (!confirmed) {
-            event.preventDefault();
-        }
-    });
-
     if (!form || !commentsList || !commentContent || !parentCommentInput) {
         return;
     }
@@ -129,6 +109,12 @@ document.addEventListener("DOMContentLoaded", function () {
         action="index.php?pfad=updateComment"
         class="edit-comment-form"
     >
+        <input
+            type="hidden"
+            name="csrf_token"
+            value="${escapeHtml(formData.get("csrf_token"))}"
+        >
+
         <input
             type="hidden"
             name="comment_id"
@@ -157,6 +143,12 @@ document.addEventListener("DOMContentLoaded", function () {
             action="index.php?pfad=deleteComment"
             class="delete-comment-form"
         >
+            <input
+                type="hidden"
+                name="csrf_token"
+                value="${escapeHtml(formData.get("csrf_token"))}"
+            >
+
             <input
                 type="hidden"
                 name="comment_id"
@@ -172,6 +164,7 @@ document.addEventListener("DOMContentLoaded", function () {
             <button
                 type="submit"
                 class="delete-comment-button"
+                onclick="return confirm('Möchtest du diesen Kommentar wirklich löschen?');"
             >
                 Kommentar löschen
             </button>
@@ -206,6 +199,12 @@ document.addEventListener("DOMContentLoaded", function () {
         action="index.php?pfad=updateComment"
         class="edit-comment-form"
     >
+        <input
+            type="hidden"
+            name="csrf_token"
+            value="${escapeHtml(formData.get("csrf_token"))}"
+        >
+
         <input
             type="hidden"
             name="comment_id"
@@ -234,6 +233,12 @@ document.addEventListener("DOMContentLoaded", function () {
             action="index.php?pfad=deleteComment"
             class="delete-comment-form"
         >
+            <input
+                type="hidden"
+                name="csrf_token"
+                value="${escapeHtml(formData.get("csrf_token"))}"
+            >
+
             <input
                 type="hidden"
                 name="comment_id"
@@ -249,6 +254,7 @@ document.addEventListener("DOMContentLoaded", function () {
             <button
                 type="submit"
                 class="delete-comment-button"
+                onclick="return confirm('Möchtest du diesen Kommentar wirklich löschen?');"
             >
                 Kommentar löschen
             </button>

js/profile.js

@@ -1,29 +0,0 @@
-document.addEventListener("DOMContentLoaded", function () {
-
-    const deleteAccountForm = document.querySelector(".confirm-delete-account");
-
-    if (deleteAccountForm) {
-        deleteAccountForm.addEventListener("submit", function (event) {
-
-            if (!confirm("Möchtest du deinen Account und alle deine Beiträge wirklich unwiderruflich löschen?")) {
-                event.preventDefault();
-            }
-
-        });
-    }
-
-    const deleteArticleForms = document.querySelectorAll(".confirm-delete-article");
-
-    deleteArticleForms.forEach(function (form) {
-
-        form.addEventListener("submit", function (event) {
-
-            if (!confirm("Möchtest du diesen Artikel wirklich löschen?")) {
-                event.preventDefault();
-            }
-
-        });
-
-    });
-
-});

php/controller/logout-controller.php

@@ -1,6 +1,10 @@
 <?php
+if (session_status() === PHP_SESSION_NONE) {
+    session_start();
+}
+
 $_SESSION = [];
 session_destroy();
 
-header("Location: index.php");
+header("Location: ../../index.php");
 exit();

php/model/DatabaseArticleManager.php

@@ -310,20 +310,21 @@ class DatabaseArticleManager implements ArticleManagerDAO {
 
         try {
             $db = $this->getConnection();
-            
+
             $sql = "SELECT id, title, content, author, category, tags, created 
-                FROM articles 
+                    FROM articles 
-                WHERE title LIKE :keyword 
+                    WHERE title LIKE :keyword 
-                OR content LIKE :keyword
+                       OR content LIKE :keyword
-                OR tags LIKE :keyword;";
+                       OR tags LIKE :keyword";
 
             $command = $db->prepare($sql);
             if (!$command) {
                 throw new InternalServerErrorException("internal_error");
             }
 
-            // Wildcards für die SQL-Suche hinzufügen
+            // Wildcards für die Suche hinzufügen
             $searchParam = '%' . $cleankeyword . '%';
+
             $success = $command->execute([
                 ":keyword" => $searchParam
             ]);
@@ -336,11 +337,10 @@ class DatabaseArticleManager implements ArticleManagerDAO {
             $filteredArticles = [];
 
             foreach ($rows as $row) {
-                $articleId = intval($row['id']);
+                $likes = $this->getLikesForArticle(intval($row['id']));
-                $likes = $this->getLikesForArticle($articleId);
 
                 $filteredArticles[] = new Article(
-                    $articleId,
+                    intval($row['id']),
                     $row['title'] ?? '',
                     $row['content'] ?? '',
                     $row['author'] ?? '',
@@ -358,7 +358,6 @@ class DatabaseArticleManager implements ArticleManagerDAO {
         }
     }
 
-
     /**
      * Holt alle User-IDs, die einen bestimmten Beitrag geliked haben.
      *