- 🇱🇹Lithuania Evaldas Užkuras Kaunas, LT
Adding patch to change priority for routes alter to run at the very end of routes build
- Status changed to Needs review
over 1 year ago 6:57am 21 February 2023
The new version does support some more advanced callbacks and does set the route name as a parameter on the route, but that does mean that some routes are missing that information then.
For example, we're seeing that problem with all routes defined by page_manager, because those are defined after \Drupal\new_relic_rpm\EventSubscriber\RoutingTransactionNameSubscriber::addTransactionNamesToRoutes() runs.
Add back a fallback to \Drupal\new_relic_rpm\RouteEnhancer\TransactionNameEnhancer(), if no _transaction_name is defined, set it then?
Alternatively, the event priority could also be adjusted to run very late and catch all routes except those that somehow set an even lower priority.
Needs review
2.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Adding patch to change priority for routes alter to run at the very end of routes build