Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.#907690: Breadcrumbs don't work for dynamic paths & local tasks #2 left stale menu_links entries in the db. Due to the way breadcrumbs are calculated now, this results in the access callbacks for all menu callbacks being called on every page.
Patch is there for core in #932134: No upgrade path for MENU CALLBACK API change, but this affects live sites that started before that patch went in and updated HEAD past it already, and it may be that patch gets marked duplicate of one to properly remove stale menu_links entries for router paths, so posting it here too.
Also need to check if 4 is a valid value for menu_links created by d7, since that core update could potentially wipe them out (although they'd be recreated next menu_rebuild().
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | callbacks.patch | 899 bytes | catch |
Comments
Comment #1
catchComment #2
catchtype = 4 isn't a problem, doesn't appear in a D7 menu_router table, so the core update will be harmless once it goes in, but we still need this for head2head.
Comment #3
David_Rothstein CreditAttribution: David_Rothstein commentedClosing this since no one is likely to need Drupal 7 HEAD to HEAD upgrade path patches anymore, and to clear out the queue for possible Drupal 8 usage (#2455949: [policy, no patch] Re-activate the head2head project and use it for beta2beta upgrades in the short-term).