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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pillarsdotnet’s picture

pillarsdotnet CreditAttribution: pillarsdotnet commented

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

gustav’s picture

gustav CreditAttribution: 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 CreditAttribution: Damien Tournoud commented
Version: 6.x-dev » 7.x-dev
Priority: Minor » Normal
FileSize
275801-typo-system_theme_data-d6.patch648 bytes
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 CreditAttribution: 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) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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