#24325 closed enhancement (wontfix)
[wp-includes/load.php] LINE 270: ini_set should be verify before call
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Severity: | minor | Version: | |
| Component: | General | Keywords: | |
| Focuses: | Cc: |
Description
Block @2 should be encapsulate in @1 if.
Most of hosting disable ini_set function and this will resolve some users frustration about multiple junk line of logs or error display.
This is my first report, so I apologize if something is wrong.
@1
if (function_exists('ini_set')) {
@2 CODE
}
@2
if ( WP_DEBUG_DISPLAY )
ini_set( 'display_errors', 1 );
elseif ( null !== WP_DEBUG_DISPLAY )
ini_set( 'display_errors', 0 );
if ( WP_DEBUG_LOG ) {
ini_set( 'log_errors', 1 );
ini_set( 'error_log', WP_CONTENT_DIR . '/debug.log' );
}
Change History (3)
Note: See
TracTickets for help on using
tickets.
Looks like most of other instances in core use error suppression:
@ini_set(...).Not sure if that would make sense here though. These are the ones that only run with
WP_DEBUGon, so ifini_set()is disabled, hiding that fact would make debugging harder.