Forum implementiert #34
Reference in New Issue
Block a user
Delete Branch "Forum"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@@ -183,0 +187,4 @@<h2 class="section-title">Meine Kommentare</h2><div class="articles-list">Würde ich in ein eigenes Div packen, nicht in das gleiche Div wie die Beiträge
@@ -183,0 +201,4 @@<p><?php echo nl2br(htmlspecialchars($comment->getContent())); ?></p>Müssen Kommentare auch wieder gelöscht werden dürfen?
Ich weiß, dass das schwierig für den Kommentar-Baum ist...
Sonst lassen wir das erst so, wenn es zu umständlich ist.
Schaffe ich heute nicht mehr
@@ -66,3 +66,3 @@<div class="article-view-tags-list"><?php// Falls $tags ein String ist (z.B. "Web, CSS"), in ein Array umwandeln?
@@ -59,4 +60,3 @@<?php endif; ?></div><!-- Beitrags-Endbereich (Tags) -->?
@@ -54,4 +55,2 @@<!-- Beitrags-Inhalt --><div class="article-view-content"><?php if (isset($content)): ?><!-- nl2br für Zeilenumbrüche -->?
@@ -34,3 +35,2 @@?><?php unset($_SESSION["message"]); ?><!-- Metadaten & Titel -->Warum?
@@ -15,4 +33,1 @@<?php endif; ?><?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "missing_id"): ?><p class="alert-message is-error">Es ist ein Fehler aufgetreten. Die ID konnte nicht ausgelesen werden. Bitte versuche es erneut.Warum werden die Error-Nachrichten gelöscht?!
@@ -8,2 +25,3 @@}?><!-- Hauptcontainer für die Beitragsansicht (Ausschließlich der Content-Bereich) -->Warum wurde der Kommentar entfernt?
@@ -52,7 +52,7 @@ if ($pfad === "deleteAccount") {<link rel="stylesheet" href="css/showArticle.css"><link rel="stylesheet" href="css/message.css"><script src="js/paginator.js" async></script>Warum löscht du das Skript?!
@@ -0,0 +1,56 @@<?phpsession_start();Bitte vorher prüfen, ob die Session schon gestartet wurde.
@@ -0,0 +47,4 @@}} catch (PDOException $e) {throw new RuntimeException("Kommentardatenbank konnte nicht erstellt werden.");Die genaue Formulierung der Fehlernachrichten wird ja in den views übernommen. Hier könnte man Fehlerkürzel wie
RuntimeException("internal_error") verwenden. In der view wird aus der Exception dann gelesen, was in dieser Stand. Wenn dort dann internal_error stand, dann wird eine bestimmte Fehlernachricht angezeigt.
@@ -0,0 +67,4 @@return $db;} catch (PDOException $e) {throw new RuntimeException("Verbindung zur Kommentardatenbank fehlgeschlagen.");s. oben
@@ -0,0 +135,4 @@return intval($db->lastInsertId());} catch (PDOException $e) {throw new RuntimeException("Kommentar konnte nicht gespeichert werden.");s. oben
@@ -0,0 +175,4 @@return $this->mapRowsToComments($command);} catch (PDOException $e) {throw new RuntimeException("Kommentare konnten nicht geladen werden.");s. oben
@@ -0,0 +215,4 @@return $this->mapRowsToComments($command);} catch (PDOException $e) {throw new RuntimeException("Kommentare konnten nicht geladen werden.");s. oben
Ich finde die Implementierung an sich richtig gut! Sie funktioniert und sieht gut aus.
Auch das DAO-Pattern ist vernünftig umgesetzt.
Aber mir fällt auf, dass du viele Zeilen einfach löscht, obwohl sie nichts mit deiner Funktionalität zu tun haben. Ich schätze mal, dass du den Code einem LLM gibst und die Ausgabe der LLM einfach wieder reinkopierst. Bitte kopiere nur Code-Fragmente und nie vollständige Dateien!!!
Die KI kennt nicht die vollständige Implementation, denkt, dass einige Codezeilen unnötig sind und löscht diese einfach. Wenn du den gesamten Code einfach kopierst, dann fällt dir das nicht mal auf.
Ich kann mich auch irren. Dann musst du mir bitte begründen, warum du die Zeilen oben gelöscht hast.