fix service availability issue

Author: matthieu-rolland

classes/Hook.php

@@ -1130,7 +1130,7 @@ private static function getHookModuleFilter()
         }
 
         try {
-            $hookModuleFilter = $serviceContainer->get('prestashop.hook.module.filter');
+            $hookModuleFilter = $serviceContainer->get(HookModuleFilter::class);
         } catch (Exception $e) {
             return null;
         }

config/services/common.yml

@@ -26,7 +26,6 @@ services:
         public: false
         tags: ['container.env_var_processor']
 
-    prestashop.hook.module.filter:
-      class: PrestaShop\PrestaShop\Core\Hook\HookModuleFilter
+    PrestaShop\PrestaShop\Core\Hook\HookModuleFilter:
       arguments:
         - !tagged core.hook_module_exec_filter

src/Core/Hook/HookModuleFilter.php

@@ -26,6 +26,18 @@
 
 namespace PrestaShop\PrestaShop\Core\Hook;
 
+/**
+ * This service is responsible for filtering the list of modules for a given hook that is returned by
+ * the getHookModuleExecList method from Hook.php. It is called at the very end of getHookModuleExecList.
+ *
+ * How to use it to filter a list of modules for a hook:
+ *
+ * In your module, create a service which implements the HookModuleFilterInterface and give it
+ * the tag named core.hook_module_exec_filter. Then in your service, you can filter the list of modules
+ * in the filterHookModuleExecList method, according to your own logic.
+ *
+ * Your service will automatically be sent in this class's constructor, and be used to filter the list of modules.
+ */
 class HookModuleFilter implements HookModuleFilterInterface
 {
     private $hookModuleFilters;

src/Core/Hook/HookModuleFilterInterface.php

@@ -26,6 +26,10 @@
 
 namespace PrestaShop\PrestaShop\Core\Hook;
 
+/**
+ * This interface must be implemented by all services that will be used by the HookModuleFilter service.
+ * See HookModuleFilter.php for more explanations.
+ */
 interface HookModuleFilterInterface
 {
     public function filterHookModuleExecList(array $modules, string $hookName): array;

src/PrestaShopBundle/Resources/config/services/core/common.yml

@@ -1,2 +1,7 @@
 imports:
   - { resource: ./circuit_breaker.yml }
+
+services:
+  PrestaShop\PrestaShop\Core\Hook\HookModuleFilter:
+    arguments:
+      - !tagged core.hook_module_exec_filter