libraries_get_path() uses a static variable, should use drupal_static()

CommentFileSizeAuthor
libraries-static.patch582 bytesAaronBauman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community

Now that I think I have understood drupal_static() I'm pretty sure this is correct.
sun's call, though.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented

Yes, good catch. Although it doesn't seem to affect our tests (yet), most statics should be replaced with drupal_static()s in D7.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Reviewed & tested by the community » Fixed

http://drupal.org/cvs?commit=446336
Thanks very much for the patch!

Status: Fixed » Closed (fixed)

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