Update updateArticle-controller.php

content/createArticle.php

@@ -4,12 +4,28 @@ if (!isset($_SESSION["user"])) {
     header("Location: index.php?pfad=login");
     exit();
 }
+
+// -------------------------------------------------------------------------
+// Aktuelle Blockliste ermitteln: Session-Fallback (nach Validierungsfehler
+// oder Zwischen-Submit "add_block"/"delete_block") oder leeres Array (Start).
+// -------------------------------------------------------------------------
+$rawContent = '[]';
+if (isset($_SESSION['old_content']) && !empty($_SESSION['old_content'])) {
+    $rawContent = $_SESSION['old_content'];
+    unset($_SESSION['old_content']);
+}
+
+$blocks = json_decode($rawContent, true);
+if (!is_array($blocks)) {
+    $blocks = [];
+}
+$blocks = array_values($blocks); // sequentielle Indizes 0..n-1 sicherstellen
 ?>
 <!--
     Seite: Beitrag erstellen
     Inhalt: Formular für die Erstellung eines neuen Beitrags
 -->
-<form method="post" action="php/controller/createArticle-controller.php" id="editor-form" class="article-editor-scope.editor-container article-editor-scope editor-container">
+<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">
 
     <main class="editor-main">
         <?php include_once "includes/alertMessages.php"?>
@@ -18,27 +34,49 @@ if (!isset($_SESSION["user"])) {
                value="<?php echo htmlspecialchars($_SESSION['old_title'] ?? ''); unset($_SESSION['old_title']); ?>"
                placeholder="Titel hier eingeben" required>
 
-        <!-- Hier werden die dynamischen divs via JavaScript eingefügt -->
+        <!--
-        <div id="block-container"></div>
+            Content-Blöcke: werden serverseitig als echte, benannte Formularfelder gerendert
+            (blocks[i][type], blocks[i][text] bzw. blocks[i][image]). Dadurch funktioniert das
+            Hinzufügen/Entfernen von Blöcken und der Bild-Upload auch ganz ohne JavaScript über
+            einen normalen Formular-Submit. Ist JavaScript aktiv, fängt js/editor.js diese
+            Submits ab und erledigt dieselbe Änderung lokal im DOM, ohne den Server zu belasten.
+        -->
+        <div id="block-container">
+            <?php foreach ($blocks as $i => $block): ?>
+                <?php
+                $blockType = $block['type'] ?? '';
+                if ($blockType !== 'text' && $blockType !== 'image') {
+                    continue; // unbekannter/kaputter Block wird übersprungen
+                }
+                ?>
+                <div class="editor-block article-editor-scope" data-index="<?php echo (int)$i; ?>">
+                    <input type="hidden" name="blocks[<?php echo (int)$i; ?>][type]" value="<?php echo htmlspecialchars($blockType); ?>">
 
-        <!-- Plus-Button und das Pop-up-Menü -->
+                    <?php if ($blockType === 'text'): ?>
-        <div id="add-block-control" class="article-editor-scope add-block-control">
+                        <textarea name="blocks[<?php echo (int)$i; ?>][text]"
-            <button type="button" id="plus-button" class="article-editor-scope plus-button">+</button>
+                                  placeholder="Schreibe deinen Textblock..."><?php echo htmlspecialchars($block['value'] ?? ''); ?></textarea>
-            <div id="block-popup" class="article-editor-scope block-popup hidden">
+                    <?php else: /* image */ ?>
-                <button type="button" data-type="text">Textblock</button>
+                        <?php if (!empty($block['value'])): ?>
-                <button type="button" data-type="image">Bild einfügen</button>
+                            <img src="<?php echo htmlspecialchars($block['value']); ?>"
-            </div>
+                                 class="block-image-preview"
+                                 style="max-width:200px;display:block;margin-top:10px;">
+                            <input type="hidden" name="blocks[<?php echo (int)$i; ?>][existing_image]" value="<?php echo htmlspecialchars($block['value']); ?>">
+                        <?php endif; ?>
+                        <input type="file" name="blocks[<?php echo (int)$i; ?>][image]" accept="image/*">
+                    <?php endif; ?>
+
+                    <button type="submit" name="editor_action" value="delete_block:<?php echo (int)$i; ?>" class="delete-block-btn">✕</button>
+                </div>
+            <?php endforeach; ?>
         </div>
 
-        <!-- Unsichtbares Textfeld, das die JSON-Daten hält und an den Controller postet -->
+        <div id="add-block-control" class="article-editor-scope add-block-control">
-        <textarea id="content" name="content" style="display:none;"><?php
+            <button type="button" id="plus-button" class="article-editor-scope plus-button">+</button>
-            if (isset($_SESSION['old_content']) && !empty($_SESSION['old_content'])){
+            <div id="block-popup" class="article-editor-scope block-popup">
-                echo htmlspecialchars($_SESSION['old_content']);
+                <button type="submit" name="editor_action" value="add_text" data-type="text">Textblock</button>
-                unset($_SESSION['old_content']);
+                <button type="submit" name="editor_action" value="add_image" data-type="image">Bild einfügen</button>
-            } else {
+            </div>
-                echo '[]'; // Standardmäßig ein leeres JSON-Array
+        </div>
-            }
-            ?></textarea>
     </main>
 
     <!-- Seitenleiste -->

includes/article-block-helper.php

@@ -0,0 +1,69 @@
+<?php
+/**
+ * Baut die Blockliste aus den POST-Daten (blocks[i][type], blocks[i][text],
+ * blocks[i][existing_image]) und ggf. hochgeladenen Dateien (blocks[i][image])
+ * zusammen. Läuft bei JEDEM Submit (Zwischen-Schritt "Block hinzufügen/löschen"
+ * UND finales Speichern/Veröffentlichen), damit neu ausgewählte Bilder in jedem
+ * Fall persistiert werden, bevor PHP die temporäre Upload-Datei nach
+ * Request-Ende verwirft.
+ *
+ * Wird sowohl vom createArticle- als auch vom updateArticle-Controller genutzt.
+ *
+ * @param array  $postBlocks  $_POST['blocks'] ?? []
+ * @param array  $fileBlocks  $_FILES['blocks'] ?? []
+ * @param string $uploadDir   absoluter Pfad zum uploads-Verzeichnis (mit trailing slash)
+ * @return array Liste von ['type' => 'text'|'image', 'value' => string]
+ */
+function rebuildBlocksFromPost(array $postBlocks, array $fileBlocks, string $uploadDir): array {
+    $allowedExtensions = ['jpg', 'jpeg', 'png', 'gif', 'webp'];
+
+    $keys = array_keys($postBlocks);
+    if (isset($fileBlocks['name']) && is_array($fileBlocks['name'])) {
+        $keys = array_unique(array_merge($keys, array_keys($fileBlocks['name'])));
+    }
+    sort($keys, SORT_NUMERIC);
+
+    $blocks = [];
+
+    foreach ($keys as $key) {
+        $type = $postBlocks[$key]['type'] ?? null;
+
+        if ($type === 'text') {
+            $blocks[] = [
+                'type'  => 'text',
+                'value' => $postBlocks[$key]['text'] ?? '',
+            ];
+        } elseif ($type === 'image') {
+            // Vorbelegung: bereits vorhandenes Server-Bild (falls Datei nicht ersetzt wird)
+            $value = $postBlocks[$key]['existing_image'] ?? '';
+
+            $hasUpload = isset($fileBlocks['error'][$key]['image'])
+                && $fileBlocks['error'][$key]['image'] === UPLOAD_ERR_OK;
+
+            if ($hasUpload) {
+                $tmpName      = $fileBlocks['tmp_name'][$key]['image'];
+                $originalName = $fileBlocks['name'][$key]['image'];
+                $extension    = strtolower(pathinfo($originalName, PATHINFO_EXTENSION));
+                if (!in_array($extension, $allowedExtensions, true)) {
+                    $extension = 'jpg';
+                }
+
+                $fileName    = 'img_' . uniqid() . '.' . $extension;
+                $destination = $uploadDir . $fileName;
+
+                if (move_uploaded_file($tmpName, $destination)) {
+                    $value = 'uploads/' . $fileName;
+                }
+                // Bei Fehler: alter Wert (falls vorhanden) bleibt erhalten, Block wird nicht verworfen
+            }
+
+            $blocks[] = [
+                'type'  => 'image',
+                'value' => $value,
+            ];
+        }
+        // unbekannter/fehlender type -> Block wird ignoriert
+    }
+
+    return $blocks;
+}

php/controller/createArticle-controller.php

@@ -5,134 +5,120 @@ if (session_status() === PHP_SESSION_NONE) {
 require_once '../model/LocalArticleManager.php';
 require_once '../model/ArticleManager.php';
 require_once '../validator/article-validator.php';
+require_once '../../includes/article-block-helper.php';
 
 if (!isset($_SESSION["user"])) {
     header("Location: index.php?pfad=login");
     exit();
 }
-    if ($_SERVER["REQUEST_METHOD"] === "POST") {
-        $_SESSION["old_title"]    = $_POST["title"] ?? '';
-        $_SESSION["old_content"]  = $_POST["content"] ?? '';
-        $_SESSION["old_category"] = $_POST["category"] ?? '';
-        $_SESSION["old_tags"]     = $_POST["tags"] ?? '';
 
-        if(!isset($_POST["title"]) ||!isset($_POST["content"]) || !isset($_POST["category"])){
+if ($_SERVER["REQUEST_METHOD"] === "POST") {
-            $_SESSION["message"] = "missing_parameters";
+
+    $uploadDir = __DIR__ . '/../../uploads/';
+    if (!file_exists($uploadDir)) {
+        mkdir($uploadDir, 0755, true);
+    }
+
+    // Formularzustand (Titel/Tags/Kategorie/Blöcke) immer sichern, damit er nach einem
+    // Redirect (PRG-Pattern oder Validierungsfehler) wieder angezeigt werden kann.
+    $_SESSION["old_title"]    = $_POST["title"] ?? '';
+    $_SESSION["old_category"] = $_POST["category"] ?? '';
+    $_SESSION["old_tags"]     = $_POST["tags"] ?? '';
+
+    $blocks = rebuildBlocksFromPost($_POST['blocks'] ?? [], $_FILES['blocks'] ?? [], $uploadDir);
+    $_SESSION["old_content"] = json_encode($blocks, JSON_UNESCAPED_UNICODE);
+
+    // ---------------------------------------------------------------------
+    // Zwischenspeichern
+    // ---------------------------------------------------------------------
+    if (isset($_POST['editor_action']) && $_POST['editor_action'] !== '') {
+        $action = $_POST['editor_action'];
+
+        if ($action === 'add_text') {
+            $blocks[] = ['type' => 'text', 'value' => ''];
+        } elseif ($action === 'add_image') {
+            $blocks[] = ['type' => 'image', 'value' => ''];
+        } elseif (str_starts_with($action, 'delete_block:')) {
+            $deleteIndex = (int) substr($action, strlen('delete_block:'));
+            unset($blocks[$deleteIndex]);
+            $blocks = array_values($blocks);
+        }
+
+        $_SESSION["old_content"] = json_encode($blocks, JSON_UNESCAPED_UNICODE);
+        header("location: ../../index.php?pfad=createArticle");
+        exit();
+    }
+
+    // ---------------------------------------------------------------------
+    // Echtes Veröffentlichen
+    // ---------------------------------------------------------------------
+    if (!isset($_POST["title"]) || !isset($_POST["category"])) {
+        $_SESSION["message"] = "missing_parameters";
+        header("location: ../../index.php?pfad=createArticle");
+        exit();
+    } else {
+        $title    = $_POST["title"];
+        $content  = json_encode($blocks, JSON_UNESCAPED_UNICODE);
+        $author   = $_SESSION["user_email"];
+        $category = $_POST["category"];
+        $tags     = $_POST['tags'] ?? '';
+
+        // -------------------------------- Validierung der Daten: -------------------------
+        if (!articleTitleValidator($title)) {
+            $_SESSION["message"] = "invalid_title";
+            header("location: ../../index.php?pfad=createArticle");
+            exit();
+        }
+
+        if (!articleContentValidator($content)) {
+            $_SESSION["message"] = "invalid_content";
+            header("location: ../../index.php?pfad=createArticle");
+            exit();
+        }
+
+        if (!articleCategoryValidator($category)) {
+            $_SESSION["message"] = "invalid_category";
+            header("location: ../../index.php?pfad=createArticle");
+            exit();
+        }
+
+        if (!articleTagValidator($tags)) {
+            $_SESSION["message"] = "invalid_tags";
             header("location: ../../index.php?pfad=createArticle");
             exit();
         } else {
-            $title = $_POST["title"];
+            $cleanedTags = [];
-            $content = $_POST["content"];
+            $rawTags = explode(',', $tags);
-            $author = $_SESSION["user_email"];
+            foreach ($rawTags as $rawTag) {
-            $category = $_POST["category"];
+                // Leerzeichen am Anfang/Ende des einzelnen Tags entfernen:
-            $tags = $_POST['tags'] ?? '';
+                $tag = trim($rawTag);
-
+                $cleanedTags[] = $tag;
-            // -------------------------------- Validierung der Daten: -------------------------
-            if (!articleTitleValidator($title)) {
-                $_SESSION["message"] = "invalid_title";
-                header("location: ../../index.php?pfad=createArticle");
-                exit();
             }
+            // Duplikate entfernen:
+            $cleanedTags = array_unique($cleanedTags);
+            $cleanedTags = implode(',', $cleanedTags);
+        }
 
-            if (!articleContentValidator($content)) {
+        // ----------------- Übertragung der validierten Daten in ArticleManager: ---------------------------
-                $_SESSION["message"] = "invalid_content";
+        try {
-                header("location: ../../index.php?pfad=createArticle");
+            $articleManager = ArticleManager::getInstance();
-                exit();
+            // $content enthält bereits die finalen "uploads/..."-Pfade (kein Base64 mehr),
-            }
+            // da rebuildBlocksFromPost() Datei-Uploads sofort verarbeitet.
+            $articleManager->addArticle($title, $content, $author, $category, $cleanedTags);
 
-            if (!articleCategoryValidator($category)) {
+            // Formulardaten nach erfolgreichem Erstellen aus der Session löschen
-                $_SESSION["message"] = "invalid_category";
+            unset($_SESSION["old_title"], $_SESSION["old_content"], $_SESSION["old_category"], $_SESSION["old_tags"]);
-                header("location: ../../index.php?pfad=createArticle");
-                exit();
-            }
 
-            if (!articleTagValidator($tags)) {
+        } catch (\Throwable $e) {
-                $_SESSION["message"] = "invalid_tags";
+            $_SESSION["message"] = "internal_error";
-                header("location: ../../index.php?pfad=createArticle");
+            header("location: ../../index.php?pfad=createArticle");
-                exit();
-            } else {
-                $cleanedTags = [];
-                $rawTags = explode(',', $tags);
-                foreach ($rawTags as $rawTag) {
-                    // Leerzeichen am Anfang/Ende des einzelnen Tags entfernen:
-                    $tag = trim($rawTag);
-                    $cleanedTags[] = $tag;
-                }
-                // Duplikate entfernen:
-                $cleanedTags = array_unique($cleanedTags);
-                $cleanedTags = implode(',', $cleanedTags);
-            }
-
-            // ----------------- Base64-Bilder verarbeiten und auf Server speichern -----------------
-            $blocks = json_decode($content, true);
-            $uploadDir = __DIR__ . '/../../uploads/';
-
-            if (!file_exists($uploadDir)) {
-                mkdir($uploadDir, 0755, true);
-            }
-
-            if (is_array($blocks)) {
-                foreach ($blocks as &$block) {
-                    // sicherstellen, dass 'type' und 'value' existieren:
-                    if (isset($block['type']) && isset($block['value']) && $block['type'] === 'image' && str_starts_with($block['value'], 'data:image/')) {
-
-                        // Base64-String zerlegen
-                        $parts = explode(',', $block['value']);
-
-                        // falls der String korrupt ist und kein Komma hat
-                        if (count($parts) < 2) {
-                            continue;
-                        }
-
-                        $metadata = $parts[0];
-                        $base64Data = $parts[1];
-
-                        // Dateiendung ermitteln
-                        preg_match('/data:image\/(?<extension>.*?);/', $metadata, $matches);
-                        $extension = $matches['extension'] ?? 'jpg';
-                        if ($extension === 'jpeg') {
-                            $extension = 'jpg';
-                        }
-
-                        // Eindeutigen Dateinamen generieren
-                        $fileName = 'img_' . uniqid() . '.' . $extension;
-                        $filePath = $uploadDir . $fileName;
-
-                        // Datei im /uploads speichern:
-                        if (file_put_contents($filePath, base64_decode($base64Data)) !== false) {
-                            // temporären Base64-String durch den echten Pfad ersetzen
-                            $block['value'] = 'uploads/' . $fileName;
-                        } else {
-                            $_SESSION["message"] = "image_upload_error";
-                            header("location: ../../index.php?pfad=createArticle");
-                            exit();
-                        }
-                    }
-                }
-                unset($block);
-            }
-
-            // Aktualisiertes Array wieder in JSON konvertieren
-            $finalContent = json_encode($blocks, JSON_UNESCAPED_UNICODE);
-
-            // ----------------- Übertragung der validierten Daten in ArticleManager: ---------------------------
-            try {
-                $articleManager = ArticleManager::getInstance();
-                $articleManager->addArticle($title, $content, $author, $category, $cleanedTags);
-
-                // Formulardaten nach erfolgreichem Erstellen aus der Session löschen
-                unset($_SESSION["old_title"], $_SESSION["old_content"], $_SESSION["old_category"], $_SESSION["old_tags"]);
-
-            } catch (\Throwable $e){
-                $_SESSION["message"] = "internal_error";
-                header("location: ../../index.php?pfad=createArticle");
-                exit();
-            }
-
-            $_SESSION["message"] = "new_article";
-            // Weiterleitung zur Homepage
-            header("location: ../../index.php");
             exit();
         }
+
+        $_SESSION["message"] = "new_article";
+        // Weiterleitung zur Homepage
+        header("location: ../../index.php");
+        exit();
     }
+}
 ?>

php/controller/updateArticle-controller.php

@@ -7,78 +7,13 @@ require_once '../model/LocalArticleManager.php';
 require_once '../model/ArticleManager.php';
 require_once '../model/Article.php';
 require_once '../validator/article-validator.php';
+require_once '../../includes/article-block-helper.php';
 
 if (!isset($_SESSION["user_email"])) {
     header("Location: index.php?pfad=login");
     exit();
 }
 
-/**
- * Baut die Blockliste aus den POST-Daten (blocks[i][type], blocks[i][text],
- * blocks[i][existing_image]) und ggf. hochgeladenen Dateien (blocks[i][image])
- * zusammen. Läuft bei JEDEM Submit (Zwischen-Schritt "Block hinzufügen/löschen"
- * UND finales Speichern), damit neu ausgewählte Bilder in jedem Fall persistiert
- * werden, bevor PHP die temporäre Upload-Datei nach Request-Ende verwirft.
- *
- * @param array  $postBlocks  $_POST['blocks'] ?? []
- * @param array  $fileBlocks  $_FILES['blocks'] ?? []
- * @param string $uploadDir   absoluter Pfad zum uploads-Verzeichnis (mit trailing slash)
- * @return array Liste von ['type' => 'text'|'image', 'value' => string]
- */
-function rebuildBlocksFromPost(array $postBlocks, array $fileBlocks, string $uploadDir): array {
-    $allowedExtensions = ['jpg', 'jpeg', 'png', 'gif', 'webp'];
-
-    $keys = array_keys($postBlocks);
-    if (isset($fileBlocks['name']) && is_array($fileBlocks['name'])) {
-        $keys = array_unique(array_merge($keys, array_keys($fileBlocks['name'])));
-    }
-    sort($keys, SORT_NUMERIC);
-
-    $blocks = [];
-
-    foreach ($keys as $key) {
-        $type = $postBlocks[$key]['type'] ?? null;
-
-        if ($type === 'text') {
-            $blocks[] = [
-                'type'  => 'text',
-                'value' => $postBlocks[$key]['text'] ?? '',
-            ];
-        } elseif ($type === 'image') {
-            // Vorbelegung: bereits vorhandenes Server-Bild (falls Datei nicht ersetzt wird)
-            $value = $postBlocks[$key]['existing_image'] ?? '';
-
-            $hasUpload = isset($fileBlocks['error'][$key]['image'])
-                && $fileBlocks['error'][$key]['image'] === UPLOAD_ERR_OK;
-
-            if ($hasUpload) {
-                $tmpName      = $fileBlocks['tmp_name'][$key]['image'];
-                $originalName = $fileBlocks['name'][$key]['image'];
-                $extension    = strtolower(pathinfo($originalName, PATHINFO_EXTENSION));
-                if (!in_array($extension, $allowedExtensions, true)) {
-                    $extension = 'jpg';
-                }
-
-                $fileName    = 'img_' . uniqid() . '.' . $extension;
-                $destination = $uploadDir . $fileName;
-
-                if (move_uploaded_file($tmpName, $destination)) {
-                    $value = 'uploads/' . $fileName;
-                }
-                // Bei Fehler: alter Wert (falls vorhanden) bleibt erhalten, Block wird nicht verworfen
-            }
-
-            $blocks[] = [
-                'type'  => 'image',
-                'value' => $value,
-            ];
-        }
-        // unbekannter/fehlender type -> Block wird ignoriert
-    }
-
-    return $blocks;
-}
-
 if ($_SERVER["REQUEST_METHOD"] === "POST") {
 
     if (isset($_GET["id"]) && !empty($_GET["id"])) {
@@ -118,7 +53,9 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
     $_SESSION["old_content"] = json_encode($blocks, JSON_UNESCAPED_UNICODE);
 
     // ---------------------------------------------------------------------
-    // Zwischenspeichern
+    // Zwischen-Schritt: Block hinzufügen oder entfernen (kein echtes Speichern).
+    // Wird bei aktivem JavaScript per preventDefault() abgefangen und lokal im
+    // DOM erledigt (js/editor.js) – ohne JS läuft dieser Server-Roundtrip.
     // ---------------------------------------------------------------------
     if (isset($_POST['editor_action']) && $_POST['editor_action'] !== '') {
         $action = $_POST['editor_action'];
@@ -191,7 +128,12 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
         // ----------------- Verwaiste Bilder aufräumen -----------------
         // Bilder, die im alten (gespeicherten) Content vorkamen, im neuen aber nicht
         // mehr referenziert werden, wurden vom Nutzer entfernt oder ersetzt -> löschen.
-        // TODO: Bilder, die innerhalb derselben Bearbeitungs-Sitzung neu hochgeladen und noch vor dem finalen Speichern wieder entfernt wurden, werden hierüber nicht erfasst (sie tauchten nie im alten Content auf) und bleiben als Datei liegen. Für eine vollständige Bereinigung würde sich ein
+        // Hinweis/TODO: Bilder, die innerhalb derselben Bearbeitungs-Sitzung neu
+        // hochgeladen und noch vor dem finalen Speichern wieder entfernt wurden,
+        // werden hierüber nicht erfasst (sie tauchten nie im alten Content auf) und
+        // bleiben als Datei liegen. Für eine vollständige Bereinigung würde sich ein
+        // regelmäßiger Cleanup-Job anbieten, der verwaiste Dateien im uploads/-Ordner
+        // mit den in der Datenbank referenzierten Pfaden abgleicht.
         $oldBlocks = json_decode($article->getContent(), true);
         $oldImagePaths = [];
         if (is_array($oldBlocks)) {