Merge pull request #6563 from annando/ap-forum

[friendica.git] / mod / filerm.php

diff --git a/mod/filerm.php b/mod/filerm.php
index 733d67b593fdcaff4c7737955262bb63e724cb30..b1f9f8d74b88989518469f238694ad7e8a6bdd10 100644 (file)
--- a/mod/filerm.php
+++ b/mod/filerm.php
@@ -1,31 +1,40 @@
 <?php
 
 use Friendica\App;
-use Friendica\Core\System;
-
-function filerm_content(App $a) {
-
-       if (! local_user()) {
-               killme();
+use Friendica\Core\Logger;
+use Friendica\Model\FileTag;
+use Friendica\Util\XML;
+
+function filerm_content(App $a)
+{
+       if (! local_user())
+       {
+               exit();
        }
 
-       $term = unxmlify(trim($_GET['term']));
-       $cat = unxmlify(trim($_GET['cat']));
+       $term = XML::unescape(trim($_GET['term']));
+       $cat = XML::unescape(trim(defaults($_GET, 'cat', '')));
 
        $category = (($cat) ? true : false);
-       if ($category) {
+
+       if ($category)
+       {
                $term = $cat;
        }
 
        $item_id = (($a->argc > 1) ? intval($a->argv[1]) : 0);
 
-       logger('filerm: tag ' . $term . ' item ' . $item_id);
+       Logger::log('filerm: tag ' . $term . ' item ' . $item_id  . ' category ' . ($category ? 'true' :  'false'));
 
        if ($item_id && strlen($term)) {
-               file_tag_unsave_file(local_user(),$item_id,$term, $category);
+               if (FileTag::unsaveFile(local_user(), $item_id, $term, $category)) {
+                       info('Item removed');
+               }
+       }
+       else {
+               info('Item was not deleted');
        }
 
-       //$a->internalRedirect('network');
-
-       killme();
+       $a->internalRedirect('/network?f=&file=' . rawurlencode($term));
+       exit();
 }