We needed a back-port of the menu_get_item caching done in Drupal 7 (http://drupal.org/node/643984) for performance reasons. This prevents a relatively expensive query from being called repeatedly when cache is available.

CommentFileSizeAuthor
#1 menu_get_item_caching_drupal6_backport-1.patch1.28 KBcsavio
menu_get_item_caching_drupal6_backport.patch990 bytescsavio

Comments

csavio’s picture

csavio commented
StatusFileSize
new menu_get_item_caching_drupal6_backport-1.patch1.28 KB

I forgot to add a comment and move two things that don't need to be processed if cache is available. This is closer to the Drupal 7 implementation. http://api.drupal.org/api/drupal/includes--menu.inc/function/menu_get_item

rjbrown99’s picture

rjbrown99 commented

Also see this issue: #1234830: Revert cache_menu patch removal, add a $conf setting instead (was: cache_menu: huge table size). It addresses a change in logic that was implemented in D7 to roll this back.

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.