X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;ds=sidebyside;f=includes%2Ferrors.inc;h=a9b68898498fb0e4f9854e678ad155b86a3bfa57;hb=b3d1d752776bf325ddb7e5955e177b92b59fc94b;hp=4bf5463f669c98ca607542a680d237edba21972f;hpb=6fc84e9d80eed7fa71a19b385801c0fbb15508d3;p=fa-stable.git
diff --git a/includes/errors.inc b/includes/errors.inc
index 4bf5463f..a9b68898 100644
--- a/includes/errors.inc
+++ b/includes/errors.inc
@@ -12,11 +12,6 @@
$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'=>'#ffff00', 'txt'=>'#ff5500', 'bd'=>'#ff9900'),
- E_USER_NOTICE => array('bg'=>'#ccffcc', 'txt'=>'#007700', 'bd'=>'#33cc00'));
-
//-----------------------------------------------------------------------------
// Error handler - collects all php/user messages for
// display in message box.
@@ -24,10 +19,19 @@ $msg_colors = array(
function error_handler($errno, $errstr, $file, $line) {
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);
- else // log all not displayed messages
+ else if($errno&~E_NOTICE)// log all not displayed messages
error_log(user_company() . ':' . $_SESSION["wa_current_user"]->loginname.':'
. basename($file) .":$line: $errstr");
@@ -36,14 +40,19 @@ function error_handler($errno, $errstr, $file, $line) {
//------------------------------------------------------------------------------
// 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) {
@@ -57,23 +66,16 @@ function fmt_errors($center=true) {
}
}
$str = $msg[1];
- $c = $msg_colors[$type];
if ($msg[0] < E_USER_ERROR && $msg[2] != null)
$str .= ' '._('in file').': '.$msg[2].' '._('at line ').$msg[3];
-
- $content .= "
"
- .$str." |
";
- }
-
- $str = "
\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.
@@ -98,9 +100,15 @@ function 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)
{
@@ -114,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;
}
@@ -126,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;
}
@@ -136,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)
{
@@ -152,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;