Merge pull request #31 from EmicoEcommerce/fix-return-landingpage-url

Fix: return landingpage url
EmicoEcommerce · Sep 19, 2024 · a63f1b7 · a63f1b7
2 parents 7ce708c + 15b7a61
commit a63f1b7
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 4 deletions.
diff --git a/src/Plugin/Block/LayeredNavigation/RenderLayered/RendererPlugin.php b/src/Plugin/Block/LayeredNavigation/RenderLayered/RendererPlugin.php
@@ -2,6 +2,7 @@
 
 namespace Tweakwise\AttributeLandingTweakwise\Plugin\Block\LayeredNavigation\RenderLayered;
 
+use Emico\AttributeLanding\Model\LandingPageContext;
 use Tweakwise\AttributeLandingTweakwise\Model\FilterManager;
 use Tweakwise\Magento2Tweakwise\Model\Catalog\Layer\Filter\Item;
 use Magento\Framework\View\Element\Template;
@@ -16,13 +17,19 @@ class RendererPlugin
      */
     protected $filterManager;
 
+    /**
+     * @var LandingPageContext
+     */
+    protected $landingPageContext;
+
     /**
      * DefaultRendererPlugin constructor.
      * @param FilterManager $filterManager
      */
-    public function __construct(FilterManager $filterManager)
+    public function __construct(FilterManager $filterManager, LandingPageContext $landingPageContext)
     {
         $this->filterManager = $filterManager;
+        $this->landingPageContext = $landingPageContext;
     }
 
     /**
@@ -36,7 +43,18 @@ public function afterRenderAnchorHtmlTagAttributes(
         string $result,
         Item $filterItem
     ) {
-        if (!$this->filterManager->findLandingPageUrlForFilterItem($filterItem)) {
+
+        $returnToDefaultPage = false;
+
+        $landingPage = $this->landingPageContext->getLandingPage();
+        if ($landingPage) {
+            $landingPageUrl = $landingPage->getUrlPath();
+            if (stripos($result, $landingPageUrl) === false) {
+                $returnToDefaultPage = true;
+            }
+        }
+
+        if (!$this->filterManager->findLandingPageUrlForFilterItem($filterItem) && !$returnToDefaultPage) {
             return $result;
         }
 

diff --git a/src/Plugin/PathSlugStrategyPlugin.php b/src/Plugin/PathSlugStrategyPlugin.php
@@ -196,6 +196,15 @@ public function afterGetCategoryFilterSelectUrl(
      */
     public function afterGetOriginalUrl(PathSlugStrategy $original, string $result, MagentoHttpRequest $request): string
     {
-        return $result;
+        $landingPage = $this->landingPageContext->getLandingPage();
+        if ($landingPage === null) {
+            return $result;
+        }
+
+        if (strpos($result, $landingPage->getUrlPath()) !== false) {
+            return $result;
+        }
+
+        return $landingPage->getUrlPath();
     }
 }
diff --git a/src/Plugin/UrlPlugin.php b/src/Plugin/UrlPlugin.php
@@ -116,7 +116,9 @@ public function aroundGetRemoveFilter(Url $subject, Closure $proceed, Item $filt
         if (preg_match('|' . $landingPage->getUrlRewriteRequestPath() . '(.*)|', $removeUrl, $matches)) {
             $category = $this->getLayer()->getCurrentCategory();
             $categoryUrl = $category->getUrl();
-            $removeUrl = $categoryUrl . $matches[1];
+
+            //ensure there is one slash between category and the rest of the url
+            $removeUrl = rtrim($categoryUrl, '/') . '/' . ltrim($matches[1], '/');
         }
 
         return $removeUrl;