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

Comments

deviantintegral’s picture

deviantintegral commented
Status: Active » Needs review
StatusFileSize
new 1196812.1-use_drupal_static.patch1.73 KB

Tada!

deekayen’s picture

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.