Download & Extend

Boost Cache Status Block

Project:Boost
Version:6.x-1.03
Component:Caching logic
Category:support request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

I have boost set NOT to cache on errors.

I use the Boost Cache Status block to provide cache details so I can see if anonymous users are being served the cache and also to clear the cache for views where the node has changed.

However, the cache status block always returns the error "There are php errors or drupal messages on this page, preventing boost from caching."

This is triggered if there is either an error or a message AND either the "do not cache on errors" or "do not cache on messages" is on. As "do not cache on errors" is on AND there is always a message (I use Revisioning), I never see cache info and the flush button, even if the cache exists.

Should/can the line in Boost module be:

<?php
           
if ( (BOOST_HALT_ON_ERRORS && error) || (BOOST_HALT_ON_MESSAGES && $drupal_msg != 0) ) {
?>

rather than
<?php
           
if ((BOOST_HALT_ON_ERRORS || BOOST_HALT_ON_MESSAGES) && ($error || $drupal_msg != 0)) {
?>

?

Comments

#1

Status:active» needs review

Thanks, this is a follow up to this issue
#586138: Split "Do not cache if error on page" into drupal msgs and php errors

AttachmentSize
boost-598044.patch 1.58 KB

#2

Status:needs review» fixed

committed

#3

Status:fixed» closed (fixed)

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

nobody click here