Fix currentAsLink not working when determining state via voters. (#328

)

The renderer was checking the items "current" state using its
`isCurrent()` method instead of the matcher, hence it would only work
when the state was manually set on the item via its `setCurrent()`
method.

src/Knp/Menu/Renderer/ListRenderer.php

@@ -188,7 +188,7 @@ protected function renderItem(ItemInterface $item, array $options): string
      */
     protected function renderLink(ItemInterface $item, array $options = []): string
     {
-        if ($item->getUri() && (!$item->isCurrent() || $options['currentAsLink'])) {
+        if ($item->getUri() && (!$this->matcher->isCurrent($item) || $options['currentAsLink'])) {
             $text = $this->renderLinkElement($item, $options);
         } else {
             $text = $this->renderSpanElement($item, $options);

tests/Knp/Menu/Tests/Renderer/AbstractRendererTest.php

@@ -4,6 +4,7 @@
 
 use Knp\Menu\Matcher\Matcher;
 use Knp\Menu\Matcher\MatcherInterface;
+use Knp\Menu\Matcher\Voter\UriVoter;
 use Knp\Menu\MenuFactory;
 use Knp\Menu\MenuItem;
 use Knp\Menu\Tests\MenuTestCase;
@@ -223,6 +224,18 @@ public function testRenderWithCurrentItemAsLink(): void
         $this->assertEquals($rendered, $this->renderer->render($menu));
     }
 
+    public function testRenderWithCurrentItemAsLinkUsingMatcherWithVoters(): void
+    {
+        $matcher = new Matcher([new UriVoter('/about')]);
+        $this->renderer = $this->createRenderer($matcher);
+
+        $menu = new MenuItem('test', new MenuFactory());
+        $menu->addChild('About', ['uri' => '/about']);
+
+        $rendered = '<ul><li class="current first last"><a href="/about">About</a></li></ul>';
+        $this->assertEquals($rendered, $this->renderer->render($menu));
+    }
+
     public function testRenderWithCurrentItemNotAsLink(): void
     {
         $menu = new MenuItem('test', new MenuFactory());
@@ -233,6 +246,18 @@ public function testRenderWithCurrentItemNotAsLink(): void
         $this->assertEquals($rendered, $this->renderer->render($menu, ['currentAsLink' => false]));
     }
 
+    public function testRenderWithCurrentItemNotAsLinkUsingMatcherWithVoters(): void
+    {
+        $matcher = new Matcher([new UriVoter('/about')]);
+        $this->renderer = $this->createRenderer($matcher);
+
+        $menu = new MenuItem('test', new MenuFactory());
+        $menu->addChild('About', ['uri' => '/about']);
+
+        $rendered = '<ul><li class="current first last"><span>About</span></li></ul>';
+        $this->assertEquals($rendered, $this->renderer->render($menu, ['currentAsLink' => false]));
+    }
+
     public function testRenderSubMenuPortionWithClassAndTitle(): void
     {
         $this->pt2->setChildrenAttribute('class', 'parent2_class')->setChildrenAttribute('title', 'parent2 title');