- Issue created by @alexpott
- First commit to issue fork.
- Merge request !12927Issue #3539739: Add route name to outbound path processing → (Open) created by riyas_nr
Added route_name to the
$optionsarray inUrlGenerator::generateFromRoute()to support route-specific logic in outbound path processors. Moving to NR.- 🇬🇧United Kingdom alexpott 🇪🇺🌍
I found a way around this in the contrib project but I still think this would be good to do given we're adding the route and we have the info.
It'd be good to have some test coverage somewhere I think you might be able to add a test processor in \Drupal\Tests\Core\Routing\UrlGeneratorTest to do this.
- 🇬🇧United Kingdom catch
I think this is a duplicate of 🐛 Outbound path processors miss the route name Needs work , we should at least close that once this is in and add credit etc. - short for time today so did not check approaches between the two issues or anything.
- 🇬🇧United Kingdom alexpott 🇪🇺🌍
This is totally a duplicate. That one is includes route parameters which this one misses. It'd be great to add the test from this one to that MR and then mark this as a duplicate.
@riyas_nr can you do that? Thanks!
Thanks @alexpott, I’ve added the test coverage to the other MR. This can be closed as a duplicate.