Merge pull request #73 from matks/dont-hook-ps-featured

Avoid force hooking of ps_featuredproducts on displayHome hook

Author: Progi1984

classes/Handler/ModuleHandler.php

@@ -25,8 +25,6 @@
 class ModuleHandler
 {
     /**
-     * isModuleEnabled
-     *
      * @param string $moduleName
      *
      * @return bool
@@ -47,12 +45,27 @@ public function isModuleEnabled($moduleName)
             return false;
         }
 
-        return $module->registerHook('displayHome');
+        return true;
     }
 
     /**
-     * uninstallModule
+     * @param string $moduleName
+     * @param string $hookName
      *
+     * @return bool
+     */
+    public function isModuleEnabledAndHookedOn($moduleName, $hookName)
+    {
+        $module = Module::getInstanceByName($moduleName);
+
+        if (false === $this->isModuleEnabled($moduleName)) {
+            return false;
+        }
+
+        return $module->isRegisteredInHook($hookName);
+    }
+
+    /**
      * @param string $moduleName
      *
      * @return bool

classes/Hook/HookDisplayHome.php

@@ -53,7 +53,7 @@ public function run()
         $gaScripts = '';
 
         // Home featured products
-        if ($moduleHandler->isModuleEnabled('ps_featuredproducts')) {
+        if ($moduleHandler->isModuleEnabledAndHookedOn('ps_featuredproducts', 'displayHome')) {
             $category = new Category($this->context->shop->getCategory(), $this->context->language->id);
             $productWrapper = new ProductWrapper($this->context);
             $homeFeaturedProducts = $productWrapper->wrapProductList(