add ability to match request for node activation (#46)

README.md

@@ -18,6 +18,10 @@ class MainMenuTreeBuilder implements MenuTreeBuilderInterface
                 ->child('stream')
                     ->setRoute('social_media_stream')
                     ->setRequiredPermissions(['ROLE_SOCIAL_STREAM'])
+                    ->setAdditionalActiveRequestMatcher(static function (Request $request): bool {
+                        // additionally will mark node as active if the request path starts with '/foo_bar'
+                        return str_starts_with($request->getPathInfo(), '/foo_bar');
+                    })
                 ->end()
                 ->child('update_status')
                     ->setRoute('social_media_update_status')

src/Node/Node.php

@@ -3,6 +3,7 @@
 namespace DAMA\MenuBundle\Node;
 
 use Symfony\Component\ExpressionLanguage\Expression;
+use Symfony\Component\HttpFoundation\Request;
 
 class Node
 {
@@ -46,6 +47,11 @@ class Node
      */
     protected $additionalActiveRoutes = [];
 
+    /**
+     * @var callable(Request): bool|null
+     */
+    protected $additionalActiveRequestMatcher;
+
     /**
      * @var array<string|Expression|mixed>
      */
@@ -456,4 +462,24 @@ public static function resetCounter(): void
     {
         self::$counter = 0;
     }
+
+    /**
+     * @return callable(Request): bool|null
+     */
+    public function getAdditionalActiveRequestMatcher(): ?callable
+    {
+        return $this->additionalActiveRequestMatcher;
+    }
+
+    /**
+     * @param callable(Request): bool|null $additionalActiveRequestMatcher
+     *
+     * @return $this
+     */
+    public function setAdditionalActiveRequestMatcher(?callable $additionalActiveRequestMatcher): self
+    {
+        $this->additionalActiveRequestMatcher = $additionalActiveRequestMatcher;
+
+        return $this;
+    }
 }

src/NodeVisitor/NodeActivator.php

@@ -10,10 +10,7 @@
  */
 final class NodeActivator implements NodeVisitorInterface
 {
-    /**
-     * @var RequestStack
-     */
-    private $requestStack;
+    private RequestStack $requestStack;
 
     public function __construct(RequestStack $requestStack)
     {
@@ -28,6 +25,14 @@ public function visit(Node $node): void
 
         if (in_array($request->get('_route'), $node->getAllActiveRoutes())) {
             $node->setActive(true);
+
+            return;
+        }
+
+        if (($callable = $node->getAdditionalActiveRequestMatcher())
+            && $callable($request)
+        ) {
+            $node->setActive(true);
         }
     }
 }

tests/Functional/FunctionalTest.php

@@ -25,6 +25,10 @@ public function testRenderMenuWithUserFoo(): void
 
         $this->assertCount(2, $crawler->filter('a[href="/test_2"]'));
         $this->assertCount(0, $crawler->filter('a[href="/test_1"]'));
+
+        $crawler = $client->request('GET', '/test_3');
+        $this->assertSame(200, $client->getResponse()->getStatusCode());
+        $this->assertCount(2, $crawler->filter('li.menu-item-active > a[href="/test_2"]'));
     }
 
     public function testRenderMenuWithUserBar(): void

tests/Functional/TestMenuTreeBuilder.php

@@ -4,6 +4,7 @@
 
 use DAMA\MenuBundle\MenuTree\MenuTreeBuilderInterface;
 use DAMA\MenuBundle\Node\Node;
+use Symfony\Component\HttpFoundation\Request;
 
 class TestMenuTreeBuilder implements MenuTreeBuilderInterface
 {
@@ -29,6 +30,9 @@ public function build(Node $root): void
                 ->end()
                 ->child('bar-sub-2')
                     ->setRoute('test_2')
+                    ->setAdditionalActiveRequestMatcher(static function (Request $request): bool {
+                        return str_starts_with($request->getPathInfo(), '/test_');
+                    })
                 ->end()
             ->end()
         ;

tests/Functional/routing.yml

@@ -15,3 +15,9 @@ test_2:
     controller: 'Symfony\Bundle\FrameworkBundle\Controller\TemplateController::templateAction'
     defaults:
         template: test_menu.html.twig
+
+test_3:
+    path: /test_3
+    controller: 'Symfony\Bundle\FrameworkBundle\Controller\TemplateController::templateAction'
+    defaults:
+        template: test_menu.html.twig