Fix: The "extid" field wasn't updated

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

diff --git a/src/Module/Filer/RemoveTag.php b/src/Module/Filer/RemoveTag.php
index 5ecaf4c2f9cdda530aa5fe711811bf44f3222427..667bc390c3b5225bcab2049a2ef4103912523ce2 100644 (file)
--- a/src/Module/Filer/RemoveTag.php
+++ b/src/Module/Filer/RemoveTag.php
@@ -1,10 +1,30 @@
 <?php
+/**
+ * @copyright Copyright (C) 2010-2021, 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\BaseModule;
+use Friendica\Database\DBA;
 use Friendica\DI;
-use Friendica\Model\FileTag;
+use Friendica\Model\Post;
 use Friendica\Network\HTTPException;
 use Friendica\Util\XML;
 
@@ -27,26 +47,33 @@ class RemoveTag extends BaseModule
                $term = XML::unescape(trim($_GET['term'] ?? ''));
                $cat = XML::unescape(trim($_GET['cat'] ?? ''));
 
-               $category = (($cat) ? true : false);
-
-               if ($category) {
+               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')
+                       'term' => $term,
+                       'item' => $item_id,
+                       'type' => $type
                ]);
 
                if ($item_id && strlen($term)) {
-                       if (FileTag::unsaveFile(local_user(), $item_id, $term, $category)) {
-                               info('Item removed');
+                       $item = Post::selectFirst(['uri-id'], ['id' => $item_id]);
+                       if (!DBA::isResult($item)) {
+                               return;                         
+                       }
+                       if (!Post\Category::deleteFileByURIId($item['uri-id'], local_user(), $type, $term)) {
+                               notice(DI::l10n()->t('Item was not removed'));
                        }
                } else {
-                       info('Item was not deleted');
+                       notice(DI::l10n()->t('Item was not deleted'));
                }
 
-               $app->internalRedirect('network?file=' . rawurlencode($term));
+               if ($type == Post\Category::FILE) {
+                       DI::baseUrl()->redirect('filed?file=' . rawurlencode($term));
+               }
        }
 }