Remove the activity

[friendica.git] / src / Module / Filer / RemoveTag.php

diff --git a/src/Module/Filer/RemoveTag.php b/src/Module/Filer/RemoveTag.php
index 0246f1db36db4c274b7dd0dd34e3bb997cea1ec0..fccc90f9d2f5d03d80b58f9e595015b1d896e1fd 100644 (file)
--- a/src/Module/Filer/RemoveTag.php
+++ b/src/Module/Filer/RemoveTag.php
@@ -1,51 +1,121 @@
 <?php
+/**
+ * @copyright Copyright (C) 2010-2022, the Friendica project
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
+ */
 
 namespace Friendica\Module\Filer;
 
+use Friendica\App;
 use Friendica\BaseModule;
-use Friendica\Model\FileTag;
+use Friendica\Core\L10n;
+use Friendica\Core\Session\Capability\IHandleUserSessions;
+use Friendica\Core\System;
+use Friendica\Database\DBA;
+use Friendica\Model\Post;
+use Friendica\Module\Response;
+use Friendica\Navigation\SystemMessages;
 use Friendica\Network\HTTPException;
-use Friendica\Util\XML;
+use Friendica\Util\Profiler;
+use Psr\Log\LoggerInterface;
 
 /**
  * Remove a tag from a file
  */
 class RemoveTag extends BaseModule
 {
-       public static function content(array $parameters = [])
+       /** @var SystemMessages */
+       private $systemMessages;
+       /** @var IHandleUserSessions */
+       private $userSession;
+
+       public function __construct(SystemMessages $systemMessages, L10n $l10n, App\BaseURL $baseUrl, App\Arguments $args, LoggerInterface $logger, Profiler $profiler, Response $response, IHandleUserSessions $userSession, array $server, array $parameters = [])
+       {
+               parent::__construct($l10n, $baseUrl, $args, $logger, $profiler, $response, $server, $parameters);
+
+               $this->systemMessages = $systemMessages;
+               $this->userSession    = $userSession;
+       }
+
+       protected function post(array $request = [])
+       {
+               System::httpError($this->removeTag($request));
+       }
+
+       protected function content(array $request = []): string
        {
-               if (!local_user()) {
+               if (!$this->userSession->getLocalUserId()) {
                        throw new HTTPException\ForbiddenException();
                }
 
-               $app = self::getApp();
-               $logger = $app->getLogger();
+               $this->removeTag($request, $type, $term);
 
-               $item_id = (($app->argc > 1) ? intval($app->argv[1]) : 0);
+               if ($type == Post\Category::FILE) {
+                       $this->baseUrl->redirect('filed?file=' . rawurlencode($term));
+               }
 
-               $term = XML::unescape(trim($_GET['term'] ?? ''));
-               $cat = XML::unescape(trim($_GET['cat'] ?? ''));
+               return '';
+       }
 
-               $category = (($cat) ? true : false);
+       /**
+        * @param array       $request The $_REQUEST array
+        * @param string|null $type    Output parameter with the computed type
+        * @param string|null $term    Output parameter with the computed term
+        *
+        * @return int The relevant HTTP code
+        *
+        * @throws \Exception
+        */
+       private function removeTag(array $request, string &$type = null, string &$term = null): int
+       {
+               $item_id = $this->parameters['id'] ?? 0;
 
-               if ($category) {
+               $term = trim($request['term'] ?? '');
+               $cat = trim($request['cat'] ?? '');
+
+               if (!empty($cat)) {
+                       $type = Post\Category::CATEGORY;
                        $term = $cat;
+               } else {
+                       $type = Post\Category::FILE;
                }
 
-               $logger->info('Filer - Remove Tag', [
-                       'term'     => $term,
-                       'item'     => $item_id,
-                       'category' => ($category ? 'true' : 'false')
+               $this->logger->info('Filer - Remove Tag', [
+                       'term' => $term,
+                       'item' => $item_id,
+                       'type' => $type
                ]);
 
-               if ($item_id && strlen($term)) {
-                       if (FileTag::unsaveFile(local_user(), $item_id, $term, $category)) {
-                               info('Item removed');
-                       }
-               } else {
-                       info('Item was not deleted');
+               if (!$item_id || !strlen($term)) {
+                       $this->systemMessages->addNotice($this->l10n->t('Item was not deleted'));
+                       return 401;
+               }
+
+               $item = Post::selectFirst(['uri-id'], ['id' => $item_id]);
+               if (!DBA::isResult($item)) {
+                       return 404;
+               }
+
+               if (!Post\Category::deleteFileByURIId($item['uri-id'], $this->userSession->getLocalUserId(), $type, $term)) {
+                       $this->systemMessages->addNotice($this->l10n->t('Item was not removed'));
+                       return 500;
                }
 
-               $app->internalRedirect('network?file=' . rawurlencode($term));
+               return 200;
        }
 }