MAGE-1326: refactor assets observer and introduce RenderingManager class

Author: damcou

Observer/AddAlgoliaAssetsObserver.php

@@ -4,15 +4,13 @@
 
 use Algolia\AlgoliaSearch\Exceptions\AlgoliaException;
 use Algolia\AlgoliaSearch\Helper\ConfigHelper;
-use Algolia\AlgoliaSearch\Registry\CurrentCategory;
 use Algolia\AlgoliaSearch\Service\AlgoliaCredentialsManager;
-use Magento\Catalog\Model\Category;
+use Algolia\AlgoliaSearch\Service\RenderingManager;
 use Magento\Framework\App\Request\Http;
 use Magento\Framework\Event\Observer;
 use Magento\Framework\Event\ObserverInterface;
 use Magento\Framework\Exception\NoSuchEntityException;
 use Magento\Framework\View\Layout;
-use Magento\Framework\View\Page\Config as PageConfig;
 use Magento\Store\Model\StoreManagerInterface;
 
 /**
@@ -22,13 +20,11 @@ class AddAlgoliaAssetsObserver implements ObserverInterface
 {
     public function __construct(
         protected ConfigHelper $config,
-        protected CurrentCategory $category,
+        protected RenderingManager $renderingManager,
         protected StoreManagerInterface $storeManager,
-        protected PageConfig $pageConfig,
         protected Http $request,
         protected AlgoliaCredentialsManager $algoliaCredentialsManager
-    )
-    {}
+    ) {}
 
     /**
      * @throws NoSuchEntityException|AlgoliaException
@@ -39,42 +35,16 @@ public function execute(Observer $observer): void
         if ($actionName === 'swagger_index_index') {
             return;
         }
-        $storeId = $this->storeManager->getStore()->getId();
-        if ($this->config->isEnabledFrontEnd($storeId)) {
-            if ($this->algoliaCredentialsManager->checkCredentials($storeId)) {
-                if ($this->config->isAutoCompleteEnabled($storeId) || $this->config->isInstantEnabled($storeId)) {
-                    /** @var Layout $layout */
-                    $layout = $observer->getData('layout');
-                    $layout->getUpdate()->addHandle('algolia_search_handle');
-
-                    $this->loadPreventBackendRenderingHandle($layout, $storeId);
-                }
-            }
-        }
-    }
-
-    private function loadPreventBackendRenderingHandle(Layout $layout, int $storeId): void
-    {
-        if (!$this->config->preventBackendRendering($storeId)) {
-            return;
-        }
-
-        $category = $this->category->get();
 
-        if (!$category->getId()) {
-            return;
-        }
+        $storeId = $this->storeManager->getStore()->getId();
 
-        if (!$this->config->replaceCategories($storeId)) {
-            return;
-        }
+        if ($this->config->isEnabledFrontEnd($storeId) && $this->algoliaCredentialsManager->checkCredentials($storeId)) {
+            /** @var Layout $layout */
+            $layout = $observer->getData('layout');
 
-        $displayMode = $this->config->getBackendRenderingDisplayMode($storeId);
-        if ($displayMode === 'only_products'
-            && $category->getData('display_mode') === \Magento\Catalog\Model\Category::DM_PAGE) {
-            return;
+            $this->renderingManager->handleFrontendAssets($layout, $storeId);
+            $this->renderingManager->handleBackendRendering($layout, $actionName, $storeId);
         }
-
-        $layout->getUpdate()->addHandle('algolia_search_handle_prevent_backend_rendering');
     }
+
 }

Service/RenderingManager.php

@@ -0,0 +1,87 @@
+<?php
+
+namespace Algolia\AlgoliaSearch\Service;
+
+use Algolia\AlgoliaSearch\Helper\ConfigHelper;
+use Algolia\AlgoliaSearch\Helper\Configuration\AutocompleteHelper;
+use Algolia\AlgoliaSearch\Helper\Configuration\InstantSearchHelper;
+use Algolia\AlgoliaSearch\Registry\CurrentCategory;
+use Magento\Catalog\Model\Category;
+use Magento\Framework\View\Layout;
+use Magento\Store\Model\StoreManagerInterface;
+
+class RenderingManager
+{
+    public function __construct(
+        protected ConfigHelper $config,
+        protected AutocompleteHelper $autocompleteConfigHelper,
+        protected InstantSearchHelper $instantSearchConfigHelper,
+        protected CurrentCategory $category,
+        protected StoreManagerInterface $storeManager
+    ) {}
+
+    /**
+     * @param Layout $layout
+     * @param int $storeId
+     * @return void
+     */
+    public function handleFrontendAssets(Layout $layout, int $storeId): void
+    {
+        // If an Algolia frontend feature is enabled, add the frontend assets
+        if (!$this->hasAlgoliaFrontend($storeId)) {
+           return;
+        }
+
+        $layout->getUpdate()->addHandle('algolia_search_handle');
+    }
+
+    /**
+     * @param Layout $layout
+     * @param string $actionName
+     * @param int $storeId
+     * @return void
+     */
+    public function handleBackendRendering(Layout $layout, string $actionName, int $storeId): void
+    {
+        // If the page is not a category or the catalogsearch page or if no Algolia frontend feature is enabled, no need to go further
+        if (!$this->isSearchPage($actionName) || !$this->hasAlgoliaFrontend($storeId)) {
+            return;
+        }
+
+        // @todo replace this check with the new backend rendering feature (MAGE-1325)
+        if (!$this->config->preventBackendRendering($storeId)) {
+            return;
+        }
+
+        $category = $this->category->get();
+        // Legacy check regarding category display mode (we don't want to hide the static blocks if there's not product list
+        if ($category->getId() && $this->instantSearchConfigHelper->shouldReplaceCategories($storeId)) {
+            $displayMode = $this->config->getBackendRenderingDisplayMode($storeId);
+
+            if ($displayMode === 'only_products' && $category->getData('display_mode') === Category::DM_PAGE) {
+                return;
+            }
+        }
+
+        $layout->getUpdate()->addHandle('algolia_search_handle_prevent_backend_rendering');
+    }
+
+    /**
+     * @param int $storeId
+     * @return bool
+     */
+    protected function hasAlgoliaFrontend(int $storeId): bool
+    {
+        return $this->autocompleteConfigHelper->isEnabled($storeId) ||
+            $this->instantSearchConfigHelper->isEnabled($storeId);
+    }
+
+    /**
+     * @param string $actionName
+     * @return bool
+     */
+    protected function isSearchPage(string $actionName): bool
+    {
+        return $actionName === 'catalog_category_view' || $actionName === 'catalogsearch_result_index';
+    }
+}