From 0cbb3d53823c927ca905b84afe33cc0ce109b0cd Mon Sep 17 00:00:00 2001 From: matks Date: Tue, 29 Sep 2020 10:23:28 +0200 Subject: [PATCH] Replace SQL query to fetch modules hooked on stat hook by function --- controllers/admin/AdminStatsTabController.php | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/controllers/admin/AdminStatsTabController.php b/controllers/admin/AdminStatsTabController.php index b1009baa00e02..6369edce01c3f 100644 --- a/controllers/admin/AdminStatsTabController.php +++ b/controllers/admin/AdminStatsTabController.php @@ -175,16 +175,10 @@ public function checkModulesNames($a, $b) protected function getModules() { - $sql = 'SELECT h.`name` AS hook, m.`name` - FROM `' . _DB_PREFIX_ . 'module` m - LEFT JOIN `' . _DB_PREFIX_ . 'hook_module` hm ON hm.`id_module` = m.`id_module` - LEFT JOIN `' . _DB_PREFIX_ . 'hook` h ON hm.`id_hook` = h.`id_hook` - WHERE h.`name` = \'displayAdminStatsModules\' - AND m.`active` = 1 - GROUP BY hm.id_module - ORDER BY hm.`position`'; - - return Db::getInstance()->executeS($sql); + return array_map( + function ($moduleArray) {return ['name' => $moduleArray['module']]; }, + Hook::getHookModuleExecList('displayAdminStatsModules') + ); } public function displayStats()