Instead of polluting the global namespace, this is a perfect use case for drupal_static().

CommentFileSizeAuthor
#1 1196812.1-use_drupal_static.patch1.73 KBdeviantintegral
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

deviantintegral’s picture

deviantintegral CreditAttribution: deviantintegral commented
Status: Active » Needs review
FileSize
1196812.1-use_drupal_static.patch1.73 KB

Tada!

deekayen’s picture

deekayen CreditAttribution: deekayen commented
Status: Needs review » Closed (won't fix)

It wasn't a static variable before and making it static now doesn't make sense to me. Removing global declarations isn't the documented intention of the drupal_static function.