X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=includes%2Ferrors.inc;h=5deae97bfa29c8887ec878f51b9665271489b242;hb=3b06c6c4718610a408cae2e9b76c3134ca30b84c;hp=89aae72fe29f2e1570b70fb904b39927ddad5d8a;hpb=78fe8cb8f56510ba3f6a1720b207a33b5828e071;p=fa-stable.git diff --git a/includes/errors.inc b/includes/errors.inc index 89aae72f..5deae97b 100644 --- a/includes/errors.inc +++ b/includes/errors.inc @@ -62,7 +62,8 @@ function error_handler($errno, $errstr, $file, $line) { // Please use restrainedly to not risk loss of important messages $excluded_warnings = array( 'html_entity_decode', 'htmlspecialchars', // nevermind encodings, special chars are processed anyway - 'should be compatible with that' // ignore cpdf/frontreport wrapper warnings + 'should be compatible with that', // ignore cpdf/frontreport wrapper warnings + 'mysql extension is deprecated' // ignore strict warning in 5.4 ); foreach($excluded_warnings as $ref) { if (strpos($errstr, $ref) !== false) { @@ -70,13 +71,13 @@ function error_handler($errno, $errstr, $file, $line) { } } - if ($go_debug>1) { - $bt = get_backtrace(true, 1); - } + $bt = $go_debug>1 ? get_backtrace(true, 1) : array(); // error_reporting==0 when messages are set off with @ if ($errno & error_reporting()) { - $messages[] = array($errno, $errstr, $file, $line, @$bt); + // suppress duplicated errors + if (!in_array(array($errno, $errstr, $file, $line, @$bt), $messages)) + $messages[] = array($errno, $errstr, $file, $line, @$bt); } else if($errno&~E_NOTICE) { // log all not displayed messages $user = @$_SESSION["wa_current_user"]->loginname; @@ -117,6 +118,7 @@ function fmt_errors($center=false) { $content = ''; // clean other messages } } + $str = $msg[1]; if (!in_array($msg[0], array(E_USER_NOTICE, E_USER_ERROR, E_USER_WARNING)) && $msg[2] != null) $str .= ' '._('in file').': '.$msg[2].' '._('at line ').$msg[3]; @@ -149,12 +151,16 @@ function error_box() { Helper to avoid sparse log notices. */ function end_flush() { - global $Ajax, $transaction_level; + global $Ajax; if (isset($Ajax)) $Ajax->run(); - // flush all output buffers (works also with exit inside any div levels) - while(ob_get_level()) ob_end_flush(); + + // on some (but not all) php versions zlib extension adds 1 additional level of buffering, + // so flush the last buffer outside the loop to be on safe side + while(ob_get_level() > 1) + ob_end_flush(); + @ob_end_flush(); // if any transaction was aborted unexpectedly rollback changes cancel_transaction(); @@ -188,10 +194,14 @@ function display_db_error($msg, $sql_statement=null, $exit=true) } $str .= "

"; - if($msg) - trigger_error($str, E_USER_ERROR); - else // $msg can be null here only in debug mode, otherwise the error is ignored - trigger_error($str, E_USER_WARNING); + if (!$go_debug) + error_log($str); + else { + if($msg) + trigger_error($str, E_USER_ERROR); + else // $msg can be null here only in debug mode, otherwise the error is ignored + trigger_error($str, E_USER_WARNING); + } if ($exit) exit; }