X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=blobdiff_plain;f=class%2Ferror.class.php;h=cef939bdad5f3b0875d93753ab16387822c8d51d;hp=44b7b6dfb72da3661aae8dd1b6adb63dd4d099f7;hb=47ccfad452e8d345542d09e59112cac317cffed8;hpb=1455d7ecf10b340d84a04b189f2d90abf883ef30

diff --git a/class/error.class.php b/class/error.class.php
index 44b7b6df..cef939bd 100644
--- a/class/error.class.php
+++ b/class/error.class.php
@@ -1,19 +1,26 @@
 <?php
+
 /**
  * error.class.php
  *
- * Copyright (c) 2005 The SquirrelMail Project Team
- * Licensed under the GNU GPL. For full terms see the file COPYING.
- *
  * This contains the custom error handler for SquirrelMail.
  *
+ * @copyright &copy; 2005-2006 The SquirrelMail Project Team
+ * @license http://opensource.org/licenses/gpl-license.php GNU Public License
  * @version $Id$
  * @package squirrelmail
  */
 
+/** Used defines */
 define('SQM_NOTICE',0);
 define('SQM_WARNING',1);
 define('SQM_ERROR',2);
+define('SQM_STRICT',3);
+
+// php5 E_STRICT constant (compatibility with php4)
+if (! defined('E_STRICT')) define('E_STRICT',2048);
+// Set docref_root (fixes URLs that link to php manual)
+if (ini_get('docref_root')=='') ini_set('docref_root','http://www.php.net/');
 
 /**
  * Error Handler class
@@ -62,11 +69,31 @@ class ErrorHandler {
                   );
         $iType = NULL;
         $aErrorCategory = array();
-        /*
-         * The following errors cannot be handled by a user defined error handler:
-         * E_ERROR, E_PARSE, E_CORE_ERROR, E_CORE_WARNING, E_COMPILE_ERROR, E_COMPILE_WARNING
+
+        /**
+         * Get current error reporting level.
+         *
+         * PHP 4.1.2 does not return current error reporting level in ini_get (php 5.1b3 and
+         * 4.3.10 does). Retrieve current error reporting level while setting error reporting
+         * to ini value and reset it to retrieved value.
          */
-        switch ($iErrNo) {
+        $iCurErrLevel = error_reporting(ini_get('error_reporting'));
+        error_reporting($iCurErrLevel);
+
+        /**
+         * Check error_reporting value before logging error.
+         * Don't log errors that are disabled by @ (error_reporting = 0). Some SquirrelMail scripts
+         * (sq_mb_list_encodings(), ldap function calls in functions/abook_ldap_server.php)
+         * handle errors themselves and @ is used to disable generic php error messages.
+         */
+        if ($iErrNo & $iCurErrLevel) {
+            /*
+             * The following errors cannot be handled by a user defined error handler:
+             * E_ERROR, E_PARSE, E_CORE_ERROR, E_CORE_WARNING, E_COMPILE_ERROR, E_COMPILE_WARNING
+             */
+            switch ($iErrNo) {
+            case E_STRICT:
+                $iType = (is_null($iType)) ? SQM_STRICT : $iType;
             case E_NOTICE:
                 $iType = (is_null($iType)) ? SQM_NOTICE : $iType;
             case E_WARNING:
@@ -74,8 +101,8 @@ class ErrorHandler {
                 $aErrorCategory[] = 'PHP';
                 $aError['message'] = $sErrStr;
                 $aError['extra'] = array(
-                                          'FILE' => $sErrFile,
-                                          'LINE' => $iErrLine) ;;
+                                         'FILE' => $sErrFile,
+                                         'LINE' => $iErrLine) ;;
                 // what todo with $aContext?
                 break;
             case E_USER_ERROR:
@@ -93,29 +120,29 @@ class ErrorHandler {
                         $aError = array_merge($aError,$aErrorTemp);
                         // special error handling below
                         if ($aError['category'] & SQM_ERROR_IMAP) {
-                             $aErrorCategory[] = 'IMAP';
-                             // imap related error handling inside
+                            $aErrorCategory[] = 'IMAP';
+                            // imap related error handling inside
                         }
                         if ($aError['category'] & SQM_ERROR_FS) {
-                             $aErrorCategory[] = 'FILESYSTEM';
-                             // filesystem related error handling inside
+                            $aErrorCategory[] = 'FILESYSTEM';
+                            // filesystem related error handling inside
                         }
                         if ($aError['category'] & SQM_ERROR_SMTP) {
-                             $aErrorCategory[] = 'SMTP';
-                             // smtp related error handling inside
+                            $aErrorCategory[] = 'SMTP';
+                            // smtp related error handling inside
                         }
                         if ($aError['category'] & SQM_ERROR_LDAP) {
-                             $aErrorCategory[] = 'LDAP';
-                             // ldap related error handling inside
+                            $aErrorCategory[] = 'LDAP';
+                            // ldap related error handling inside
                         }
                         if ($aError['category'] & SQM_ERROR_DB) {
-                             $aErrorCategory[] = 'DATABASE';
-                             // db related error handling inside
+                            $aErrorCategory[] = 'DATABASE';
+                            // db related error handling inside
                         }
                         if ($aError['category'] & SQM_ERROR_PLUGIN) {
-                             $aErrorCategory[] = 'PLUGIN';
-                             do_hook_function('error_handler_plugin',$aError);
-                             // plugin related error handling inside
+                            $aErrorCategory[] = 'PLUGIN';
+                            do_hook_function('error_handler_plugin',$aError);
+                            // plugin related error handling inside
                         }
                         //if ($aError['category'] & SQM_ERROR_X) {
                         //     $aErrorCategory[] = 'X';
@@ -129,17 +156,20 @@ class ErrorHandler {
                 }
                 break;
             default: break;
+            }
+
+            $aErrorTpl = array(
+                'type'      => $iType,
+                'category'  => $aErrorCategory,
+                'message'   => $aError['message'],
+                'link'      => $aError['link'],
+                'tip'       => $aError['tip'],
+                'extra'     => $aError['extra']);
+            // Add the notice/warning/error to the existing list of notices/warnings
+            $this->aErrors[] = $aErrorTpl;
+            $this->Template->assign('aErrors',$this->aErrors);
         }
-        $aErrorTpl = array(
-                    'type'      => $iType,
-                    'category'  => $aErrorCategory,
-                    'message'   => $aError['message'],
-                    'link'      => $aError['link'],
-                    'tip'       => $aError['tip'],
-                    'extra'     => $aError['extra']);
-        // Add the notice/warning/error to the existing list of notices/warnings
-        $this->aErrors[] = $aErrorTpl;
-        $this->Template->assign('aErrors',$this->aErrors);
+
         // Show the error immediate in case of fatal errors
         if ($iType == SQM_ERROR) {
             $this->DisplayErrors();