It would be nice to see transactions named differently for different node and term types. These transactions make up the bulk of most Drupal sites, and the performance can vary wildly between bundles.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 2952545-add_bundle_transactions-2.patch | 12.79 KB | rbayliss |
Comments
Comment #2
rbayliss commentedThis is an implementation. It also allows for other dynamic transaction names to be created in the future by setting _transaction_name and _transaction_name_callback values on the route. Callbacks are converted to simple strings in the RouteEnhancer.
Comment #3
hosef commentedI have reviewed the patch and tested it on my local, and it looks great.
I think the ability to give custom names to routes will be really useful.
I am also glad there are finally some tests in the module. :)
Comment #5
hosef commented