To duplicate:

1) Open the current theme's .info file
2) Modify the stylesheets[all][] line and change the style.css file read a non-existent file (nostyle.css)
3) Empty the cache on the Performance page
4) Notice the theme does not change
5) Click around through various pages on the site. Some pages will finally cause the new .info file to load (apparently triggered by something in the menu system forcing a cache reload) and the site will go themeless.

This patch places a call system_theme_data in the drupal_flush_all_caches() function to make force the data in the system table which holds the .info file to reload immediately.

CommentFileSizeAuthor
theme_reload.patch346 bytesSteve Dondley
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dvessel’s picture

Status: Needs review » Reviewed & tested by the community

Emptying the cache actually shows the changes on the next page load. This is definitely confusing especially with the devel theming modules clear cache link which requires more than one page refreshes.

Dries’s picture

I've committed this to CVS HEAD. Should probably also go in DRUPAL-6.

webchick’s picture

Version: 6.1 » 6.x-dev

Just confirmed this bug again in 6. Changing version to hopefully get some eyes on it.

JohnAlbin’s picture

I confirm that this problem still exists in DRUPAL-6.

I also tested the patch and it works great, especially with devel's “Empty cache” link. Would mark it RTBC if it wasn’t already.

Nice fix, Steve!

forngren’s picture

Tested with 6.2 and confirmed working.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed the patch to DRUPAL-6 as well.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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