[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