Reference:
http://api.drupal.org/api/drupal/includes--bootstrap.inc/function/drupal...

Some of our static caching is for internal module functionality, like taxonomy_access_disable_list() and taxonomy_access_affected_nodes(). However, other stuff is just cached to reduce database queries, in:

  • _taxonomy_access_user_roles()
  • _taxonomy_access_get_descendants()
  • taxonomy_access_global_defaults()
  • taxonomy_access_user_list_terms()
  • taxonomy_access_user_create_terms()

The static declarations in these functions should probably be replaced by drupal_static().

CommentFileSizeAuthor
#2 use_drupal_static-1207268-2.patch2.19 KBsammyd56
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

Issue tags: +Novice

This would be a good novice issue.

sammyd56’s picture

Status: Active » Needs review
FileSize
2.19 KB

Does this look OK?

xjm’s picture

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

Anonymous’s picture

Issue summary: View changes

Updated issue summary.