Roundup of issues reported by PHPStorm

Mostly redundant checks, logic errors, dead code, etc.

1 files changed, 6 insertions, 12 deletions

diff --git a/modules-available/translation/page.inc.php b/modules-available/translation/page.inc.php
index ee1d2cde..da4205d2 100644
--- a/modules-available/translation/page.inc.php
+++ b/modules-available/translation/page.inc.php
@@ -477,9 +477,8 @@ class Page_Translation extends Page
 				$tags[$p[1]] = $tag;
 			}
 		}
-		$tags = $this->loadTagsFromPhp('/Message\s*::\s*add\w+\s*\(\s*[\'"](?<tag>[^\'"\.]*)[\'"]\s*(?<data>\)|\,.*)/i',
+		return $this->loadTagsFromPhp('/Message\s*::\s*add\w+\s*\(\s*[\'"](?<tag>[^\'"\.]*)[\'"]\s*(?<data>\)|\,.*)/i',
 			$this->getModulePhpFiles($module), $tags);
-		return $tags;
 	}
 
 	/**
@@ -510,11 +509,9 @@ class Page_Translation extends Page
 		return $tags;
 	}
 
-	private function loadUsedMenuCategories($module = false)
+	private function loadUsedMenuCategories()
 	{
-		if ($module === false) {
-			$module = $this->module;
-		}
+		$module = $this->module;
 		$skip = strlen($module->getIdentifier()) + 1;
 		$match = $module->getIdentifier() . '.';
 		$want = array();
@@ -740,15 +737,13 @@ class Page_Translation extends Page
 	private function loadModuleEditArray()
 	{
 		$tags = $this->loadUsedModuleTags();
-		$table = $this->buildTranslationTable('module', array_keys($tags), true);
-		return $table;
+		return $this->buildTranslationTable('module', array_keys($tags), true);
 	}
 
 	private function loadMenuCategoryEditArray()
 	{
 		$tags = $this->loadUsedMenuCategories();
-		$table = $this->buildTranslationTable('categories', array_keys($tags), true);
-		return $table;
+		return $this->buildTranslationTable('categories', array_keys($tags), true);
 	}
 	
 	/**
@@ -760,8 +755,7 @@ class Page_Translation extends Page
 	private function loadCustomEditArray()
 	{
 		$tags = $this->loadUsedCustomTags($this->subsection);
-		$table = $this->buildTranslationTable($this->subsection, array_keys($tags), true);
-		return $table;
+		return $this->buildTranslationTable($this->subsection, array_keys($tags), true);
 	}
 	
 	/**