Julian/phpbb_board3-portal_tlw
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[ticket/416] Reduce duplicated code in modules manager

Browse Source 
B3P-416
This commit is contained in:
Marc Alexander
2014-11-27 13:26:01 +01:00
parent c470ebe386
commit 8738847a2d
1 changed files with 18 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
portal/modules/manager.php
View File

@@ -96,6 +96,20 @@ class manager
return $this;
}
/**
* Get module object
*
* @param string $class_name Module class name
* @return null
*/
protected function get_module($class_name)
{
if (($this->module = $this->portal_helper->get_module($class_name)) === false)
{
trigger_error('CLASS_NOT_FOUND', E_USER_ERROR);
}
}
/**
* Reset module settings to default options
*
@@ -110,10 +124,7 @@ class manager
{
$module_data = $this->get_move_module_data($module_id);
if (!($this->module = $this->portal_helper->get_module($module_data['module_classname'])))
{
trigger_error('CLASS_NOT_FOUND', E_USER_ERROR);
}
$this->get_module($module_data['module_classname']);
$sql_ary = array(
'module_name' => $this->module->get_name(),
@@ -294,10 +305,7 @@ class manager
{
$module_data = $this->get_move_module_data($module_id);
if (!($this->module = $this->portal_helper->get_module($module_data['module_classname'])))
{
trigger_error('CLASS_NOT_FOUND', E_USER_ERROR);
}
$this->get_module($module_data['module_classname']);
$move_action = 0;
@@ -389,10 +397,7 @@ class manager
{
$module_data = $this->get_move_module_data($module_id);
if (!($this->module = $this->portal_helper->get_module($module_data['module_classname'])))
{
trigger_error('CLASS_NOT_FOUND', E_USER_ERROR);
}
$this->get_module($module_data['module_classname']);
$move_action = 0;
@@ -492,10 +497,7 @@ class manager
{
$module_classname = $this->request->variable('module_classname', '');
if (!($this->module = $this->portal_helper->get_module($module_classname)))
{
trigger_error('CLASS_NOT_FOUND', E_USER_ERROR);
}
$this->get_module($module_data['module_classname']);
if (confirm_box(true))
{