Merge pull request 'Abgabe03: Profilbearbeitung: Reviewnacharbeitung' (#23) from dev into Abgabe03

Reviewed-on: #23

.idea/dataSources.local.xml

@@ -1,18 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="dataSourceStorageLocal" created-in="IU-261.24374.151">
-    <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>
-      </database-info>
-      <case-sensitivity plain-identifiers="mixed" quoted-identifiers="mixed" />
-      <secret-storage>master_key</secret-storage>
-      <auth-provider>no-auth</auth-provider>
-      <schema-mapping>
-        <introspection-scope>
-          <node kind="schema" qname="@" />
-        </introspection-scope>
-      </schema-mapping>
-    </data-source>
-  </component>
-</project>

.idea/dataSources.xml

@@ -1,12 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="DataSourceManagerImpl" format="xml" multifile-model="true">
-    <data-source source="LOCAL" name="articles" uuid="315cb5c9-2b0f-435b-b602-59823b160908">
-      <driver-ref>sqlite.xerial</driver-ref>
-      <synchronize>true</synchronize>
-      <jdbc-driver>org.sqlite.JDBC</jdbc-driver>
-      <jdbc-url>jdbc:sqlite:$PROJECT_DIR$/db/articles</jdbc-url>
-      <working-dir>$ProjectFileDir$</working-dir>
-    </data-source>
-  </component>
-</project>

.idea/dataSources/315cb5c9-2b0f-435b-b602-59823b160908/storage_v2/_src_/schema/main.uQUzAA.meta

@@ -1,2 +0,0 @@
-#n:main
-!<md> [0, 0, null, null, -2147483648, -2147483648]

.idea/dataSources/data_sources_history.xml

@@ -1,42 +0,0 @@
-<DataSourcesHistory>
-  <DataSourceFromHistory isRemovedFromProject="true">
-    <data-source source="LOCAL" name="articles" uuid="a0abcd0a-1d6f-40e4-88be-f442bcb431ba">
-      <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>
-      </database-info>
-      <case-sensitivity plain-identifiers="mixed" quoted-identifiers="mixed" />
-      <driver-ref>sqlite.xerial</driver-ref>
-      <synchronize>true</synchronize>
-      <jdbc-driver>org.sqlite.JDBC</jdbc-driver>
-      <jdbc-url>jdbc:sqlite:$PROJECT_DIR$/db/articles</jdbc-url>
-      <secret-storage>master_key</secret-storage>
-      <auth-provider>no-auth</auth-provider>
-      <schema-mapping>
-        <introspection-scope>
-          <node kind="schema" qname="@" />
-        </introspection-scope>
-      </schema-mapping>
-      <working-dir>$ProjectFileDir$</working-dir>
-    </data-source>
-  </DataSourceFromHistory>
-  <DataSourceFromHistory isRemovedFromProject="false">
-    <data-source source="LOCAL" 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>
-      </database-info>
-      <case-sensitivity plain-identifiers="mixed" quoted-identifiers="mixed" />
-      <driver-ref>sqlite.xerial</driver-ref>
-      <synchronize>true</synchronize>
-      <jdbc-driver>org.sqlite.JDBC</jdbc-driver>
-      <jdbc-url>jdbc:sqlite:$PROJECT_DIR$/db/articles</jdbc-url>
-      <secret-storage>master_key</secret-storage>
-      <auth-provider>no-auth</auth-provider>
-      <schema-mapping>
-        <introspection-scope>
-          <node kind="schema" qname="@" />
-        </introspection-scope>
-      </schema-mapping>
-      <working-dir>$ProjectFileDir$</working-dir>
-    </data-source>
-  </DataSourceFromHistory>
-</DataSourcesHistory>

README.md

@@ -19,17 +19,13 @@ Jeweils im Format `Anmeldename, Passwort, Mailadresse`:
 - Bitte auf die gesetzten TODO's achten. Wenn Inhalte fehlen, sind sie i.d.R. als TODO kommentiert.
 - Die Navbar hat noch keine onHover-Effekte und erschwert etwas die Bedienung.
 - Suchleiste in der Nav ist in der mobilen Ansicht etwas schmal. Ggf. ein alternatives (kleineres) Logo implementieren.
-- Die Suchseite und Kategorieseite packen momentan alle passenden Beiträge untereinander. Später sollen zunächst 10 
-  Ergebnisse auf einer Seite angezeigt werden.
 
 ## Besonderheiten des Projektes
 - Es wurde ein einfacher Beitrags-Editor erstellt. Mit diesem können Beiträge erstellt oder bearbeitet werden.
   Es handelt es sich um eine einfache Version. Später sollen z.B. Bilder und die Positionierung der Elemente folgen.
 - Es sind drei Dummy-Beiträge für den Nutzer max.mustermann hinterlegt.
-- Beim Verwenden der Navbar-Links bitte Folgendes beachten: nur die Kategorien Physik, Mathe, Informatik führen 
-  exemplarisch auf eine Beispiel-Seite.
+- Beim Verwenden der Navbar-Links bitte Folgendes beachten: nur die Kategorien Physik, Mathe, Informatik führen exemplarisch auf eine Beispiel-Seite.
   Die anderen Links sind erstmal Platzhalter und führen auf eine 404-Seite.
-- Die Such-Seite umfasst eine Suchfunktion (aber noch nicht nach Tags) und Sortierfunktion. Jedoch fehlt noch eine 
-  Filterfunktion (z.B. nur Mathe anzeigen). 
-  Wenn die Filterfunktion implementiert ist, dann können die statischen Seiten "Informatik", "Mathe", "Physik" entfernt 
-  und dynamisch auf der Suchseite nach gefiltertem Fach angezeigt werden. (Vorläufige showCategory.php implementiert).
+- Die Such-Seite umfasst eine Suchfunktion (aber noch nicht nach Tags) und Sortierfunktion. Jedoch fehlt noch eine Filterfunktion (z.B. nur Mathe anzeigen). 
+  Wenn die Filterfunktion implementiert ist, dann können die statischen Seiten "Informatik", "Mathe", "Physik" entfernt und dynamisch auf der Suchseite 
+  nach gefiltertem Fach angezeigt werden. (Vorläufige showCategory.php implementiert)

content/createArticle.php

@@ -53,8 +53,9 @@ if (!isset($_SESSION["user"])) {
         <input type="text" id="title" name="title"
                value="<?php echo htmlspecialchars($_SESSION['old_title'] ?? ''); unset($_SESSION['old_title']); ?>"
                placeholder="Titel hier eingeben" required>
-        <textarea id="content" name="content" placeholder="Schreibe deinen Beitrag..."><?php if (isset($_SESSION['old_content']) && !empty($_SESSION['old_content'])){echo htmlspecialchars($_SESSION['old_content']); unset($_SESSION['old_content']);}elseif (isset($content) && !empty($content)){echo htmlspecialchars($content);}?></textarea>
-
+        <textarea id="content" name="content" placeholder="Schreibe deinen Beitrag...">
+            <?php echo htmlspecialchars($_SESSION['old_content'] ?? ''); unset($_SESSION['old_content']); ?>
+        </textarea>
     </main>
 
     <!-- Seitenleiste -->

content/deleteAccount.php

@@ -1,7 +1,7 @@
 <?php
-require_once "php/model/LocalUserManager.php";
+require_once "php/model/LocalUserDAO.php";
 
-$dao = new LocalUserManager();
+$dao = new LocalUserDAO();
 
 /*
     Deregistrierung

content/updateArticle.php

@@ -58,27 +58,28 @@ include_once 'php/controller/showArticle-controller.php';
                 Sie sind nicht berechtigt, diesen Beitrag zu bearbeiten.
             </p>
         <?php endif; ?>
-        <?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "not_found_article"): ?>
-            <p class="alert-message is-error">
-                Der zu bearbeitende Artikel wurde nicht gefunden. Bitte versuche es erneut.
-            </p>
-        <?php endif; ?>
         <?php
         unset($_SESSION["message"]);
         ?>
         <input type="text" id="title" name="title"
                value="<?php
-               if (isset($_SESSION['old_title']) && !empty($_SESSION['old_title'])){
+               if (isset($title) && !empty($title)){echo htmlspecialchars($title);
+               }elseif (isset($_SESSION['old_title']) && !empty($_SESSION['old_title'])){
                    echo htmlspecialchars($_SESSION['old_title']);
                    unset($_SESSION['old_title']);
-               }elseif (isset($title) && !empty($title)){
-                   echo htmlspecialchars($title);
                }
 
                ?>"
                placeholder="Titel hier eingeben" required>
-        <textarea id="content" name="content" placeholder="Schreibe deinen Beitrag..."><?php if (isset($_SESSION['old_content']) && !empty($_SESSION['old_content'])){echo htmlspecialchars($_SESSION['old_content']); unset($_SESSION['old_content']);}elseif (isset($content) && !empty($content)){echo htmlspecialchars($content);}?></textarea>
-
+        <textarea id="content" name="content" placeholder="Schreibe deinen Beitrag...">
+            <?php
+            if (isset($content) && !empty($content)){echo htmlspecialchars($content);
+            }elseif (isset($_SESSION['old_content']) && !empty($_SESSION['old_content'])){
+                echo htmlspecialchars($_SESSION['old_content']);
+                unset($_SESSION['old_content']);
+            }
+            ?>
+        </textarea>
     </main>
 
     <!-- Seitenleiste -->
@@ -137,11 +138,10 @@ include_once 'php/controller/showArticle-controller.php';
             <label for="tags">Schlagwörter</label>
             <input type="text" id="tags" name="tags"
                    value="<?php
-                   if (isset($_SESSION['old_tags']) && !empty($_SESSION['old_tags'])){
+                   if (isset($tags) && !empty($tags)){echo htmlspecialchars($tags);
+                   } elseif (isset($_SESSION['old_tags']) && !empty($_SESSION['old_tags'])){
                        echo htmlspecialchars($_SESSION['old_tags']);
                        unset($_SESSION['old_tags']);
-                   } elseif (isset($tags) && !empty($tags)){
-                       echo htmlspecialchars($tags);
                    }
                    ?>"
                    placeholder="z.B. Technik, IT (mit Komma trennen)">

db/.htaccess

@@ -1,3 +0,0 @@
-<FilesMatch "\.db">
-    deny from all
-</FilesMatch>

php/controller/createArticle-controller.php

@@ -3,7 +3,7 @@ SESSION_START();
 require_once '../model/LocalArticleManager.php';
 require_once '../model/ArticleManager.php';
 require_once '../validator/article-validator.php';
-require_once "../model/LocalUserManager.php";
+require_once "../model/LocalUserDAO.php";
 
     if ($_SERVER["REQUEST_METHOD"] === "POST") {
         $_SESSION["old_title"]    = $_POST["title"] ?? '';

php/controller/login-controller.php

@@ -1,8 +1,8 @@
 <?php
 
-require_once "php/model/LocalUserManager.php";
+require_once "php/model/LocalUserDAO.php";
 
-$dao = new LocalUserManager();
+$dao = new LocalUserDAO();
 $error = null;
 
 /*

php/controller/profile-controller.php

@@ -1,6 +1,6 @@
 <?php
 
-require_once "php/model/LocalUserManager.php";
+require_once "php/model/LocalUserDAO.php";
 require_once "php/model/Article.php";
 require_once "php/model/ArticleManager.php";
 require_once "php/validator/user-validator.php";
@@ -13,7 +13,7 @@ if (!isset($_SESSION["user"])) {
 }
 
 try {
-    $dao = new LocalUserManager();
+    $dao = new LocalUserDAO();
 
     $user = $dao->findUser($_SESSION["user_email"] ?? "");
 

php/controller/profileArticles-controller.php

@@ -5,10 +5,10 @@
 
 require_once 'php/model/Article.php';
 require_once 'php/model/ArticleManager.php';
-require_once "../model/LocalUserManager.php";
+require_once "../model/LocalUserDAO.php";
 
 try {
-    $dao = new LocalUserManager();
+    $dao = new LocalUserDAO();
     $user = $dao->findUser($_SESSION["user_email"] ?? "");
 
     $author = $user["email"];

php/controller/register-controller.php

@@ -1,6 +1,6 @@
 <?php
 
-require_once "php/model/LocalUserManager.php";
+require_once "php/model/LocalUserDAO.php";
 require_once "php/validator/user-validator.php";
 
 $error = null;
@@ -22,7 +22,7 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
         $error = "Das Passwort muss 8 bis 72 Zeichen lang sein.";
     } else {
         try {
-            $dao = new LocalUserManager();
+            $dao = new LocalUserDAO();
 
             $password = password_hash($plainPassword, PASSWORD_DEFAULT);
 

php/controller/updateArticle-controller.php

@@ -21,7 +21,7 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
 
     if (!isset($_POST["title"]) ||!isset($_POST["content"]) || !isset($_POST["category"])){
         $_SESSION["message"] = "missing_parameters";
-        header("location: ../../index.php?pfad=updateArticle&id=$id");
+        header("location: ../../index.php?pfad=updateArticle");
         exit();
     }else{
         $title = $_POST["title"];
@@ -33,31 +33,31 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
         // -------------------------------- Validierung der Daten: -------------------------
         if (!articleAuthorValidator($author)) {
             $_SESSION["message"] = "author_not_valid";
-            header("location: ../../index.php?pfad=updateArticle&id=$id");
+            header("location: ../../index.php?pfad=updateArticle");
             exit();
         }
 
         if (!articleTitleValidator($title)) {
             $_SESSION["message"] = "invalid_title";
-            header("location: ../../index.php?pfad=updateArticle&id=$id");
+            header("location: ../../index.php?pfad=updateArticle");
             exit();
         }
 
         if (!articleContentValidator($content)) {
             $_SESSION["message"] = "invalid_content";
-            header("location: ../../index.php?pfad=updateArticle&id=$id");
+            header("location: ../../index.php?pfad=updateArticle");
             exit();
         }
 
         if (!articleCategoryValidator($category)) {
             $_SESSION["message"] = "invalid_category";
-            header("location: ../../index.php?pfad=updateArticle&id=$id");
+            header("location: ../../index.php?pfad=updateArticle");
             exit();
         }
 
         if (!articleTagValidator($tags)) {
             $_SESSION["message"] = "invalid_tags";
-            header("location: ../../index.php?pfad=updateArticle&id=$id");
+            header("location: ../../index.php?pfad=updateArticle");
             exit();
         } else {
             $cleanedTags = [];
@@ -83,7 +83,7 @@ if ($_SERVER["REQUEST_METHOD"] === "POST") {
             $articleManager->updateArticle($id ,$article, $author);
         } catch (Exception $e){
             $_SESSION["message"] = $e->getMessage();
-            header("location: ../../index.php?pfad=updateArticle&id=$id");
+            header("location: ../../index.php?pfad=updateArticle");
             exit();
         }
         $_SESSION["message"] = "article_updated";

php/model/ArticleManager.php

@@ -1,6 +1,5 @@
 <?php
 require_once 'LocalArticleManager.php';
-require_once 'DatabaseArticleManager.php';
 require_once 'Article.php';
 
 /**
@@ -8,11 +7,11 @@ require_once 'Article.php';
  *
  * @author Niklas Ortmann
  */
-class ArticleManager
+class ArticleManager extends LocalArticleManager
 {
     public static function getInstance()
     {
-        $articleManager = DatabaseArticleManager::getInstance(); // Hier kann zwischen dem lokalen und datenbankbasiertem ArticleManager gewechselt werden.
+        $articleManager = LocalArticleManager::getInstance(); // TODO: später durch DataBaseArticleManager ersetzen.
 
         // Erstellen von Dummy-Beiträgen:
         if($articleManager->getArticle(1) == null ){

php/model/ArticleManagerDAO.php

@@ -20,7 +20,8 @@ interface ArticleManagerDAO
      * @param $category string Kategorie des Beitrages
      * @param $tags string optionale Schlagworte für eine bessere Suche
      *
-     * @throws InternalServerErrorException
+     * Mögliche Exceptions:
+     * TODO: Exceptions implementieren.
      */
     public function addArticle($title, $content, $author, $category, $tags);
 
@@ -36,33 +37,34 @@ interface ArticleManagerDAO
      * @throws InternalServerErrorException
      * @throws NotFoundException
      * @throws UnauthorizedAccessException
+     * /
      */
     public function updateArticle($id, $article, $author);
 
     /**
-     * Löscht einen Beitrag aus übergebener ID und dem Nutzer, der die Löschung ausführt.
+     * Löscht einen Beitrag aus übergebener ID.
      * @param $id
-     * @param $author
      * @return void
-     * @throws InternalServerErrorException
-     * @throws NotFoundException
-     * @throws UnauthorizedAccessException
+     *
+     * TODO: Exceptions implementieren.
      */
-    public function deleteArticle($id, $author);
+    public function deleteArticle($id);
 
     /**
      * Beitrag aufrufen.
      * $id ID des Beitrags
      *
      * @return Article
-     * @throws InternalServerErrorException
+     * Mögliche Exceptions:
+     * TODO: Exceptions implementieren.
      */
     public function getArticle($id);
 
     /**
      * Alle Beiträge aufrufen.
      *
-     * @throws InternalServerErrorException
+     * Mögliche Exceptions:
+     * TODO: Exceptions implementieren.
      */
     public function getAllArticles();
 
@@ -70,7 +72,7 @@ interface ArticleManagerDAO
      * Gibt alle Beiträge eines Nutzer mit einer gegebenen ID aus.
      * @param $author
      * @return Article[]
-     * @throws InternalServerErrorException
+     * TODO: Exceptions implementieren.
      */
     public function getArticlesByAuthor($author);
 
@@ -87,7 +89,6 @@ interface ArticleManagerDAO
      * Gibt alle Beiträge einer gegebenen Kategorie aus.
      * @param $category
      * @return mixed
-     * @throws InternalServerErrorException
      */
     public function getArticlesByCategory($category);
 

php/model/DatabaseArticleManager.php

@@ -1,313 +0,0 @@
-<?php
-require_once 'ArticleManagerDAO.php';
-require_once 'Article.php';
-/**
- * Klasse: Eine SQLLite3 Lösung des ArticleManagerDAO.
- *
- * @author Niklas Ortmann
- */
-class DatabaseArticleManager implements ArticleManagerDAO {
-
-    private static $instance = null;
-
-    /**
-     * Konstruktor
-     *
-     * Erstellt die Datenbankverbindung und die Tabelle articles.
-     * @throws InternalServerErrorException
-     */
-    public function __construct()
-    {
-        if (!file_exists(__DIR__ . '/../../db/articles.db')) {
-            try {
-
-                $db = $this->getConnection();
-
-                $db->exec("
-                    CREATE TABLE articles (
-                        id INTEGER PRIMARY KEY AUTOINCREMENT,
-                        title TEXT,
-                        content TEXT,
-                        author TEXT,
-                        category TEXT,
-                        tags TEXT,
-                        created TIMESTAMP DEFAULT CURRENT_TIMESTAMP
-                    );");
-                unset($db);
-            } catch (PDOException $e) {
-                throw new InternalServerErrorException($e->getMessage());
-            }
-        }
-    }
-
-    /**
-     * Baut die Verbindung zur Datenbank auf.
-     * @throws InternalServerErrorException
-     */
-    private function getConnection()
-    {
-        try {
-            $user = 'root';
-            $pw = null;
-            $dsn = 'sqlite:' . __DIR__ . '/../../db/articles.db';
-            return new PDO($dsn, $user, $pw);
-        } catch (PDOException $e) {
-            throw new InternalServerErrorException($e->getMessage());
-        }
-    }
-
-    /**
-     * Gibt die DatabaseArticleManager-Instanz zurück.
-     * @return DatabaseArticleManager
-     */
-    public static function getInstance()
-    {
-        if (self::$instance == null) {
-            self::$instance = new DatabaseArticleManager();
-        }
-        return self::$instance;
-    }
-
-    public function addArticle($title, $content, $author, $category, $tags)
-    {
-        try {
-            $db = $this->getConnection();
-
-            $sql = "INSERT INTO articles (title, content, author, category, tags) 
-                    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
-            $success = $command->execute([
-                ":title"    => $title,
-                ":content"  => $content,
-                ":author"   => $author,
-                ":category" => $category,
-                ":tags"     => $tags
-            ]);
-
-            if (!$success) {
-                throw new InternalServerErrorException("internal_error");
-            }
-
-            return intval($db->lastInsertId());
-
-        } catch (PDOException $e) {
-            throw new InternalServerErrorException($e->getMessage());
-        }
-    }
-
-    public function updateArticle($id, $article, $author)
-    {
-        if (empty($article)) {
-            throw new InternalServerErrorException("internal_error");
-        }
-
-        // Berechtigungsprüfung analog zur lokalen Implementierung:
-        if ($article->getAuthor() !== $author) {
-            throw new UnauthorizedAccessException("unauthorized_access");
-        }
-
-        try {
-            $db = $this->getConnection();
-
-            $sql = "UPDATE articles 
-                    SET title = :title, content = :content, author = :author, category = :category, tags = :tags 
-                    WHERE id = :id;";
-
-            $command = $db->prepare($sql);
-            if (!$command) {
-                throw new InternalServerErrorException("internal_error");
-            }
-
-            $success = $command->execute([
-                ":id"       => $id,
-                ":title"    => $article->getTitle(),
-                ":content"  => $article->getContent(),
-                ":author"   => $author,
-                ":category" => $article->getCategory(),
-                ":tags"     => $article->getTags()
-            ]);
-
-            // rowCount() prüft, ob eine Zeile mit dieser ID existierte und geändert werden konnte
-            if (!$success || $command->rowCount() === 0) {
-                // Falls die ID nicht existiert, prüfen wir, ob sie überhaupt da ist
-                if (!$this->getArticle($id)) {
-                    throw new NotFoundException("missing_id");
-                }
-            }
-        } catch (PDOException $e) {
-            throw new InternalServerErrorException("internal_error");
-        }
-    }
-
-    public function deleteArticle($id, $author)
-    {
-        $article = getArticle($id);
-        if (empty($article)) {
-            throw new NotFoundException("not_found_article");
-        }
-
-        // Berechtigungsprüfung:
-        if ($article->getAuthor() !== $author) {
-            throw new UnauthorizedAccessException("unauthorized_access");
-        }
-
-        try {
-            $db = $this->getConnection();
-            $sql = "DELETE FROM articles WHERE id = :id;";
-
-            $command = $db->prepare($sql);
-            if (!$command) {
-                throw new InternalServerErrorException("internal_error");
-            }
-
-            if (!$command->execute([":id" => $id])) {
-                throw new InternalServerErrorException("internal_error");
-            }
-        } catch (PDOException $exc) {
-            throw new InternalServerErrorException("internal_error");
-        }
-    }
-
-    public function getArticle($id)
-    {
-        try {
-            $db = $this->getConnection();
-            $sql = "SELECT * FROM articles WHERE id = :id;";
-
-            $command = $db->prepare($sql);
-            if (!$command) {
-                throw new InternalServerErrorException("internal_error");
-            }
-
-            $command->execute([":id" => $id]);
-            $row = $command->fetch(PDO::FETCH_ASSOC);
-
-            if ($row) {
-                return new Article(
-                    intval($row['id']),
-                    $row['title'],
-                    $row['content'],
-                    $row['author'],
-                    $row['category'],
-                    $row['tags'],
-                    $row['created']
-                );
-            }
-
-            return null;
-        } catch (PDOException $e) {
-            throw new InternalServerErrorException("internal_error");
-        }
-    }
-
-    public function getAllArticles()
-    {
-        try {
-            $db = $this->getConnection();
-            $sql = "SELECT * FROM articles;";
-
-            $command = $db->query($sql);
-            if (!$command) {
-                throw new InternalServerErrorException("internal_error");
-            }
-
-            $rows = $command->fetchAll(PDO::FETCH_ASSOC);
-            $articles = [];
-
-            foreach ($rows as $row) {
-                $articles[] = new Article(
-                    intval($row['id']),
-                    $row['title'],
-                    $row['content'],
-                    $row['author'],
-                    $row['category'],
-                    $row['tags'],
-                    $row['created']
-                );
-            }
-
-            return $articles;
-        } catch (PDOException $e) {
-            throw new InternalServerErrorException("internal_error");
-        }
-    }
-
-    public function getArticlesByAuthor($author)
-    {
-        try {
-            $db = $this->getConnection();
-            $sql = "SELECT * FROM articles WHERE author = :author;";
-
-            $command = $db->prepare($sql);
-            if (!$command) {
-                throw new InternalServerErrorException("internal_error");
-            }
-
-            $command->execute([":author" => $author]);
-            $rows = $command->fetchAll(PDO::FETCH_ASSOC);
-            $filteredArticles = [];
-
-            foreach ($rows as $row) {
-                $filteredArticles[] = new Article(
-                    intval($row['id']),
-                    $row['title'],
-                    $row['content'],
-                    $row['author'],
-                    $row['category'],
-                    $row['tags'],
-                    $row['created']
-                );
-            }
-
-            return $filteredArticles;
-        } catch (PDOException $e) {
-            throw new InternalServerErrorException("internal_error");
-        }
-    }
-
-    public function getArticlesByCategory($category)
-    {
-        try {
-            $db = $this->getConnection();
-            $sql = "SELECT * FROM articles WHERE category = :category;";
-
-            $command = $db->prepare($sql);
-            if (!$command) {
-                throw new InternalServerErrorException("internal_error");
-            }
-
-            $command->execute([":category" => $category]);
-            $rows = $command->fetchAll(PDO::FETCH_ASSOC);
-            $filteredArticles = [];
-
-            foreach ($rows as $row) {
-                $filteredArticles[] = new Article(
-                    intval($row['id']),
-                    $row['title'],
-                    $row['content'],
-                    $row['author'],
-                    $row['category'],
-                    $row['tags'],
-                    $row['created']
-                );
-            }
-
-            return $filteredArticles;
-        } catch (PDOException $exc) {
-            throw new InternalServerErrorException("internal_error");
-        }
-    }
-
-    public function search(string $keyword): array
-    {
-        // TODO: implement search()
-        return [];
-    }
-
-}

php/model/DatabaseUserManager.php

@@ -1,26 +0,0 @@
-<?php
-require_once "UserManagerDAO.php";
-
-class DatabaseUserManager implements UserManagerDAO {
-
-    public function findUser($email)
-    {
-        // TODO: Implement findUser() method.
-    }
-
-    public function addUser($email, $vorname, $nachname, $password)
-    {
-        // TODO: Implement addUser() method.
-    }
-
-    public function updateUser($oldEmail, $newEmail, $vorname, $nachname, $password = null)
-    {
-        // TODO: Implement updateUser() method.
-    }
-
-    public function deleteUser($email)
-    {
-        // TODO: Implement deleteUser() method.
-    }
-
-}

php/model/LocalArticleManager.php

@@ -66,7 +66,7 @@ class LocalArticleManager implements ArticleManagerDAO {
     public function updateArticle($id, $article, $author)
     {
         if (empty($article)) {
-            throw new NotFoundException("not_found_article");
+            throw new InternalServerErrorException("internal_error");
         }
 
         // Berechtigungsprüfung:
@@ -102,18 +102,8 @@ class LocalArticleManager implements ArticleManagerDAO {
         }
     }
 
-    public function deleteArticle($id, $author)
+    public function deleteArticle($id)
     {
-        $article = getArticle($id);
-        if (empty($article)) {
-            throw new NotFoundException("not_found_article");
-        }
-
-        // Berechtigungsprüfung:
-        if ($article->getAuthor() !== $author) {
-            throw new UnauthorizedAccessException("unauthorized_access");
-        }
-
         $articles = $this->getAllArticles();
         $articleFound = false;
 
@@ -208,7 +198,7 @@ class LocalArticleManager implements ArticleManagerDAO {
             }
         }
 
-        return $filteredArticles;
+    return $filteredArticles;
     }
 
     public function getArticlesByCategory($category)

php/model/LocalUserDAO.php

@@ -1,15 +1,11 @@
 <?php
 
-require_once "UserManagerDAO.php";
+require_once "UserDAOInterface.php";
 
-class LocalUserManager implements UserManagerDAO {
+class LocalUserDAO implements UserDAOInterface {
 
     private string $file = "data/users.json";
 
-    public static function getInstance(){
-        // TODO: implement the getIsntance method.
-    }
-
     /**
      * Lädt alle Benutzer aus der JSON-Datei.
      *
@@ -162,8 +158,6 @@ class LocalUserManager implements UserManagerDAO {
     /**
      * Löscht einen Benutzer anhand seiner E-Mail-Adresse.
      *
-     * TODO: wenn ein Benutzer gelöscht wird, sollten dann auch seine Beiträge gelöscht werden?
-     *
      * @param string $email E-Mail-Adresse des zu löschenden Benutzers
      * @return bool true, wenn der Benutzer gelöscht wurde, sonst false
      * @throws RuntimeException wenn die Benutzerdaten nicht gelesen oder gespeichert werden können

php/model/UserDAOInterface.php

@@ -6,7 +6,7 @@
  * Definiert die Methoden, die jede UserDAO-Implementierung
  * bereitstellen muss.
  */
-interface UserManagerDAO {
+interface UserDAOInterface {
 
     /**
      * Sucht einen Benutzer anhand seiner E-Mail-Adresse.

php/model/UserManager.php

@@ -1,10 +0,0 @@
-<?php
-require_once "UserManagerDAO.php";
-
-class UserManager extends LocalUserManager{
-
-    public static function getInstance(){
-        // TODO: implement this.
-        // TODO: dummy-user anlegen: - `max.mustermann, test123, mustermann@web.de` (analog zu ArticleManager)
-    }
-}