divide by zero error if looking at performance/boost page right after crawler starts [#564990]

    * warning: Division by zero in ./sites/all/modules/boost/boost.admin.inc on line 550.
    * warning: Division by zero in ./sites/all/modules/boost/boost.admin.inc on line 560.

Any ideas?

Comment	File	Size	Author
#1	boost-564990.patch	923 bytes	mikeytown2
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

mikeytown2 CreditAttribution: mikeytown2 commented 1 September 2009 at 04:25

Status:

Active

» Needs review

File	Size
boost-564990.patch	923 bytes

Yeah it's a rare occurrence. The crawler started but nothing has been crawled yet and your looking at the live stats on the performance/boost page. If you don't look, the error shouldn't occur.

Comment #2

austintnacious CreditAttribution: austintnacious commented 1 September 2009 at 04:43

Got it!

All's good now!

After a few (re)settings everything went well.

BTW, I'm not sure if this is relevant but I was wondering if the setting to 'not cache pages with errors or Drupal messages' might have been contributing here.
I had the setting checked and I'm wondering if having an active Drupal message was preventing boost from caching URLs > thus giving the error because I'm watching the counter > thus preventing caching > thus causing the drupal status message > etc.

Seems like it started to work after I un-checked the option and saved the settings but that may have just been coincidence.

I installed your patch anyway and the boost settings page seems to work OK with it.
Don't know if that counts as enough 'review' to set to 'reviewed'.

Anyway, thanks again for an awesome module, you've just made things a whole lot better. . .

Comment #3

mikeytown2 CreditAttribution: mikeytown2 commented 1 September 2009 at 18:21

Title:	Boost works @localhost, @live status report says 'OK' but cron gives WSOD, cache files not written and error given @ Perf config	» divide by zero error if looking at performance/boost page right after crawler starts
Status:	Needs review	» Reviewed & tested by the community

This says it all
http://img387.imageshack.us/i/dividebyzero.jpg/