Overriding Matcher::isAncestor method

[BoShurik]

Hello.

I have a simple menu which contains top level links (e.g. Home, Orders, News etc). Concrete news are not included, as I don't render them and there are many of them.

Before 3.8.0 I extend \Knp\Menu\Matcher\Matcher and override isAncestor method so it can detect child pages by request uri and mark parent items as ancestors.

Now Matcher is final and I can't do that. I tried to decorate Matcher but it uses recursive calls so it's impossible to achieve what I want.

Is it possible to revert this change? Or there is another way to achieve this? (except copy pasting Matcher)

[garak]

The class was made final exactly to avoid the burden of supporting extensions.
It's anyway a small class, and so I don't see big problems in copying/pasting it to your custom implementation.

[stof]

My approach there would be to actually configure the News page as being matched as active for any concrete news (as those are not rendered anyway). But it is true that I don't make any difference between a current item and an ancestor item with children not rendered in my menu styles.

[BoShurik]

@stof Nice, I'll try that