X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=includes%2Ferrors.inc;h=a9b68898498fb0e4f9854e678ad155b86a3bfa57;hb=b3d1d752776bf325ddb7e5955e177b92b59fc94b;hp=536d23ad8a50008181d48e2a2b98ab3423eadb3d;hpb=96a7ff2ca1ad19cd6c8e064da2184f4e2f4dce26;p=fa-stable.git diff --git a/includes/errors.inc b/includes/errors.inc index 536d23ad..a9b68898 100644 --- a/includes/errors.inc +++ b/includes/errors.inc @@ -1,59 +1,81 @@ . +***********************************************************************/ $messages = array(); // container for system messages $before_box = ''; // temporary container for output html data before error box -$msg_colors = array( - E_USER_ERROR => array('bg'=>'#ffcccc', 'txt'=>'#dd2200', 'bd'=>'#cc3300'), - E_USER_WARNING => array('bg'=>'#ffcccc', 'txt'=>'#dd2200', 'bd'=>'#cc3300'), - E_USER_NOTICE => array('bg'=>'#ccffcc', 'txt'=>'#007700', 'bd'=>'#33cc00')); - //----------------------------------------------------------------------------- // Error handler - collects all php/user messages for // display in message box. function error_handler($errno, $errstr, $file, $line) { - global $messages; + global $messages, $go_debug; + + // skip well known warnings we don't care about. + // Please use restrainedly to not risk loss of important messages + $excluded_warnings = array('html_entity_decode', 'htmlspecialchars'); + foreach($excluded_warnings as $ref) { + if (strpos($errstr, $ref) !== false) { + return true; + } + } // error_reporting==0 when messages are set off with @ if ($errno & error_reporting()) - $messages[] = array($errno, $errstr, $file, $line); - + $messages[] = array($errno, $errstr, $file, $line); + else if($errno&~E_NOTICE)// log all not displayed messages + error_log(user_company() . ':' . $_SESSION["wa_current_user"]->loginname.':' + . basename($file) .":$line: $errstr"); + return true; } //------------------------------------------------------------------------------ // Formats system messages before insert them into message
// FIX center is unused now -function fmt_errors($center=true) { - global $messages, $msg_colors; +function fmt_errors($center=false) { + global $messages, $path_to_root; + $msg_class = array( + E_USER_ERROR => 'err_msg', + E_USER_WARNING =>'warn_msg', + E_USER_NOTICE => 'note_msg'); + $type = E_USER_NOTICE; - + $content = ''; +// $class = 'no_msg'; if (count($messages)) { - $content = ''; - foreach($messages as $msg) { + foreach($messages as $cnt=>$msg) { if ($msg[0]>$type) continue; - if ($msg[0]<$type && $type>E_USER_ERROR) { - $content = ''; // clean notices when we have errors - $type = E_USER_ERROR; // php or user errors + + if ($msg[0]<$type) { + if ($msg[0] == E_USER_WARNING) { + $type = E_USER_WARNING; // user warnings + $content = ''; // clean notices when we have errors + } else { + $type = E_USER_ERROR; // php or user errors + if($type == E_USER_WARNING) + $content = ''; // clean other messages + } } $str = $msg[1]; - $c = $msg_colors[$type]; - if ($msg[0]" - .$str.""; - } - - $str = "
" - . $content . "

\n"; - - } - else - $str = ''; - - return $str; + $content .= ($cnt ? '
' : '').$str; + } + $class = $msg_class[$type]; + $content = "
$content
"; + } else + if ($path_to_root=='.') + return ''; + return $content; } //----------------------------------------------------------------------------- // Error box
element. @@ -62,19 +84,31 @@ function error_box() { global $before_box; echo "
"; - $before_box = ob_get_clean(); // save html content before error box + $before_box = ob_get_clean(); // save html content before error box // Necessary restart instead of get_contents/clean calls due to a bug in php 4.3.2 - register_shutdown_function('ob_end_flush'); + register_shutdown_function('end_flush'); ob_start('output_html'); echo "
"; } +/* + Helper to avoid sparse log notices. +*/ +function end_flush () { + if (ob_get_level()) ob_end_flush(); +} function display_db_error($msg, $sql_statement=null, $exit=true) { global $db, $debug; + + $warning = $msg==null; $db_error = db_error_no(); - $str = "" . _("DATABASE ERROR :") . " $msg
"; +// $str = "" . _("DATABASE ERROR :") . " $msg
"; + if($warning) + $str = "" . _("Debug mode database warning:") . "
"; + else + $str = "" . _("DATABASE ERROR :") . " $msg
"; if ($db_error != 0) { @@ -88,9 +122,10 @@ function display_db_error($msg, $sql_statement=null, $exit=true) } $str .= "

"; - - trigger_error($str, E_USER_ERROR); - + 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; } @@ -100,7 +135,7 @@ function frindly_db_error($db_error) global $db_duplicate_error_code; if ($db_error == $db_duplicate_error_code) - { + { display_error(_("The entered information is a duplicate. Please go back and enter different values.")); return true; } @@ -110,14 +145,15 @@ function frindly_db_error($db_error) function check_db_error($msg, $sql_statement, $exit_if_error=true, $rollback_if_error=true) { - global $db; + global $db, $go_debug; $db_error = db_error_no(); if ($db_error != 0) { - if (!frindly_db_error($db_error)) - display_db_error($msg, $sql_statement, false); + if ($go_debug || !frindly_db_error($db_error)) { + display_db_error($msg, $sql_statement, false); + } if ($rollback_if_error) { @@ -126,8 +162,7 @@ function check_db_error($msg, $sql_statement, $exit_if_error=true, $rollback_if_ if ($exit_if_error) { - echo "

"; - exit; + end_page(); exit; } } return $db_error;