[LiveComponent] Fix LiveUrlSubscriber throw MethodNotAllowed

[smnandre]

Q	A
Bug fix?	yes
New feature?	no
Docs?	no
Issues	Fix #2960
License	MIT

Warning

Not sure this is enough, but If anyone wants to build on it, feel free - wont be around tomorrow.

---

The new LiveUrlSubscriber calls the Router to match the previous Url.

At that instant, the RoutingContext is most of the time in POST, and so calling router->match($previousUrl) would throw a MethodNotAllowedException

---

Most recent commit (Catch MethodNotAllowedException) fixes this issue ... but I think we have probably other use cases where it would not work.

My previous commit (reverted in the current state) Force RequestsContext to "GET" seems a more promising lead.

And the first one (reverted too in current state) No LiveProps no change was to avoid entirely the generation when no props were extracted from URL... sadly i don't think it's enough to say "no props should now be used in URL".. ?

---

Thank you @zacharylund for the investigation and @jmsche for the perfect reproducer

[Kocal]

Thanks for the reactivity 🙏🏻

[Kocal]

Btw it fixed https://symfony-devs.slack.com/archives/C3EQ7S3MJ/p1753802642567209 issue

[Amoifr]

Thx for this MR. Waiting about this live. I found a workaround adding method in my attributes
#[AsLiveComponent(name: 'filter_bar_brand_filter', template: 'front/components/common/filter_bar/brand_filter.html.twig', method:'get')]

[mpysiak]

Thanks 🙌 This bug has caused quite a few issues in Sylius. Do you have plans to release a patch soon?

[jmsche]

Hello there, I can confirm the patch fixes the reported issue :)

[Kocal]

We can release this MR today I think.

I wish we could have fixed #2963 before, but I think we will tag a 2nd release later

[GSadee]

Thanks for your quick response and PR 💪🏻

We have tested this workaround in Sylius and can confirm that it solves the problem. We would greatly appreciate a patch as soon as possible 🙏🏻

[Kocal]

Thanks, such confirmations help us to move forward ✨

cc @smnandre @kbond are we fine with merging it and releasing it today?

[davidnectarestudio]

Same problem here

[kbond]

Thank you Simon.

[kbond]

2.28.2 released