From be6057b056e2d225be854235c8bd789549852282 Mon Sep 17 00:00:00 2001
From: Philipp Holzer <admin@philipp.info>
Date: Fri, 1 Mar 2019 11:57:16 +0100
Subject: [PATCH] refactoring

---
 src/Module/Filer.php | 28 +++++++++++++---------------
 1 file changed, 13 insertions(+), 15 deletions(-)

diff --git a/src/Module/Filer.php b/src/Module/Filer.php
index 55015652db..3b7d5175c7 100644
--- a/src/Module/Filer.php
+++ b/src/Module/Filer.php
@@ -10,7 +10,7 @@ use Friendica\Model;
 use Friendica\Util\XML;
 
 /**
- * Shows the App menu
+ * Shows a dialog for adding tags to a file
  */
 class Filer extends BaseModule
 {
@@ -35,20 +35,18 @@ class Filer extends BaseModule
 		if ($item_id && strlen($term)) {
 			// file item
 			Model\FileTag::saveFile(local_user(), $item_id, $term);
-			$a->internalRedirect();
-			return;
-
-		} else {
-			// 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");
-			return Renderer::replaceMacros($tpl, [
-				'$field' => ['term', L10n::t("Save to Folder:"), '', '', $filetags, L10n::t('- select -')],
-				'$submit' => L10n::t('Save'),
-			]);
+			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");
+		return Renderer::replaceMacros($tpl, [
+			'$field' => ['term', L10n::t("Save to Folder:"), '', '', $filetags, L10n::t('- select -')],
+			'$submit' => L10n::t('Save'),
+		]);
 	}
 }