Closed (fixed)
Project:
Boost
Version:
6.x-1.x-dev
Component:
Expiration logic
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
9 Jan 2009 at 17:59 UTC
Updated:
6 May 2009 at 07:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
mikeytown2 commentedThese 2 are duplicates
#433534: Invalid argument for foreach() in /var/www/vhosts/azbigmedia.com/httpdocs/sites/all/modules/boost/boost.module on line 649.
#373444: Invalid argument supplied for foreach(): _boost_rmdir_rf()
Wondering if this would be a better way to do it
http://www.php.net/glob
http://www.php.net/foreach
Comment #2
mikeytown2 commentedComment #3
mikeytown2 commentedGoing with the format of the original patch for performance reasons.
Comment #4
mikeytown2 commentedThis needs to go in before I can do an alpha 3 release. This error will be appearing a lot more now that symlinks are no longer being used. Speaking of that... Note: The site owner needs to ftp in and clear the cache dir, since it should be free of symlinks when upgrading to this newer boost version.
Comment #5
rsvelko commentedin the release notes as an update notice we should write that cache must be cleared manually - would you do that mikey ?
Comment #6
rsvelko commentedreviewed the old and the new functions letter by letter - all ok .
I tried to test this - to make the error happen before the patch and ensure that it is gone after that - but somehow did not succeed in that.
Nevermind - the patch is trivial and keeps us safe - so RTBC. no risk here.
the old logic is preserved - we only wrapped the foreach in an if .
Comment #7
mikeytown2 commentedpatch applied
Comment #8
mikeytown2 commented