[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