The page cache is currently broken.

To reproduce:
- Install D7
- Visit admin/settings/performance, turn on the page cache to "Normal", set the cache lifetime to 5 minutes.
- Logout, reload the front page twice as an anonymous user, huge explosion ensues.

There is an IF statement with the following: $user->cache > $cache->created, but the $user variable is not defined. We just need to add the global $user declaration to the top of the function.

CommentFileSizeAuthor
drupal_cache_broken.patch480 bytesquicksketch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

Title: Page cache broken with minimum page lifetime (easy fix) » Page cache broken with minimum page lifetime
Issue tags: +Quick fix

I suppose we have tags for this sort of thing...

catch’s picture

Title: Page cache broken with minimum page lifetime » Page cache broken with minimum page lifetime (easy fix)
Status: Needs review » Reviewed & tested by the community

Looks good. We have tests for the page cache, so assuming this the fatal error bug not helping us again.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Oopsie.

Committed. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

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