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); ?>">
+
+                    <?php if ($blockType === 'text'): ?>
+                        <textarea name="blocks[<?php echo (int)$i; ?>][text]"
+                                  placeholder="Schreibe deinen Textblock..."><?php echo htmlspecialchars($block['value'] ?? ''); ?></textarea>
+                    <?php else: /* image */ ?>
+                        <?php if (!empty($block['value'])): ?>
+                            <img src="<?php echo htmlspecialchars($block['value']); ?>"
+                                 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>
 
-        <!-- Plus-Button und das Pop-up-Menü -->
         <div id="add-block-control" class="article-editor-scope add-block-control">
             <button type="button" id="plus-button" class="article-editor-scope plus-button">+</button>
-            <div id="block-popup" class="article-editor-scope block-popup hidden">
+            <div id="block-popup" class="article-editor-scope block-popup">
-                <button type="button" data-type="text">Textblock</button>
+                <button type="submit" name="editor_action" value="add_text" data-type="text">Textblock</button>
-                <button type="button" data-type="image">Bild einfügen</button>
+                <button type="submit" name="editor_action" value="add_image" data-type="image">Bild einfügen</button>
             </div>
         </div>
-
-        <!-- Unsichtbares Textfeld, das die JSON-Daten hält und an den Controller postet -->
-        <textarea id="content" name="content" style="display:none;"><?php
-            if (isset($_SESSION['old_content']) && !empty($_SESSION['old_content'])){
-                echo htmlspecialchars($_SESSION['old_content']);
-                unset($_SESSION['old_content']);
-            } else {
-                echo '[]'; // Standardmäßig ein leeres JSON-Array
-            }
-            ?></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,24 +5,60 @@ 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") {
+
+if ($_SERVER["REQUEST_METHOD"] === "POST") {
+
+    $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_content"]  = $_POST["content"] ?? '';
     $_SESSION["old_category"] = $_POST["category"] ?? '';
     $_SESSION["old_tags"]     = $_POST["tags"] ?? '';
 
-        if(!isset($_POST["title"]) ||!isset($_POST["content"]) || !isset($_POST["category"])){
+    $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 = $_POST["content"];
+        $content  = json_encode($blocks, JSON_UNESCAPED_UNICODE);
         $author   = $_SESSION["user_email"];
         $category = $_POST["category"];
         $tags     = $_POST['tags'] ?? '';
@@ -63,67 +99,17 @@ if (!isset($_SESSION["user"])) {
             $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();
+            // $content enthält bereits die finalen "uploads/..."-Pfade (kein Base64 mehr),
+            // da rebuildBlocksFromPost() Datei-Uploads sofort verarbeitet.
             $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){
+        } catch (\Throwable $e) {
             $_SESSION["message"] = "internal_error";
             header("location: ../../index.php?pfad=createArticle");
             exit();
@@ -134,5 +120,5 @@ if (!isset($_SESSION["user"])) {
         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)) {