[wp-trac] [WordPress Trac] #11507: Usefull errormessage is being surpressed on cron include

WordPress Trac wp-trac at lists.automattic.com
Sat Dec 19 18:34:54 UTC 2009


#11507: Usefull errormessage is being surpressed on cron include
--------------------------+-------------------------------------------------
 Reporter:  hakre         |       Owner:       
     Type:  defect (bug)  |      Status:  new  
 Priority:  normal        |   Milestone:  2.9.1
Component:  General       |     Version:  2.9  
 Severity:  normal        |    Keywords:       
--------------------------+-------------------------------------------------
Changes (by hakre):

  * milestone:  3.0 => 2.9.1


Comment:

 Replying to [comment:1 westi]:
 > I don't think this is a severe enough issue for the 2.9.1 milestone.
 Same problem here: I do not need this in 2.9.1 but there is no 2.9.2
 milestone there fore I prefer 2.9.1 over 3.0 because it's fitting better.

 > This code is probably partly this way so that it can easily be shared
 with BackPress where the file might not always be there.
 I'm talking about WordPress. When other projects share code they need to
 take care on their own. Especially on probabilities.

 > It is probably better to check for the file before including rather than
 using {{{@}}} that way we could give some feedback on the fact that cron
 isn't going to work.

 That's soo much reinventing the wheel as it has been proven plainly wrong
 in other projects. Include_Once does check for file existance. It does
 even provide a message if something goes wrong. The fix is to just remove
 the @. That simple it is.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11507#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list