deleteArticle -> Autorisierungsprüfung

php/model/ArticleManagerDAO.php

@@ -40,15 +40,15 @@ interface ArticleManagerDAO
     public function updateArticle($id, $article, $author);
 
     /**
-     * Löscht einen Beitrag aus übergebener ID.
+     * Löscht einen Beitrag aus übergebener ID und dem Nutzer, der die Löschung ausführt.
-     * TODO: sollte auch die Autorisierung prüfen...
      * @param $id
+     * @param $author
      * @return void
      * @throws InternalServerErrorException
      * @throws NotFoundException
      * @throws UnauthorizedAccessException
      */
-    public function deleteArticle($id);
+    public function deleteArticle($id, $author);
 
     /**
      * Beitrag aufrufen.

php/model/DatabaseArticleManager.php

@@ -147,9 +147,18 @@ class DatabaseArticleManager implements ArticleManagerDAO {
         }
     }
 
-    public function deleteArticle($id)
+    public function deleteArticle($id, $author)
     {
-        // TODO: Sollte auch die Autorisierung prüfen...
+        $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;";

php/model/LocalArticleManager.php

@@ -102,9 +102,18 @@ class LocalArticleManager implements ArticleManagerDAO {
         }
     }
 
-    public function deleteArticle($id)
+    public function deleteArticle($id, $author)
     {
-        // TODO: Sollte auch die Autorisierung prüfen...
+        $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;