[wp-trac] [WordPress Trac] #13185: advanced-cache.php is @included
WordPress Trac
wp-trac at lists.automattic.com
Sat May 1 22:20:35 UTC 2010
#13185: advanced-cache.php is @included
--------------------------+-------------------------------------------------
Reporter: solarissmoke | Owner: ryan
Type: defect (bug) | Status: new
Priority: low | Milestone: 3.0
Component: Cache | Version: 3.0
Severity: minor | Keywords: has-patch
--------------------------+-------------------------------------------------
Comment(by nacin):
I don't like the idea of adding a file_exists check here. We do it in
db.php and object-cache.php, but advanced-cache.php is included before
both of them and may very well serve a static file and die, not loading
any more of WP. It needs to be fast.
I'm sitting next to jjj and sivel, and we came up with this: Checking
WP_DEBUG, and if true, don't suppress, otherwise, we do suppress errors.
And no file_exists check. Thoughts?
--
Ticket URL: <http://core.trac.wordpress.org/ticket/13185#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list