Update updateArticle.php

revert debugging

README.md

@@ -7,7 +7,7 @@
 
 ## Login-Informationen für Dummy-User
 `Anmeldename, Passwort, Mailadresse`:
-- `max.mustermann, test12345, mustermann@web.de`
+- `max.mustermann, test12345, max.mustermann@web.de`
 
 ## Weitere Voraussetzungen zur Nutzung
 - Per Klick auf das Logo gelangt man auf die Home-Seite.
@@ -18,6 +18,7 @@
 - Bitte auf die gesetzten TODO's achten. Wenn Inhalte fehlen, sind sie i.d.R. als TODO kommentiert.
 - Die Suchseite und Kategorieseite packen momentan alle passenden Beiträge untereinander. Später sollen zunächst 10 
   Ergebnisse auf einer Seite angezeigt werden.
+- Wenn ein Bild aus einem Beitrag entfernt wird, dann wird noch nicht die Datei im Pfad /uploads gelöscht.
 
 ## Besonderheiten des Projektes
 - Es wurde ein einfacher Beitrags-Editor erstellt. Mit diesem können Beiträge erstellt oder bearbeitet werden.

content/updateArticle.php

@@ -65,45 +65,211 @@ include_once 'php/controller/showArticle-controller.php';
         <div class="sidebar-block">
             <label for="category">Kategorie <span class="required">*</span></label>
             <select id="category" name="category" required>
-                <option disabled selected>Kategorie wählen...</option>
+                <option disabled <?php
+                if ((!isset($_SESSION['old_category']) || empty($_SESSION['old_category'])) && (!isset($category) || empty($category))) {
+                    echo 'selected';
+                }
+                ?>>Kategorie wählen...</option>
 
                 <optgroup label="Sprachen">
-                    <option value="deutsch">Deutsch</option>
-                    <option value="englisch">Englisch</option>
-                    <option value="franzoesisch">Französisch</option>
-                    <option value="latein">Latein</option>
-                    <option value="literatur">Literatur</option>
+                    <option value="deutsch" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'deutsch') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'deutsch') { echo 'selected'; }
+                    }
+                    ?>>Deutsch</option>
+                    <option value="englisch" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'englisch') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'englisch') { echo 'selected'; }
+                    }
+                    ?>>Englisch</option>
+                    <option value="franzoesisch" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'franzoesisch') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'franzoesisch') { echo 'selected'; }
+                    }
+                    ?>>Französisch</option>
+                    <option value="latein" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'latein') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'latein') { echo 'selected'; }
+                    }
+                    ?>>Latein</option>
+                    <option value="literatur" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'literatur') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'literatur') { echo 'selected'; }
+                    }
+                    ?>>Literatur</option>
                 </optgroup>
 
                 <optgroup label="MINT">
-                    <option value="mathe">Mathematik</option>
-                    <option value="biologie">Biologie</option>
-                    <option value="chemie">Chemie</option>
-                    <option value="physik">Physik</option>
-                    <option value="informatik">Informatik</option>
-                    <option value="astronomie">Astronomie</option>
+                    <option value="mathe" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'mathe') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'mathe') { echo 'selected'; }
+                    }
+                    ?>>Mathematik</option>
+                    <option value="biologie" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'biologie') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'biologie') { echo 'selected'; }
+                    }
+                    ?>>Biologie</option>
+                    <option value="chemie" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'chemie') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'chemie') { echo 'selected'; }
+                    }
+                    ?>>Chemie</option>
+                    <option value="physik" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'physik') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'physik') { echo 'selected'; }
+                    }
+                    ?>>Physik</option>
+                    <option value="informatik" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'informatik') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'informatik') { echo 'selected'; }
+                    }
+                    ?>>Informatik</option>
+                    <option value="astronomie" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'astronomie') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'astronomie') { echo 'selected'; }
+                    }
+                    ?>>Astronomie</option>
                 </optgroup>
 
                 <optgroup label="Gesellschaft & Werte">
-                    <option value="geschichte">Geschichte</option>
-                    <option value="erdkunde">Erdkunde</option>
-                    <option value="sozialkunde">Sozialkunde</option>
-                    <option value="wirtschaft">Wirtschaftskunde</option>
-                    <option value="religion">Religion</option>
-                    <option value="ethik">Ethikunterricht</option>
-                    <option value="philosophie">Philosophie</option>
-                    <option value="psychologie">Psychologie</option>
-                    <option value="kunst">Kunst</option>
-                    <option value="musik">Musik</option>
-                    <option value="theater">Theater</option>
+                    <option value="geschichte" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'geschichte') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'geschichte') { echo 'selected'; }
+                    }
+                    ?>>Geschichte</option>
+                    <option value="erdkunde" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'erdkunde') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'erdkunde') { echo 'selected'; }
+                    }
+                    ?>>Erdkunde</option>
+                    <option value="sozialkunde" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'sozialkunde') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'sozialkunde') { echo 'selected'; }
+                    }
+                    ?>>Sozialkunde</option>
+                    <option value="wirtschaft" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'wirtschaft') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'wirtschaft') { echo 'selected'; }
+                    }
+                    ?>>Wirtschaftskunde</option>
+                    <option value="religion" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'religion') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'religion') { echo 'selected'; }
+                    }
+                    ?>>Religion</option>
+                    <option value="ethik" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'ethik') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'ethik') { echo 'selected'; }
+                    }
+                    ?>>Ethikunterricht</option>
+                    <option value="philosophie" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'philosophie') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'philosophie') { echo 'selected'; }
+                    }
+                    ?>>Philosophie</option>
+                    <option value="psychologie" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'psychologie') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'psychologie') { echo 'selected'; }
+                    }
+                    ?>>Psychologie</option>
+                    <option value="kunst" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'kunst') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'kunst') { echo 'selected'; }
+                    }
+                    ?>>Kunst</option>
+                    <option value="musik" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'musik') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'musik') { echo 'selected'; }
+                    }
+                    ?>>Musik</option>
+                    <option value="theater" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'theater') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'theater') { echo 'selected'; }
+                    }
+                    ?>>Theater</option>
                 </optgroup>
 
                 <optgroup label="Technik & Praxis">
-                    <option value="technik">Technik</option>
-                    <option value="werken">Werken</option>
-                    <option value="hauswirtschaft">Hauswirtschaft</option>
-                    <option value="sport">Sport</option>
+                    <option value="technik" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'technik') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'technik') { echo 'selected'; }
+                    }
+                    ?>>Technik</option>
+                    <option value="werken" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'werken') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'werken') { echo 'selected'; }
+                    }
+                    ?>>Werken</option>
+                    <option value="hauswirtschaft" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'hauswirtschaft') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'hauswirtschaft') { echo 'selected'; }
+                    }
+                    ?>>Hauswirtschaft</option>
+                    <option value="sport" <?php
+                    if (isset($_SESSION['old_category']) && !empty($_SESSION['old_category'])) {
+                        if ($_SESSION['old_category'] === 'sport') { echo 'selected'; }
+                    } elseif (isset($category) && !empty($category)) {
+                        if ($category === 'sport') { echo 'selected'; }
+                    }
+                    ?>>Sport</option>
                 </optgroup>
+
+                <?php
+                if (isset($_SESSION['old_category'])) {
+                    unset($_SESSION['old_category']);
+                }
+                ?>
             </select>
         </div>
 

js/editor.js

@@ -15,6 +15,8 @@ function initEditor() {
     const popup = document.getElementById("block-popup");
     const hiddenContentInput = document.getElementById("content");
 
+    const initialImages = [];
+
     // Pop-up umschalten bei Klick auf das Plus
     plusButton.addEventListener("click", () => {
         popup.classList.toggle("hidden");
@@ -35,25 +37,19 @@ function initEditor() {
         blockDiv.classList.add("editor-block");
         blockDiv.setAttribute("data-type", type);
 
-        // Löschen-Button für den Block
+        // Wenn es ein existierendes Server-Bild beim Laden ist, Pfad im globalen Array sichern
+        if (type === "image" && value && typeof value === 'string' && value.startsWith('uploads/')) {
+            initialImages.push(value);
+            blockDiv.setAttribute("data-value", value);
+        }
+
+        // Löschen-Button
         const deleteBtn = document.createElement("button");
         deleteBtn.type = "button";
         deleteBtn.innerHTML = "✕";
         deleteBtn.classList.add("delete-block-btn");
         deleteBtn.addEventListener("click", () => {
-            const blockType = blockDiv.getAttribute("data-type");
-
-            // holt den Pfad aus dem data-value Attribut des divs
-            const blockValue = blockDiv.getAttribute("data-value");
-
-            if (blockType === "image" && blockValue && blockValue.startsWith("uploads/")) {
-                const deletedInput = document.getElementById("deleted-images");
-                if (deletedInput) {
-                    let deletedList = JSON.parse(deletedInput.value || "[]");
-                    deletedList.push(blockValue);
-                    deletedInput.value = JSON.stringify(deletedList);
-                }
-            }
+            // ANPASSUNG 2B: Logik hier komplett geleert. Das '✕' entfernt den Block jetzt nur noch sicher aus dem HTML.
             blockDiv.remove();
         });
         blockDiv.appendChild(deleteBtn);
@@ -98,9 +94,12 @@ function initEditor() {
         container.appendChild(blockDiv);
     }
 
-    // Beim Abschicken: HTML-Blöcke auslesen und als JSON-String serialisieren
+    // beim Abschicken verbleibende Blöcke auslesen UND gelöschte Bilder ermitteln
     form.addEventListener("submit", function(e) {
         const blocks = [];
+        const currentImages = [];
+
+        // alle aktuell im Formular verbliebenen Blöcke scannen
         container.querySelectorAll(".editor-block").forEach(blockDiv => {
             const type = blockDiv.getAttribute("data-type");
             let value = "";
@@ -108,20 +107,61 @@ function initEditor() {
             if (type === "text") {
                 value = blockDiv.querySelector("textarea").value;
             } else if (type === "image") {
+
+                const imgTag = blockDiv.querySelector("img");
+                if (imgTag) {
+                    const srcValue = imgTag.getAttribute("src") || "";
+                    // Wenn es ein neues Bild ist, nutzen wir das data-value (Base64)
+                    if (srcValue.startsWith('data:image/')) {
                         value = blockDiv.getAttribute("data-value") || "";
+                    } else {
+                        value = srcValue;
+                    }
+                }
+
+                // Pfade sammeln, die der Nutzer NICHT gelöscht hat (für den Abgleich)
+                if (value && value.startsWith('uploads/')) {
+                    currentImages.push(value);
+                }
             }
 
             blocks.push({ type: type, value: value });
         });
 
+        // das reguläre unsichtbare Content-Feld befüllen
         hiddenContentInput.value = JSON.stringify(blocks);
+
+        // Differenz berechnen: Welche Bilder aus 'initialImages' fehlen in 'currentImages' ?
+        const deletedImages = initialImages.filter(img => !currentImages.includes(img));
+
+        // das 'deleted_images'-Feld dynamisch befüllen und an den Controller senden
+        let deletedInput = document.getElementById("deleted-images");
+        if (!deletedInput) {
+            deletedInput = document.createElement("input");
+            deletedInput.type = "hidden";
+            deletedInput.id = "deleted-images";
+            deletedInput.name = "deleted_images";
+            form.appendChild(deletedInput);
+        }
+        deletedInput.value = JSON.stringify(deletedImages);
     });
 
     // Existierende Blöcke laden (stellt alte Daten aus der Session wieder her)
     try {
-        const initialBlocks = JSON.parse(hiddenContentInput.value);
+        const initialBlocks = JSON.parse(hiddenContentInput.value.trim());
         if (Array.isArray(initialBlocks)) {
-            initialBlocks.forEach(b => addBlockElement(b.type, b.value));
+            initialBlocks.forEach(b => {
+                if (b.type === "image" && b.value && typeof b.value === 'string' && !b.value.startsWith('data:image/')) {
+                    let cleanPath = b.value.trim().replace(/\\\//g, '/'); // Verwandelt \/ in /
+
+                    initialImages.push(cleanPath);
+                    addBlockElement(b.type, cleanPath);
+
+                } else {
+                    addBlockElement(b.type, b.value);
+                }
+            });
+            console.log("Erfolgreich registrierte Start-Bilder:", initialImages);
         }
     } catch(e) {
         if (hiddenContentInput.value.trim() !== "") {

php/controller/updateArticle-controller.php

@@ -76,9 +76,9 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
         if (!file_exists($uploadDir)) {
             mkdir($uploadDir, 0755, true);
         }
-        /*
-        // ----------------- Gelöschte Bilder über die JS-Löschliste entfernen -----------------
-        if (isset($_POST['deleted_images'])) {
+
+        // ----------------- Gelöschte Bilder über die JS-Löschliste entfernen ----------------- TODO: Gelöschte Bilder über die JS-Löschliste entfernen
+        /*if (isset($_POST['deleted_images'])) {
             $deletedImages = json_decode($_POST['deleted_images'], true);
 
             // Wir ermitteln den physisch echten, absoluten Pfad zum uploads-Ordner auf der Festplatte
@@ -104,51 +104,6 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
                 }
             }
         }*/
-        // ----------------- Gelöschte Bilder über die JS-Löschliste entfernen -----------------
-        if (isset($_POST['deleted_images'])) {
-
-            $rawPostData = $_POST['deleted_images'];
-
-            // DIAGNOSE 1: Wenn die Daten gar nicht ankommen oder leer sind, brechen wir ab und zeigen es an
-            if (empty($rawPostData) || $rawPostData === '[]') {
-                // Falls das Feld leer ankommt, entkommentieren Sie die nächste Zeile zum Debuggen:
-                die("Fehler: Das Feld 'deleted_images' kam leer oder als '[]' im Controller an!");
-            }
-
-            // Eventuell maskierte Anführungszeichen (Sicherheits-Overhead) bereinigen
-            $rawPostData = stripslashes($rawPostData);
-
-            $deletedImages = json_decode($rawPostData, true);
-
-            // DIAGNOSE 2: Wenn das JSON nicht dekodiert werden kann, zeigen wir den JSON-Fehler an
-            if (json_last_error() !== JSON_ERROR_NONE) {
-                die("JSON-Dekodierungsfehler im Controller: " . json_last_error_msg() . " | Rohdaten: " . htmlspecialchars($rawPostData));
-            }
-
-            $uploadDir = realpath(__DIR__ . '/../../uploads') . DIRECTORY_SEPARATOR;
-
-            if (is_array($deletedImages)) {
-                foreach ($deletedImages as $imagePath) {
-                    $filename = basename($imagePath);
-                    $fullDeletePath = $uploadDir . $filename;
-
-                    // DIAGNOSE 3: Wir prüfen, was genau PHP anstellt
-                    if (file_exists($fullDeletePath)) {
-                        if (!unlink($fullDeletePath)) {
-                            $error = error_get_last();
-                            die("Datei existiert unter " . $fullDeletePath . ", aber unlink() schlug fehl. Grund: " . $error['message']);
-                        } else {
-                            // ERFOLG! Datei wurde gelöscht. Sie können diese Zeile später entfernen.
-                            // die("Erfolgreich gelöscht: " . $fullDeletePath);
-                        }
-                    } else {
-                        // PHP findet die Datei nicht. Wir geben den exakten Pfad aus, an dem gesucht wurde
-                        die("Fehler: PHP sucht die Datei, aber sie existiert nicht unter diesem Pfad: <br><strong>" . $fullDeletePath . "</strong>");
-                    }
-                }
-            }
-        }
-// ----------------- Ende der Lösch-Logik -----------------
 
         // ----------------------- NEU hinzugefügte Base64-Bilder: --------------------------
         if (is_array($blocks)) {

php/model/ArticleManager.php

@@ -19,7 +19,7 @@ class ArticleManager
     public static function getInstance()
     {
         $articleManager = DatabaseArticleManager::getInstance(); // Hier kann zwischen dem lokalen und datenbankbasiertem ArticleManager gewechselt werden.
-        /*
+
         // 100 fiktionale Fachbeiträge:
         $dummyArticles = [
             // --- INFORMATIK & MATHE (1-20) ---
@@ -145,16 +145,24 @@ class ArticleManager
                 // Verteilt die 10 Autoren gleichmäßig (ID 1 -> Autor 1, ID 10 -> Autor 10, ID 11 -> Autor 1)
                 $authorEmail = $authors[($id - 1) % 10];
 
+                $blockStructure = [
+                    [
+                        'type' => 'text',
+                        'value' => $data[1] // Der originale Text aus dem Dummy-Array
+                    ]
+                ];
+                $jsonContent = json_encode($blockStructure, JSON_UNESCAPED_UNICODE);
+
                 $articleManager->addArticle(
                     $data[0], // Titel
-                    $data[1], // Inhalt
+                    $jsonContent, // Inhalt
                     $authorEmail, // Rotierende Autoren-E-Mail
                     $data[2], // Kategorie
                     $data[3]  // Tags
                 );
             }
         }
-        */
+
         return $articleManager;
     }