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.
I'm reviewing a site with an older version of backup_migrate.
#2017027: Incorrect use of l() in a translated string changed a call from l() to url().
This fixes a performance issue during menu_rebuild() with an empty theme registry cache.
menu_rebuild() triggers this code via backup_migrate_menu()
l() is called.
l() triggers a theme registry rebuild.
This can take several seconds on some sites.
During this time, the router isn't written, so other requests get held in a lock_wait().
This would be fine because it was fixed, except there's a new call to l().
Patch coming up.
Comment | File | Size | Author |
---|---|---|---|
#8 | backup_migrate-n2338841-8.patch | 1.45 KB | DamienMcKenna |
|
Comments
Comment #1
catchComment #2
dasha_v CreditAttribution: dasha_v commentedSee new patch attached for the latest version of the module.
Comment #3
couturier CreditAttribution: couturier as a volunteer commentedThis may or may not need to be rewritten based on the 7.x-3.2 release.
Comment #4
DamienMcKennaComment #6
DamienMcKennaRerolled.
Comment #8
DamienMcKennaPEBCAK.
Comment #10
DamienMcKennaCommitted. Thanks.