A typo in _system_theme_data() is causing its cache to never be used.

CommentFileSizeAuthor
#3 275801-typo-system_theme_data-d7.patch633 bytesdamien tournoud
#3 275801-typo-system_theme_data-d6.patch648 bytesdamien tournoud
system_module.patch.txt458 bytesgribnif

Comments

pillarsdotnet’s picture

pillarsdotnet commented

Confirmed that this error persists in the newly-released version 6.3 of Drupal.

gustav’s picture

gustav commented
Status: Needs review » Reviewed & tested by the community

This simple typo fix should be committed quickly because it also makes bug http://drupal.org/node/280950 less severe.

damien tournoud’s picture

damien tournoud commented
Version: 6.x-dev » 7.x-dev
Priority: Minor » Normal
StatusFileSize
new 275801-typo-system_theme_data-d6.patch648 bytes
new 275801-typo-system_theme_data-d7.patch633 bytes

Bugs have to be fixed in the development version first, then backported.

Moreover, please learn how to make a proper patch (they are easier to apply): http://drupal.org/patch/create

In the meantime, here are patches for 7.x and 6.x. This is a trivial patch, so keeping the RTBC. Dries, if you don't mind ;)

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

I've committed this patch to CVS HEAD and DRUPAL-6. Thanks guys.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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