Merge pull request #6563 from annando/ap-forum

[friendica.git] / mod / filerm.php

diff --git a/mod/filerm.php b/mod/filerm.php
index c520fec7ac0f48a5adcdf13630f76a1026bd611c..b1f9f8d74b88989518469f238694ad7e8a6bdd10 100644 (file)
--- a/mod/filerm.php
+++ b/mod/filerm.php
@@ -1,21 +1,40 @@
 <?php
 
-function filerm_content(&$a) {
+use Friendica\App;
+use Friendica\Core\Logger;
+use Friendica\Model\FileTag;
+use Friendica\Util\XML;
 
-       if(! local_user()) {
-               killme();
+function filerm_content(App $a)
+{
+       if (! local_user())
+       {
+               exit();
+       }
+
+       $term = XML::unescape(trim($_GET['term']));
+       $cat = XML::unescape(trim(defaults($_GET, 'cat', '')));
+
+       $category = (($cat) ? true : false);
+
+       if ($category)
+       {
+               $term = $cat;
        }
 
-       $term = unxmlify(trim($_GET['term']));
        $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);
+       if ($item_id && strlen($term)) {
+               if (FileTag::unsaveFile(local_user(), $item_id, $term, $category)) {
+                       info('Item removed');
+               }
+       }
+       else {
+               info('Item was not deleted');
+       }
 
-       if(x($_SESSION,'return_url'))
-               goaway($a->get_baseurl() . '/' . $_SESSION['return_url']);
-       
-       killme();
+       $a->internalRedirect('/network?f=&file=' . rawurlencode($term));
+       exit();
 }