From: Philipp Holzer <admin@philipp.info>
Date: Sat, 4 May 2019 08:30:05 +0000 (+0200)
Subject: Move mod/filerm to src/Module/FilerM
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=a052b098fc9daa7380cc4c660d46187662f8ec65;p=friendica.git

Move mod/filerm to src/Module/FilerM
---

diff --git a/mod/filerm.php b/mod/filerm.php
deleted file mode 100644
index 9013dd62b4..0000000000
--- a/mod/filerm.php
+++ /dev/null
@@ -1,40 +0,0 @@
-<?php
-
-use Friendica\App;
-use Friendica\Core\Logger;
-use Friendica\Model\FileTag;
-use Friendica\Util\XML;
-
-function filerm_content(App $a)
-{
-	if (! local_user())
-	{
-		exit();
-	}
-
-	$term = XML::unescape(trim(defaults($_GET, 'term', '')));
-	$cat = XML::unescape(trim(defaults($_GET, 'cat', '')));
-
-	$category = (($cat) ? true : false);
-
-	if ($category)
-	{
-		$term = $cat;
-	}
-
-	$item_id = (($a->argc > 1) ? intval($a->argv[1]) : 0);
-
-	Logger::log('filerm: tag ' . $term . ' item ' . $item_id  . ' category ' . ($category ? 'true' :  'false'));
-
-	if ($item_id && strlen($term)) {
-		if (FileTag::unsaveFile(local_user(), $item_id, $term, $category)) {
-			info('Item removed');
-		}
-	}
-	else {
-		info('Item was not deleted');
-	}
-
-	$a->internalRedirect('/network?f=&file=' . rawurlencode($term));
-	exit();
-}
diff --git a/src/App/Router.php b/src/App/Router.php
index b94c0bb58d..48d63224ed 100644
--- a/src/App/Router.php
+++ b/src/App/Router.php
@@ -109,7 +109,8 @@ class Router
 			$collector->addRoute(['GET'], '/{guid}/status_message',              Module\Diaspora\Fetch::class);
 			$collector->addRoute(['GET'], '/{guid}/reshare',                     Module\Diaspora\Fetch::class);
 		});
-		$this->routeCollector->addRoute(['GET'],         '/filer[/{id:\d+}]',    Module\Filer::class);
+		$this->routeCollector->addRoute(['GET'],         '/filer[/{id:\d+}]',    Module\Filer\SaveTag::class);
+		$this->routeCollector->addRoute(['GET'],         '/filerm/{id:\d+}',     Module\Filer\RemoveTag::class);
 		$this->routeCollector->addRoute(['GET'],         '/followers/{owner}',   Module\Followers::class);
 		$this->routeCollector->addRoute(['GET'],         '/following/{owner}',   Module\Following::class);
 		$this->routeCollector->addGroup('/group', function (RouteCollector $collector) {
diff --git a/src/Module/Filer.php b/src/Module/Filer.php
deleted file mode 100644
index da59084da0..0000000000
--- a/src/Module/Filer.php
+++ /dev/null
@@ -1,55 +0,0 @@
-<?php
-
-namespace Friendica\Module;
-
-use Friendica\BaseModule;
-use Friendica\Core\L10n;
-use Friendica\Core\PConfig;
-use Friendica\Core\Renderer;
-use Friendica\Model;
-use Friendica\Util\XML;
-
-/**
- * Shows a dialog for adding tags to a file
- */
-class Filer extends BaseModule
-{
-	public static function init()
-	{
-		if (!local_user()) {
-			info(L10n::t('You must be logged in to use this module'));
-			self::getApp()->internalRedirect();
-		}
-	}
-
-	public static function rawContent()
-	{
-		$a = self::getApp();
-		$logger = $a->getLogger();
-
-		$term = XML::unescape(trim(defaults($_GET, 'term', '')));
-		// @TODO: Replace with parameter from router
-		$item_id = (($a->argc > 1) ? intval($a->argv[1]) : 0);
-
-		$logger->info('filer', ['tag' => $term, 'item' => $item_id]);
-
-		if ($item_id && strlen($term)) {
-			// file item
-			Model\FileTag::saveFile(local_user(), $item_id, $term);
-			info(L10n::t('Filetag %s saved to item', $term));
-		}
-
-		// return filer dialog
-		$filetags = PConfig::get(local_user(), 'system', 'filetags');
-		$filetags = Model\FileTag::fileToList($filetags, 'file');
-		$filetags = explode(",", $filetags);
-
-		$tpl = Renderer::getMarkupTemplate("filer_dialog.tpl");
-		echo Renderer::replaceMacros($tpl, [
-			'$field' => ['term', L10n::t("Save to Folder:"), '', '', $filetags, L10n::t('- select -')],
-			'$submit' => L10n::t('Save'),
-		]);
-
-		exit;
-	}
-}
diff --git a/src/Module/Filer/RemoveTag.php b/src/Module/Filer/RemoveTag.php
new file mode 100644
index 0000000000..bddaaf9dc2
--- /dev/null
+++ b/src/Module/Filer/RemoveTag.php
@@ -0,0 +1,51 @@
+<?php
+
+namespace Friendica\Module\Filer;
+
+use Friendica\BaseModule;
+use Friendica\Model\FileTag;
+use Friendica\Network\HTTPException;
+use Friendica\Util\XML;
+
+/**
+ * Remove a tag from a file
+ */
+class RemoveTag extends BaseModule
+{
+	public static function content()
+	{
+		if (!local_user()) {
+			throw new HTTPException\ForbiddenException();
+		}
+
+		$app = self::getApp();
+		$logger = $app->getLogger();
+
+		$item_id = (($app->argc > 1) ? intval($app->argv[1]) : 0);
+
+		$term = XML::unescape(trim(defaults($_GET, 'term', '')));
+		$cat = XML::unescape(trim(defaults($_GET, 'cat', '')));
+
+		$category = (($cat) ? true : false);
+
+		if ($category) {
+			$term = $cat;
+		}
+
+		$logger->info('Filer - Remove Tag', [
+			'term'     => $term,
+			'item'     => $item_id,
+			'category' => ($category ? 'true' : 'false')
+		]);
+
+		if ($item_id && strlen($term)) {
+			if (FileTag::unsaveFile(local_user(), $item_id, $term, $category)) {
+				info('Item removed');
+			}
+		} else {
+			info('Item was not deleted');
+		}
+
+		$app->internalRedirect('/network?f=&file=' . rawurlencode($term));
+	}
+}
diff --git a/src/Module/Filer/SaveTag.php b/src/Module/Filer/SaveTag.php
new file mode 100644
index 0000000000..f5d6115199
--- /dev/null
+++ b/src/Module/Filer/SaveTag.php
@@ -0,0 +1,55 @@
+<?php
+
+namespace Friendica\Module\Filer;
+
+use Friendica\BaseModule;
+use Friendica\Core\L10n;
+use Friendica\Core\PConfig;
+use Friendica\Core\Renderer;
+use Friendica\Model;
+use Friendica\Util\XML;
+
+/**
+ * Shows a dialog for adding tags to a file
+ */
+class SaveTag extends BaseModule
+{
+	public static function init()
+	{
+		if (!local_user()) {
+			info(L10n::t('You must be logged in to use this module'));
+			self::getApp()->internalRedirect();
+		}
+	}
+
+	public static function rawContent()
+	{
+		$a = self::getApp();
+		$logger = $a->getLogger();
+
+		$term = XML::unescape(trim(defaults($_GET, 'term', '')));
+		// @TODO: Replace with parameter from router
+		$item_id = (($a->argc > 1) ? intval($a->argv[1]) : 0);
+
+		$logger->info('filer', ['tag' => $term, 'item' => $item_id]);
+
+		if ($item_id && strlen($term)) {
+			// file item
+			Model\FileTag::saveFile(local_user(), $item_id, $term);
+			info(L10n::t('Filetag %s saved to item', $term));
+		}
+
+		// return filer dialog
+		$filetags = PConfig::get(local_user(), 'system', 'filetags');
+		$filetags = Model\FileTag::fileToList($filetags, 'file');
+		$filetags = explode(",", $filetags);
+
+		$tpl = Renderer::getMarkupTemplate("filer_dialog.tpl");
+		echo Renderer::replaceMacros($tpl, [
+			'$field' => ['term', L10n::t("Save to Folder:"), '', '', $filetags, L10n::t('- select -')],
+			'$submit' => L10n::t('Save'),
+		]);
+
+		exit;
+	}
+}