Merge pull request #2743 from romainruaud/fix_virtual-category-router

Fix #2628 : Only allow the router to match if we are under the subtre…
Smile-SA · Oct 14, 2022 · f8d5b47 · f8d5b47
2 parents 1ab6c15 + 2b598ec
commit f8d5b47
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/module-elasticsuite-virtual-category/Controller/Router.php b/src/module-elasticsuite-virtual-category/Controller/Router.php
@@ -103,10 +103,12 @@ public function match(RequestInterface $request): ?ActionInterface
             ['router' => $this, 'condition' => $condition]
         );
 
-        if ($appliedRoot && $appliedRoot->getId()) {
-            $this->virtualCategoryRoot->setAppliedRootCategory($appliedRoot);
+        if (!$appliedRoot || !$appliedRoot->getId()) {
+            return null;
         }
 
+        $this->virtualCategoryRoot->setAppliedRootCategory($appliedRoot);
+
         $productRewrite = $this->getProductRewrite($identifier);
         if ($productRewrite) {
             $request->setAlias(UrlInterface::REWRITE_REQUEST_PATH_ALIAS, $productRewrite->getRequestPath());