From 6fa4014e1904d2e0b3032a442bd99fd95abec25c Mon Sep 17 00:00:00 2001 From: sThiebaut Date: Fri, 30 Dec 2011 10:33:02 +0000 Subject: [PATCH] // clean code git-svn-id: http://dev.prestashop.com/svn/v1/branches/1.5.x@11926 b9a71923-0436-4b27-9f14-aed3839534dd --- controllers/admin/AdminModulesController.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/controllers/admin/AdminModulesController.php b/controllers/admin/AdminModulesController.php index e3bdee106..0f5bd2c4e 100644 --- a/controllers/admin/AdminModulesController.php +++ b/controllers/admin/AdminModulesController.php @@ -558,12 +558,15 @@ class AdminModulesControllerCore extends AdminController if (Tools::getValue('controller') != '') $_POST['tab'] = Tools::safeOutput(Tools::getValue('controller')); + // We check if method of module exists if (!method_exists($module, $method)) - throw new PrestashopException('Method can\'t be found'); + throw new PrestashopException('Method of module can\'t be found'); + // Get the return value of current method $echo = $module->{$method}(); - if ($key == 'configure' AND Module::isInstalled($module->name)) + // If the method called is "configure" (getContent method), we show the html code of configure page + if ($key == 'configure' && Module::isInstalled($module->name)) { $backlink = self::$currentIndex.'&token='.$this->token.'&tab_module='.$module->tab.'&module_name='.$module->name; $hooklink = 'index.php?tab=AdminModulesPositions&token='.Tools::getAdminTokenLite('AdminModulesPositions').'&show_modules='.(int)$module->id;