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.

CommentFileSizeAuthor
#2 2952545-add_bundle_transactions-2.patch12.79 KBrbayliss

Comments

rbayliss created an issue. See original summary.

rbayliss’s picture

Status: Active » Needs review
StatusFileSize
new12.79 KB

This 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.

hosef’s picture

Status: Needs review » Reviewed & tested by the community

I 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. :)

  • hosef committed 0b296df on 8.x-1.x authored by rbayliss
    Issue #2952545 by rbayliss, hosef: Add bundle name when setting...
hosef’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.