4 files changed, 8 insertions, 13 deletions
diff --git a/phpBB/includes/acp/acp_modules.php b/phpBB/includes/acp/acp_modules.php
index 45eba48777..c2407f15b4 100644
--- a/phpBB/includes/acp/acp_modules.php
+++ b/phpBB/includes/acp/acp_modules.php
@@ -209,7 +209,7 @@ class acp_modules
 					list($module_basename, $module_mode) = explode('::', $quick_install);
 
 					// Check if module name and mode exist...
-					$fileinfo = $module_manager->get_module_infos($module_basename, $this->module_class);
+					$fileinfo = $module_manager->get_module_infos($this->module_class, $module_basename);
 					$fileinfo = $fileinfo[$module_basename];
 
 					if (isset($fileinfo['modes'][$module_mode]))
@@ -326,7 +326,7 @@ class acp_modules
 					// Adjust auth row
 					if ($module_data['module_basename'] && $module_data['module_mode'])
 					{
-						$fileinfo = $module_manager->get_module_infos($module_data['module_basename'], $this->module_class);
+						$fileinfo = $module_manager->get_module_infos($this->module_class, $module_data['module_basename']);
 						$module_data['module_auth'] = $fileinfo[$module_data['module_basename']]['modes'][$module_data['module_mode']]['auth'];
 					}
 
@@ -358,7 +358,7 @@ class acp_modules
 				$is_cat = (!$module_data['module_basename']) ? true : false;
 
 				// Get module information
-				$module_infos = $module_manager->get_module_infos('', $this->module_class);
+				$module_infos = $module_manager->get_module_infos($this->module_class);
 
 				// Build name options
 				$s_name_options = $s_mode_options = '';
@@ -545,7 +545,7 @@ class acp_modules
 		$db->sql_freeresult($result);
 
 		// Quick adding module
-		$module_infos = $module_manager->get_module_infos('', $this->module_class);
+		$module_infos = $module_manager->get_module_infos($this->module_class);
 
 		// Build quick options
 		$s_install_options = '';
diff --git a/phpBB/install/install_install.php b/phpBB/install/install_install.php
index ce457aaef5..b06ef0b51d 100644
--- a/phpBB/install/install_install.php
+++ b/phpBB/install/install_install.php
@@ -1640,7 +1640,7 @@ class install_install extends module
 			}
 
 			// Get the modules we want to add... returned sorted by name
-			$module_info = $_module->get_module_infos('', $module_class);
+			$module_info = $_module->get_module_infos($module_class);
 
 			foreach ($module_info as $module_basename => $fileinfo)
 			{
diff --git a/phpBB/phpbb/db/migration/tool/module.php b/phpBB/phpbb/db/migration/tool/module.php
index 3b181872db..69ac71abb7 100644
--- a/phpBB/phpbb/db/migration/tool/module.php
+++ b/phpBB/phpbb/db/migration/tool/module.php
@@ -461,7 +461,7 @@ class module implements \phpbb\db\migration\tool\tool_interface
 	*/
 	protected function get_module_info($class, $basename)
 	{
-		$module = $this->module_manager->get_module_infos($basename, $class, true);
+		$module = $this->module_manager->get_module_infos($class, $basename, true);
 
 		if (empty($module))
 		{
diff --git a/phpBB/phpbb/module/module_manager.php b/phpBB/phpbb/module/module_manager.php
index da3b037b5e..707e297257 100644
--- a/phpBB/phpbb/module/module_manager.php
+++ b/phpBB/phpbb/module/module_manager.php
@@ -102,20 +102,15 @@ class module_manager
 	/**
 	 * Get available module information from module files
 	 *
-	 * @param string	$module				ID of module
 	 * @param string	$module_class		Class of the module (acp, ucp, mcp etc...)
+	 * @param string	$module				ID of module
 	 * @param bool		$use_all_available	Use all available instead of just all
 	 *										enabled extensions
 	 *
 	 * @return array	Array with module information gathered from module info files.
 	 */
-	public function get_module_infos($module = '', $module_class = null, $use_all_available = false)
+	public function get_module_infos($module_class, $module = '', $use_all_available = false)
 	{
-		if ($module_class === null)
-		{
-			throw new module_class_not_defined_exception();
-		}
-
 		$directory = $this->phpbb_root_path . 'includes/' . $module_class . '/info/';
 		$fileinfo = array();