- 🇩🇪Germany Anybody Porta Westfalica
No idea yet, but as a first step we could have a failing test for this to show it's failing and ensure it won't fail in the future?
Could someone take the time?I feel the reason might be somewhere around ConditionAccessResolverTrait or RouteMatchInterface in https://git.drupalcode.org/project/theme_switcher/-/blob/8.x-1.x/src/The... but we need to take a deeper look.
Also we should isolate the issue to be sure it's not a side effect of one of the other (currently open) issues!
- 🇳🇱Netherlands Belidan
The solution #5 provided actually helped me solve a bug I've been having with the module. Thank you! Worked like a charm.
- 🇫🇷France is.manu
Hi, i got the same issue but here more explain:
The bug is not on the path "/user/*".
The but is on the "Content Type" and the option: "Content>Node From URL" ("@node.node_route_context:node").
If your are on a system page the context "@node.node_route_context:node" failed (for sure) and trigger the try/catch in
theme_switcher/src/Theme/ThemeSwitcherNegotiator.php on line 100 and the command "return false;" stop everything.That explain that the #5 works as a workarround.
We can replace "return false;" by "continue;" for fix that.
- 🇺🇸United States jsutta United States
I've attached a patch containing @is.manu's workaround from #9. I'm not sure if this is the correct solution, but it worked for me, which is good enough for now.
- Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.2 & MySQL 8last update
11 months ago Waiting for branch to pass - Status changed to Needs review
11 months ago 2:16pm 1 March 2024 - Status changed to Needs work
8 months ago 11:51am 23 May 2024 - 🇩🇪Germany Anybody Porta Westfalica
Could someone please turn this into a MR and add tests?
- 🇪🇸Spain fjgarlin
🐛 The remaining rule are ignored on missing context Fixed fixed this as well I think, so this issue can probably be closed as a duplicate.
- 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
Change made in 🐛 The remaining rule are ignored on missing context Fixed .