I am a new user of this module and am initially impressed with its capabilities. With all new modules, I tend to run them for a bit with the highest level of error/warning/notice checking enabled. On one of my websites, I was getting complaints in the watchdog file about headers_list and undefined index in $menu_args. The attached patch fixed the two issues for me.

CommentFileSizeAuthor
#1 fixes_for_php_warnings-1350416-1.patch1.15 KBlars toomre

Comments

lars toomre’s picture

lars toomre commented
StatusFileSize
new fixes_for_php_warnings-1350416-1.patch1.15 KB

Attaching the patch file now that there is an issue number...

lars toomre’s picture

lars toomre commented

Reviewing another of my sites, I observe that there are other PHP warnings that need to be corrected in the D6 boost module. A more complete patch will be forthcoming in the next few days.

bgm’s picture

bgm commented
Status: Active » Fixed

Thanks for the patch. Committed to 6.x-1.x.

lars toomre’s picture

lars toomre commented

@bgm - You are welcome for the fix to those PHP warnings.

The Drupal 6 version of this module currently generates many warnings for me when run under E_STRICT PHP mode. Would you also like patches that address issues like "E_STRICT: Only variables should be passed by reference in 'sites/all/modules/boost/boost.module'". I am slowly modifying my version to reduce such trace log messages due to the boost module.

Status: Fixed » Closed (fixed)

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