Merge pull request 'Opt-In & Optimierung für Datenschutz und Nutzungsbedingungen' (#67) from devNachCSRF into dev

Reviewed-on: #67

.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/profile.php

@@ -1,5 +1,4 @@
 <?php
-include_once 'php/controller/profile-controller.php';
 
 $user = $user ?? null;
 $isEditMode = (isset($_GET["edit"]) && $_GET["edit"] === "1") || !empty($error);

content/search-results.php

@@ -137,6 +137,17 @@ $categories = [
             <p class="s-res-meta"><span id="s-res-result-count"><?php echo $totalResultsCount; ?></span> Treffer für Ihre Suchanfrage "<?php echo htmlspecialchars($query); ?>"</p>
         </div>
 
+        <p
+                id="results-loading"
+                class="results-loading"
+                role="status"
+                aria-live="polite"
+                hidden
+        >
+            <span class="results-spinner" aria-hidden="true"></span>
+            Suchergebnisse werden geladen …
+        </p>
+
         <!-- Ergebnisliste -->
         <div class="s-res-list">
             <?php if (!empty($results)): ?>

content/showArticle.php

@@ -510,6 +510,18 @@ if ($replyAuthor === null) {
                 <button type="submit" class="button">
                     Kommentar senden
                 </button>
+
+                <p
+                        id="comment-loading"
+                        class="comment-loading"
+                        role="status"
+                        aria-live="polite"
+                        hidden
+                >
+                    <span class="comment-spinner" aria-hidden="true"></span>
+                    Kommentar wird gesendet …
+                </p>
+
             </form>
         <?php else: ?>
             <div class="comment-login-hint">

content/showCategory.php

@@ -113,6 +113,17 @@ $resultCount = count($results);
             </p>
         </div>
 
+        <p
+                id="results-loading"
+                class="results-loading"
+                role="status"
+                aria-live="polite"
+                hidden
+        >
+            <span class="results-spinner" aria-hidden="true"></span>
+            Kategoriebeiträge werden geladen …
+        </p>
+
         <!-- Ergebnisliste -->
         <div class="s-res-list">
             <?php if (!empty($results)): ?>

css/main.css

@@ -415,3 +415,33 @@ h1 {
 .edit-comment-details[open] .edit-comment-button::before {
     content: "▲ ";
 }
+.comment-loading {
+    display: flex;
+    align-items: center;
+    gap: 8px;
+    margin-top: 10px;
+}
+
+.comment-loading[hidden] {
+    display: none;
+}
+
+.comment-spinner {
+    width: 16px;
+    height: 16px;
+    border: 2px solid #cbd5e1;
+    border-top-color: #1f2937;
+    border-radius: 50%;
+    animation: comment-spinner-rotation 0.8s linear infinite;
+}
+
+@keyframes comment-spinner-rotation {
+    to {
+        transform: rotate(360deg);
+    }
+}
+
+#comment-form button[type="submit"]:disabled {
+    cursor: wait;
+    opacity: 0.65;
+}

css/search-results.css

@@ -271,4 +271,33 @@ CSS für die Suchergebnis-Seite
         align-items: flex-start;
     }
 }
+/* Ladeanzeige beim Nachladen von Such- und Kategorieergebnissen */
+.results-loading {
+    display: flex;
+    align-items: center;
+    gap: 8px;
+    margin: 15px 0;
+    color: #4a5568;
+    font-size: 0.95rem;
+}
+
+.results-loading[hidden] {
+    display: none;
+}
+
+.results-spinner {
+    width: 16px;
+    height: 16px;
+    flex-shrink: 0;
+    border: 2px solid #cbd5e1;
+    border-top-color: #3182ce;
+    border-radius: 50%;
+    animation: results-spinner-rotation 0.8s linear infinite;
+}
+
+@keyframes results-spinner-rotation {
+    to {
+        transform: rotate(360deg);
+    }
+}
 

includes/navbar.php

@@ -16,7 +16,7 @@ Globales Menü, wird via PHP später in alle Seiten eingebunden
 
     <!-- Mobile navbar mit Burger-Symbol -->
     <input type="checkbox" id="nav-toggle" class="nav__checkbox">
-    <label for="nav-toggle" id="mobile-nav">
+    <label for="nav-toggle" class="mobile-nav">
         <span>☰</span>
     </label>
 
@@ -151,7 +151,7 @@ Globales Menü, wird via PHP später in alle Seiten eingebunden
             </a>
 
             <div class="nav__search">
-                <?php include_once 'search.php'; ?>
+                <?php include_once __DIR__ . '/search.php'; ?>
             </div>
 
             <a href="index.php?pfad=createArticle" class="nav__item nav__button">
@@ -165,7 +165,7 @@ Globales Menü, wird via PHP später in alle Seiten eingebunden
         <?php else: ?>
 
             <div class="nav__search">
-                <?php include_once 'search.php'; ?>
+                <?php include_once __DIR__ . '/search.php'; ?>
             </div>
 
             <a href="index.php?pfad=login" class="nav__item nav__button">

index.php

@@ -2,9 +2,11 @@
 if (session_status() === PHP_SESSION_NONE) {
     session_start();
 }
+
 include_once "includes/csrf.php";
 include_once "php/controller/index-controller.php";
 ?>
+
 <!DOCTYPE html>
 <html lang="de">
 <head>

js/comments.js

@@ -11,7 +11,19 @@ document.addEventListener("DOMContentLoaded", function () {
     const parentCommentInput = document.getElementById("parent-comment-id");
     const replyInfo = document.getElementById("reply-info");
 
-    if (!form || !commentsList || !commentContent || !parentCommentInput) {
+    const submitButton = form
+        ? form.querySelector('button[type="submit"]')
+        : null;
+
+    const loadingMessage = document.getElementById("comment-loading");
+
+    if (
+        !form
+        || !commentsList
+        || !commentContent
+        || !parentCommentInput
+        || !submitButton
+    ) {
         return;
     }
 
@@ -53,6 +65,23 @@ document.addEventListener("DOMContentLoaded", function () {
     form.addEventListener("submit", function (event) {
         event.preventDefault();
 
+        /*
+         * Verhindert einen erneuten Submit, während der vorherige
+         * Kommentar noch gespeichert wird.
+         */
+        if (submitButton.disabled) {
+            return;
+        }
+
+        const originalButtonText = submitButton.textContent;
+
+        submitButton.disabled = true;
+        submitButton.textContent = "Wird gesendet …";
+
+        if (loadingMessage) {
+            loadingMessage.hidden = false;
+        }
+
         const formData = new FormData(form);
         const parentCommentId = parentCommentInput.value;
 
@@ -293,6 +322,14 @@ document.addEventListener("DOMContentLoaded", function () {
             .catch(function (error) {
                 console.error(error);
                 alert("Kommentar konnte nicht gesendet werden.");
+            })
+            .finally(function () {
+                submitButton.disabled = false;
+                submitButton.textContent = originalButtonText;
+
+                if (loadingMessage) {
+                    loadingMessage.hidden = true;
+                }
             });
     });
 

js/search-results.js

@@ -25,6 +25,7 @@
 
     let listContainer, sortRadios, categorySelect, limitSelect;
     let prevBtn, nextBtn, numbersContainer, jsNav, resultCountEl;
+    let loadingElement;
 
     function init() {
         listContainer = document.querySelector('.s-res-list');
@@ -38,23 +39,41 @@
         numbersContainer = document.getElementById('dynamic-page-numbers');
         jsNav = document.getElementById('js-page-navigation');
         resultCountEl = document.getElementById('s-res-result-count');
+        loadingElement = document.getElementById('results-loading');
 
         const checkedRadio = document.querySelector('.sort-radio:checked');
         state.sort = checkedRadio ? checkedRadio.value : 'alphabet';
         state.category = categorySelect ? categorySelect.value : 'all';
         state.itemsPerPage = limitSelect ? (parseInt(limitSelect.value, 10) || 10) : 10;
 
+        if (loadingElement) {
+            loadingElement.hidden = false;
+        }
+
         fetchFullDataset()
             .then(function (data) {
                 state.allItems = data.results || [];
                 attachEvents();
                 render();
-                if (jsNav) jsNav.style.display = 'flex';
+
+                if (jsNav) {
+                    jsNav.style.display = 'flex';
+                }
             })
             .catch(function (err) {
-                // JSON-Endpoint nicht erreichbar: die serverseitig gerenderte
-                // (No-JS-)Ansicht bleibt sichtbar und funktioniert weiter.
-                console.error('Suchergebnisse konnten nicht nachgeladen werden:', err);
+                /*
+                 * Falls das Nachladen nicht funktioniert, bleibt die bereits
+                 * serverseitig ausgegebene Ergebnisliste sichtbar.
+                 */
+                console.error(
+                    'Suchergebnisse konnten nicht nachgeladen werden:',
+                    err
+                );
+            })
+            .finally(function () {
+                if (loadingElement) {
+                    loadingElement.hidden = true;
+                }
             });
     }
 

js/showCategory.js

@@ -21,6 +21,7 @@
 
     let listContainer, sortRadios, searchInput, limitSelect;
     let prevBtn, nextBtn, numbersContainer, jsNav, resultCountEl;
+    let loadingElement;
     let searchDebounceTimer;
 
     function init() {
@@ -35,23 +36,41 @@
         numbersContainer = document.getElementById('dynamic-page-numbers');
         jsNav = document.getElementById('js-page-navigation');
         resultCountEl = document.getElementById('s-res-result-count');
+        loadingElement = document.getElementById('results-loading');
 
         const checkedRadio = document.querySelector('.sort-radio:checked');
         state.sort = checkedRadio ? checkedRadio.value : 'alphabet';
         state.query = searchInput ? searchInput.value : '';
         state.itemsPerPage = limitSelect ? (parseInt(limitSelect.value, 10) || 10) : 10;
 
+        if (loadingElement) {
+            loadingElement.hidden = false;
+        }
+
         fetchFullDataset()
             .then(function (data) {
                 state.allItems = data.results || [];
                 attachEvents();
                 render();
-                if (jsNav) jsNav.style.display = 'flex';
+
+                if (jsNav) {
+                    jsNav.style.display = 'flex';
+                }
             })
             .catch(function (err) {
-                // JSON-Endpoint nicht erreichbar: die serverseitig gerenderte
-                // (No-JS-)Ansicht bleibt sichtbar und funktioniert weiter.
-                console.error('Kategorie-Beiträge konnten nicht nachgeladen werden:', err);
+                /*
+                 * Falls das Nachladen nicht funktioniert, bleibt die bereits
+                 * serverseitig ausgegebene Kategorieansicht sichtbar.
+                 */
+                console.error(
+                    'Kategorie-Beiträge konnten nicht nachgeladen werden:',
+                    err
+                );
+            })
+            .finally(function () {
+                if (loadingElement) {
+                    loadingElement.hidden = true;
+                }
             });
     }
 

php/controller/index-controller.php

@@ -20,6 +20,8 @@ if ($pfad === "login") {
     include_once "php/controller/confirm-register-controller.php";
 } elseif ($pfad === "confirm-password") {
     include_once "php/controller/confirm-password-controller.php";
+} elseif ($pfad === "profile") {
+    include_once "php/controller/profile-controller.php";
 } elseif ($pfad === "updateComment") {
     include_once "php/controller/updateComment-controller.php";
 } elseif($pfad === "deleteComment") {

php/model/DatabaseArticleManager.php

@@ -63,9 +63,12 @@ class DatabaseArticleManager implements ArticleManagerDAO {
                     VALUES (:title, :content, :author, :category, :tags);";
 
             $command = $db->prepare($sql);
+            if (!$command) {
+                throw new InternalServerErrorException("internal_error");
+            }
 
             // Verknüpft die übergebenen Parameter exakt mit den SQL-Platzhaltern
-            $command->execute([
+            $success = $command->execute([
                 ":title"    => $title,
                 ":content"  => $content,
                 ":author"   => $author,
@@ -73,10 +76,13 @@ class DatabaseArticleManager implements ArticleManagerDAO {
                 ":tags"     => $tags
             ]);
 
+            if (!$success) {
+                throw new InternalServerErrorException("internal_error");
+            }
+
             return intval($db->lastInsertId());
 
         } catch (PDOException $e) {
-
             throw new InternalServerErrorException("internal_error");
         }
     }
@@ -321,7 +327,6 @@ class DatabaseArticleManager implements ArticleManagerDAO {
         }
     }
 
-
     /**
      * Holt alle User-IDs, die einen bestimmten Beitrag geliked haben.
      *