From c399c9d6bcaec8dd26d2cea50253b977f9439af8 Mon Sep 17 00:00:00 2001
From: tjeujansen <tjansen@emico.nl>
Date: Fri, 25 Oct 2024 14:13:17 +0200
Subject: [PATCH] Use plugin instead of preference

---
 src/Block/Product/ListProduct.php        | 87 ------------------------
 src/Plugin/Block/Product/ListProduct.php | 78 +++++++++++++++++++++
 src/etc/di.xml                           |  5 +-
 3 files changed, 82 insertions(+), 88 deletions(-)
 delete mode 100644 src/Block/Product/ListProduct.php
 create mode 100644 src/Plugin/Block/Product/ListProduct.php

diff --git a/src/Block/Product/ListProduct.php b/src/Block/Product/ListProduct.php
deleted file mode 100644
index 41d018d..0000000
--- a/src/Block/Product/ListProduct.php
+++ /dev/null
@@ -1,87 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Tweakwise\AttributeLandingTweakwise\Block\Product;
-
-use Emico\AttributeLanding\Model\LandingPageRepository;
-use Magento\Catalog\Api\CategoryRepositoryInterface;
-use Magento\Catalog\Block\Product\Context;
-use Magento\Catalog\Helper\Output as OutputHelper;
-use Magento\Catalog\Model\Layer\Resolver;
-use Magento\Framework\App\RequestInterface;
-use Magento\Framework\Data\Helper\PostHelper;
-use Magento\Framework\Registry;
-use Magento\Framework\Stdlib\CookieManagerInterface;
-use Magento\Framework\Url\Helper\Data;
-use Tweakwise\AttributeLandingTweakwise\Model\AjaxResultInitializer\LandingPageInitializer;
-use Tweakwise\Magento2Tweakwise\Block\Product\ListProduct as TweakwiseListProduct;
-use Tweakwise\AttributeLandingTweakwise\Model\Catalog\Layer\Url\RewriteResolver\LandingPageResolver;
-use Tweakwise\Magento2Tweakwise\Helper\Cache;
-use Tweakwise\Magento2Tweakwise\Model\Config;
-
-class ListProduct extends TweakwiseListProduct
-{
-    public function __construct(
-        Context $context,
-        PostHelper $postDataHelper,
-        Resolver $layerResolver,
-        CategoryRepositoryInterface $categoryRepository,
-        Data $urlHelper,
-        private readonly Config $tweakwiseConfig,
-        private readonly CookieManagerInterface $cookieManager,
-        private readonly Cache $cacheHelper,
-        private readonly Registry $registry,
-        private readonly RequestInterface $request,
-        private LandingPageRepository $landingPageRepository,
-        array $data = [],
-        ?OutputHelper $outputHelper = null
-    ) {
-        parent::__construct(
-            $context,
-            $postDataHelper,
-            $layerResolver,
-            $categoryRepository,
-            $urlHelper,
-            $tweakwiseConfig,
-            $cookieManager,
-            $cacheHelper,
-            $registry,
-            $request,
-            $data,
-            $outputHelper
-        );
-    }
-
-    /**
-     * @param string $route
-     * @param array $params
-     * @return string
-     */
-    public function getUrl($route = '', $params = [], $queryParams = [])
-    {
-        if ($this->request->getModuleName() === 'emico_attributelanding') {
-            $landingPageId = (int)$this->request->getParam('id');
-            if (!empty($landingPageId)) {
-                $landingPage    = $this->landingPageRepository->getById($landingPageId);
-                $filters        = $this->registry->registry('alp_filters' . $landingPageId);
-                $filterTemplate = $landingPage->getTweakwiseFilterTemplate();
-                $sortTemplate   = $landingPage->getTweakwiseSortTemplate();
-
-                foreach ($filters as $filter) {
-                    $queryParams[$filter->getFacet()] = $filter->getValue();
-                }
-
-                if (!empty($filterTemplate)) {
-                    $queryParams['tn_ft'] = $filterTemplate;
-                }
-
-                if (!empty($sortTemplate)) {
-                    $queryParams['tn_st'] = $sortTemplate;
-                }
-            }
-        }
-
-        return parent::getUrl($route, $params, $queryParams);
-    }
-}
diff --git a/src/Plugin/Block/Product/ListProduct.php b/src/Plugin/Block/Product/ListProduct.php
new file mode 100644
index 0000000..88b9fa5
--- /dev/null
+++ b/src/Plugin/Block/Product/ListProduct.php
@@ -0,0 +1,78 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tweakwise\AttributeLandingTweakwise\Plugin\Block\Product;
+
+use Emico\AttributeLanding\Api\LandingPageRepositoryInterface;
+use Magento\Framework\App\RequestInterface;
+use Magento\Framework\Exception\LocalizedException;
+use Magento\Framework\Exception\NoSuchEntityException;
+use Magento\Framework\Registry;
+use Tweakwise\Magento2Tweakwise\Block\Product\ListProduct as Subject;
+use Tweakwise\Magento2Tweakwise\Helper\Cache;
+
+class ListProduct
+{
+    /**
+     * @param RequestInterface $request
+     * @param Cache $cacheHelper
+     * @param LandingPageRepositoryInterface $landingPageRepository
+     * @param Registry $registry
+     */
+    public function __construct(
+        private readonly RequestInterface $request,
+        private readonly Cache $cacheHelper,
+        private readonly LandingPageRepositoryInterface $landingPageRepository,
+        private readonly Registry $registry
+    ) {
+    }
+
+    /**
+     * @param Subject $subject
+     * @param string $route
+     * @param array $params
+     * @return array
+     */
+    public function beforeGetUrl(Subject $subject, $route = '', $params = [])
+    {
+        $landingPageId = (int)$this->request->getParam('id');
+        if (
+            !$this->isAttributeLandingRequest() ||
+            !$landingPageId ||
+            !$this->cacheHelper->personalMerchandisingCanBeApplied() ||
+            $route !== 'page_cache/block/esi'
+        ) {
+            return [$route, $params];
+        }
+
+        try {
+            $landingPage = $this->landingPageRepository->getById($landingPageId);
+        } catch (NoSuchEntityException | LocalizedException $e) {
+            return [$route, $params];
+        }
+
+        $filters = $this->registry->registry(sprintf('alp_filters_%s', $landingPageId)) ?? [];
+        $filterTemplate = $landingPage->getTweakwiseFilterTemplate();
+        $sortTemplate = $landingPage->getTweakwiseSortTemplate();
+
+        foreach ($filters as $filter) {
+            $params['_query'][$filter->getFacet()] = $filter->getValue();
+        }
+
+        if (!empty($filterTemplate)) {
+            $params['_query']['tn_ft'] = $filterTemplate;
+        }
+
+        if (!empty($sortTemplate)) {
+            $params['_query']['tn_st'] = $sortTemplate;
+        }
+
+        return [$route, $params];
+    }
+
+    private function isAttributeLandingRequest(): bool
+    {
+        return $this->request->getModuleName() === 'emico_attributelanding';
+    }
+}
diff --git a/src/etc/di.xml b/src/etc/di.xml
index 17b038b..ac7fa49 100644
--- a/src/etc/di.xml
+++ b/src/etc/di.xml
@@ -1,7 +1,6 @@
 <?xml version="1.0" ?>
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
     <preference for="Emico\AttributeLanding\Model\FilterHider\FilterHiderInterface" type="Tweakwise\AttributeLandingTweakwise\Model\FilterHider\TweakwiseFilterHider"/>
-    <preference for="Tweakwise\Magento2Tweakwise\Block\Product\ListProduct" type="Tweakwise\AttributeLandingTweakwise\Block\Product\ListProduct"/>
     <type name="Emico\AttributeLanding\Model\FilterApplier\AggregateFilterApplier">
         <arguments>
             <argument name="appliers" xsi:type="array">
@@ -65,4 +64,8 @@
                 type="Tweakwise\AttributeLandingTweakwise\Plugin\Block\LayeredNavigation\RenderLayered\RendererPlugin"/>
     </type>
 
+    <type name="Tweakwise\Magento2Tweakwise\Block\Product\ListProduct">
+        <plugin name="Tweakwise_AttributeLandingTweakwise_Plugin_Block_Product_ListProduct"
+                type="Tweakwise\AttributeLandingTweakwise\Plugin\Block\Product\ListProduct"/>
+    </type>
 </config>