[CLEANUP] Error handling classes 78/29178/4
authorSteffen Müller <typo3@t3node.com>
Fri, 4 Apr 2014 17:40:01 +0000 (19:40 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sat, 12 Apr 2014 13:56:44 +0000 (15:56 +0200)
* fixes CGL issues in annotations,
* fixes wrong annotations,
* introduces use statements

Resolves: #57661
Releases: 6.2
Change-Id: I41363888c523aedd13a795572d8c66430c0f34e3
Reviewed-on: https://review.typo3.org/29178
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Oliver Klee
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
14 files changed:
typo3/sysext/core/Classes/Error/AbstractExceptionHandler.php
typo3/sysext/core/Classes/Error/DebugExceptionHandler.php
typo3/sysext/core/Classes/Error/ErrorHandler.php
typo3/sysext/core/Classes/Error/ErrorHandlerInterface.php
typo3/sysext/core/Classes/Error/ExceptionHandlerInterface.php
typo3/sysext/core/Classes/Error/Http/AbstractClientErrorException.php
typo3/sysext/core/Classes/Error/Http/AbstractServerErrorException.php
typo3/sysext/core/Classes/Error/Http/BadRequestException.php
typo3/sysext/core/Classes/Error/Http/ForbiddenException.php
typo3/sysext/core/Classes/Error/Http/PageNotFoundException.php
typo3/sysext/core/Classes/Error/Http/ServiceUnavailableException.php
typo3/sysext/core/Classes/Error/Http/StatusException.php
typo3/sysext/core/Classes/Error/Http/UnauthorizedException.php
typo3/sysext/core/Classes/Error/ProductionExceptionHandler.php

index 01505f4..3cf7785 100644 (file)
@@ -23,6 +23,8 @@ namespace TYPO3\CMS\Core\Error;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+
 /**
  * An abstract exception handler
  *
@@ -30,14 +32,15 @@ namespace TYPO3\CMS\Core\Error;
  *
  * @author Ingo Renner <ingo@typo3.org>
  */
-abstract class AbstractExceptionHandler implements \TYPO3\CMS\Core\Error\ExceptionHandlerInterface, \TYPO3\CMS\Core\SingletonInterface {
+abstract class AbstractExceptionHandler implements ExceptionHandlerInterface, \TYPO3\CMS\Core\SingletonInterface {
 
        const CONTEXT_WEB = 'WEB';
        const CONTEXT_CLI = 'CLI';
+
        /**
         * Displays the given exception
         *
-        * @param Exception $exception The exception object
+        * @param \Exception $exception The exception object
         * @return void
         */
        public function handleException(\Exception $exception) {
@@ -53,7 +56,7 @@ abstract class AbstractExceptionHandler implements \TYPO3\CMS\Core\Error\Excepti
        /**
         * Writes exception to different logs
         *
-        * @param Exception $exception The exception
+        * @param \Exception $exception The exception
         * @param string $context The context where the exception was thrown, WEB or CLI
         * @return void
         * @see \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog(), \TYPO3\CMS\Core\Utility\GeneralUtility::devLog()
@@ -62,13 +65,14 @@ abstract class AbstractExceptionHandler implements \TYPO3\CMS\Core\Error\Excepti
                $filePathAndName = $exception->getFile();
                $exceptionCodeNumber = $exception->getCode() > 0 ? '#' . $exception->getCode() . ': ' : '';
                $logTitle = 'Core: Exception handler (' . $context . ')';
-               $logMessage = 'Uncaught TYPO3 Exception: ' . $exceptionCodeNumber . $exception->getMessage() . ' | ' . get_class($exception) . ' thrown in file ' . $filePathAndName . ' in line ' . $exception->getLine();
+               $logMessage = 'Uncaught TYPO3 Exception: ' . $exceptionCodeNumber . $exception->getMessage() . ' | '
+                       . get_class($exception) . ' thrown in file ' . $filePathAndName . ' in line ' . $exception->getLine();
                if ($context === 'WEB') {
-                       $logMessage .= '. Requested URL: ' . \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('TYPO3_REQUEST_URL');
+                       $logMessage .= '. Requested URL: ' . GeneralUtility::getIndpEnv('TYPO3_REQUEST_URL');
                }
                $backtrace = $exception->getTrace();
                // Write error message to the configured syslogs
-               \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog($logMessage, $logTitle, \TYPO3\CMS\Core\Utility\GeneralUtility::SYSLOG_SEVERITY_FATAL);
+               GeneralUtility::sysLog($logMessage, $logTitle, GeneralUtility::SYSLOG_SEVERITY_FATAL);
                // When database credentials are wrong, the exception is probably
                // caused by this. Therefor we cannot do any database operation,
                // otherwise this will lead into recurring exceptions.
@@ -76,7 +80,7 @@ abstract class AbstractExceptionHandler implements \TYPO3\CMS\Core\Error\Excepti
                        // Write error message to devlog
                        // see: $TYPO3_CONF_VARS['SYS']['enable_exceptionDLOG']
                        if (TYPO3_EXCEPTION_DLOG) {
-                               \TYPO3\CMS\Core\Utility\GeneralUtility::devLog($logMessage, $logTitle, 3, array(
+                               GeneralUtility::devLog($logMessage, $logTitle, 3, array(
                                        'TYPO3_MODE' => TYPO3_MODE,
                                        'backtrace' => $backtrace
                                ));
@@ -113,7 +117,7 @@ abstract class AbstractExceptionHandler implements \TYPO3\CMS\Core\Error\Excepti
                                'error' => 2,
                                'details_nr' => 0,
                                'details' => str_replace('%', '%%', $logMessage),
-                               'IP' => \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('REMOTE_ADDR'),
+                               'IP' => GeneralUtility::getIndpEnv('REMOTE_ADDR'),
                                'tstamp' => $GLOBALS['EXEC_TIME'],
                                'workspace' => $workspace
                        );
@@ -123,9 +127,9 @@ abstract class AbstractExceptionHandler implements \TYPO3\CMS\Core\Error\Excepti
 
        /**
         * Sends the HTTP Status 500 code, if $exception is *not* a
-        * TYPO3\\CMS\\Core\\Error\\Http\\StatusException and headers are not sent, yet.
+        * TYPO3\CMS\Core\Error\Http\StatusException and headers are not sent, yet.
         *
-        * @param Exception $exception
+        * @param \Exception $exception
         * @return void
         */
        protected function sendStatusHeaders(\Exception $exception) {
index 9949b22..56d6639 100644 (file)
@@ -46,7 +46,7 @@ class DebugExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractExceptionHandl
        /**
         * Formats and echoes the exception as XHTML.
         *
-        * @param Exception $exception The exception object
+        * @param \Exception $exception The exception object
         * @return void
         * @author Robert Lemke <robert@typo3.org>
         */
@@ -54,7 +54,9 @@ class DebugExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractExceptionHandl
                $this->sendStatusHeaders($exception);
                $filePathAndName = $exception->getFile();
                $exceptionCodeNumber = $exception->getCode() > 0 ? '#' . $exception->getCode() . ': ' : '';
-               $moreInformationLink = $exceptionCodeNumber != '' ? '(<a href="' . TYPO3_URL_EXCEPTION . 'debug/' . $exception->getCode() . '" target="_blank">More information</a>)' : '';
+               $moreInformationLink = $exceptionCodeNumber !== ''
+                       ? '(<a href="' . TYPO3_URL_EXCEPTION . 'debug/' . $exception->getCode() . '" target="_blank">More information</a>)'
+                       : '';
                $backtraceCode = $this->getBacktraceCode($exception->getTrace());
                $this->writeLogEntries($exception, self::CONTEXT_WEB);
                // Set the XML prologue
@@ -64,7 +66,9 @@ class DebugExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractExceptionHandl
      PUBLIC "-//W3C//DTD XHTML 1.1//EN"
      "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">';
                // Get the browser info
-               $browserInfo = \TYPO3\CMS\Core\Utility\ClientUtility::getBrowserInfo(\TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('HTTP_USER_AGENT'));
+               $browserInfo = \TYPO3\CMS\Core\Utility\ClientUtility::getBrowserInfo(
+                       \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('HTTP_USER_AGENT')
+               );
                // Put the XML prologue before or after the doctype declaration according to browser
                if ($browserInfo['browser'] === 'msie' && $browserInfo['version'] < 7) {
                        $headerStart = $docType . LF . $xmlPrologue;
@@ -120,7 +124,7 @@ class DebugExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractExceptionHandl
        /**
         * Formats and echoes the exception for the command line
         *
-        * @param Exception $exception The exception object
+        * @param \Exception $exception The exception object
         * @return void
         * @author Robert Lemke <robert@typo3.org>
         */
@@ -156,7 +160,9 @@ Uncaught TYPO3 Exception ' . $exceptionCodeNumber . $exception->getMessage() . L
                                                if (is_object($argument)) {
                                                        $arguments .= '<span style="color:#FF8700;"><em>' . get_class($argument) . '</em></span>';
                                                } elseif (is_string($argument)) {
-                                                       $preparedArgument = strlen($argument) < 100 ? $argument : substr($argument, 0, 50) . '#tripleDot#' . substr($argument, -50);
+                                                       $preparedArgument = strlen($argument) < 100
+                                                               ? $argument
+                                                               : substr($argument, 0, 50) . '#tripleDot#' . substr($argument, -50);
                                                        $preparedArgument = str_replace(
                                                                array(
                                                                        '#tripleDot#',
@@ -167,7 +173,8 @@ Uncaught TYPO3 Exception ' . $exceptionCodeNumber . $exception->getMessage() . L
                                                                ),
                                                                htmlspecialchars($preparedArgument)
                                                        );
-                                                       $arguments .= '"<span style="color:#FF8700;" title="' . htmlspecialchars($argument) . '">' . $preparedArgument . '</span>"';
+                                                       $arguments .= '"<span style="color:#FF8700;" title="' . htmlspecialchars($argument) . '">'
+                                                               . $preparedArgument . '</span>"';
                                                } elseif (is_numeric($argument)) {
                                                        $arguments .= '<span style="color:#FF8700;">' . (string) $argument . '</span>';
                                                } else {
@@ -176,7 +183,8 @@ Uncaught TYPO3 Exception ' . $exceptionCodeNumber . $exception->getMessage() . L
                                        }
                                }
                                $backtraceCode .= '<pre style="color:#69A550; background-color: #414141; padding: 4px 2px 4px 2px;">';
-                               $backtraceCode .= '<span style="color:white;">' . (count($trace) - $index) . '</span> ' . $class . $step['function'] . '<span style="color:white;">(' . $arguments . ')</span>';
+                               $backtraceCode .= '<span style="color:white;">' . (count($trace) - $index) . '</span> ' . $class
+                                       . $step['function'] . '<span style="color:white;">(' . $arguments . ')</span>';
                                $backtraceCode .= '</pre>';
                                if (isset($step['file'])) {
                                        $backtraceCode .= $this->getCodeSnippet($step['file'], $step['line']) . '<br />';
@@ -190,7 +198,7 @@ Uncaught TYPO3 Exception ' . $exceptionCodeNumber . $exception->getMessage() . L
         * Returns a code snippet from the specified file.
         *
         * @param string $filePathAndName Absolute path and file name of the PHP file
-        * @param integer $lineNumber Line number defining the center of the code snippet
+        * @param int $lineNumber Line number defining the center of the code snippet
         * @return string The code snippet
         * @author Robert Lemke <robert@typo3.org>
         */
index 3bca9d1..098ad91 100644 (file)
@@ -23,6 +23,8 @@ namespace TYPO3\CMS\Core\Error;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+
 /**
  * Global error handler for TYPO3
  *
@@ -42,19 +44,19 @@ class ErrorHandler implements \TYPO3\CMS\Core\Error\ErrorHandlerInterface {
        /**
         * Registers this class as default error handler
         *
-        * @param integer $errorHandlerErrors The integer representing the E_* error level which should be
-        * @return void
+        * @param int $errorHandlerErrors The integer representing the E_* error level which should be
         */
        public function __construct($errorHandlerErrors) {
-                       // reduces error types to those a custom error handler can process
-               $errorHandlerErrors = $errorHandlerErrors & ~(E_COMPILE_WARNING | E_COMPILE_ERROR | E_CORE_WARNING | E_CORE_ERROR | E_PARSE | E_ERROR);
+               $excludedErrors = E_COMPILE_WARNING | E_COMPILE_ERROR | E_CORE_WARNING | E_CORE_ERROR | E_PARSE | E_ERROR;
+               // reduces error types to those a custom error handler can process
+               $errorHandlerErrors = $errorHandlerErrors & ~$excludedErrors;
                set_error_handler(array($this, 'handleError'), $errorHandlerErrors);
        }
 
        /**
         * Defines which error levels should result in an exception thrown.
         *
-        * @param integer $exceptionalErrors The integer representing the E_* error level to handle as exceptions
+        * @param int $exceptionalErrors The integer representing the E_* error level to handle as exceptions
         * @return void
         */
        public function setExceptionalErrors($exceptionalErrors) {
@@ -68,11 +70,11 @@ class ErrorHandler implements \TYPO3\CMS\Core\Error\ErrorHandlerInterface {
         * If TYPO3_MODE is 'BE' the error message is also added to the flashMessageQueue, in FE the error message
         * is displayed in the admin panel (as TsLog message)
         *
-        * @param integer $errorLevel The error level - one of the E_* constants
+        * @param int $errorLevel The error level - one of the E_* constants
         * @param string $errorMessage The error message
         * @param string $errorFile Name of the file the error occurred in
-        * @param integer $errorLine Line number where the error occurred
-        * @return void
+        * @param int $errorLine Line number where the error occurred
+        * @return bool
         * @throws \TYPO3\CMS\Core\Error\Exception with the data passed to this method if the error is registered as exceptionalError
         */
        public function handleError($errorLevel, $errorMessage, $errorFile, $errorLine) {
@@ -99,7 +101,8 @@ class ErrorHandler implements \TYPO3\CMS\Core\Error\ErrorHandlerInterface {
                                        'line ' . $errorLine;
                                die($message);
                        }
-                       // We need to manually require the exception classes in case the autoloader is not available at this point yet.
+                       // We need to manually require the exception classes in case
+                       // the autoloader is not available at this point yet.
                        // @see http://forge.typo3.org/issues/23444
                        if (!class_exists('TYPO3\\CMS\\Core\\Error\\Exception', FALSE)) {
                                require_once PATH_site . 'typo3/sysext/core/Classes/Exception.php';
@@ -125,12 +128,12 @@ class ErrorHandler implements \TYPO3\CMS\Core\Error\ErrorHandlerInterface {
                        // Write error message to the configured syslogs,
                        // see: $TYPO3_CONF_VARS['SYS']['systemLog']
                        if ($errorLevel & $GLOBALS['TYPO3_CONF_VARS']['SYS']['syslogErrorReporting']) {
-                               \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog($message, $logTitle, $severity);
+                               GeneralUtility::sysLog($message, $logTitle, $severity);
                        }
                        // Write error message to devlog extension(s),
                        // see: $TYPO3_CONF_VARS['SYS']['enable_errorDLOG']
                        if (TYPO3_ERROR_DLOG) {
-                               \TYPO3\CMS\Core\Utility\GeneralUtility::devLog($message, $logTitle, $severity + 1);
+                               GeneralUtility::devLog($message, $logTitle, $severity + 1);
                        }
                        // Write error message to TSlog (admin panel)
                        if (is_object($GLOBALS['TT'])) {
@@ -147,9 +150,14 @@ class ErrorHandler implements \TYPO3\CMS\Core\Error\ErrorHandlerInterface {
                        }
                        // Add error message to the flashmessageQueue
                        if (defined('TYPO3_ERRORHANDLER_MODE') && TYPO3_ERRORHANDLER_MODE == 'debug') {
-                               $flashMessage = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Messaging\\FlashMessage', $message, 'PHP ' . $errorLevels[$errorLevel], $severity);
+                               $flashMessage = GeneralUtility::makeInstance(
+                                       'TYPO3\\CMS\\Core\\Messaging\\FlashMessage',
+                                       $message,
+                                       'PHP ' . $errorLevels[$errorLevel],
+                                       $severity
+                               );
                                /** @var $flashMessageService \TYPO3\CMS\Core\Messaging\FlashMessageService */
-                               $flashMessageService = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Messaging\\FlashMessageService');
+                               $flashMessageService = GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Messaging\\FlashMessageService');
                                /** @var $defaultFlashMessageQueue \TYPO3\CMS\Core\Messaging\FlashMessageQueue */
                                $defaultFlashMessageQueue = $flashMessageService->getMessageQueueByIdentifier();
                                $defaultFlashMessageQueue->enqueue($flashMessage);
@@ -163,7 +171,7 @@ class ErrorHandler implements \TYPO3\CMS\Core\Error\ErrorHandlerInterface {
         * Writes an error in the sys_log table
         *
         * @param string $logMessage Default text that follows the message (in english!).
-        * @param integer $severity The eror level of the message (0 = OK, 1 = warning, 2 = error)
+        * @param int $severity The eror level of the message (0 = OK, 1 = warning, 2 = error)
         * @return void
         */
        protected function writeLog($logMessage, $severity) {
@@ -185,7 +193,7 @@ class ErrorHandler implements \TYPO3\CMS\Core\Error\ErrorHandlerInterface {
                                'error' => $severity,
                                'details_nr' => 0,
                                'details' => $logMessage,
-                               'IP' => \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('REMOTE_ADDR'),
+                               'IP' => GeneralUtility::getIndpEnv('REMOTE_ADDR'),
                                'tstamp' => $GLOBALS['EXEC_TIME'],
                                'workspace' => $workspace
                        );
index 6f8eae9..e7141d5 100644 (file)
@@ -34,15 +34,14 @@ interface ErrorHandlerInterface {
        /**
         * Registers this class as default error handler
         *
-        * @param integer $errorHandlerErrors The integer representing the E_* error level which should be
-        * @return void
+        * @param int $errorHandlerErrors The integer representing the E_* error level which should be
         */
        public function __construct($errorHandlerErrors);
 
        /**
         * Defines which error levels should result in an exception thrown.
         *
-        * @param integer $exceptionalErrors The integer representing the E_* error level to handle as exceptions
+        * @param int $exceptionalErrors The integer representing the E_* error level to handle as exceptions
         * @return void
         */
        public function setExceptionalErrors($exceptionalErrors);
@@ -54,11 +53,11 @@ interface ErrorHandlerInterface {
         * If TYPO3_MODE is 'BE' the error message is also added to the flashMessageQueue, in FE the error message
         * is displayed in the admin panel (as TsLog message)
         *
-        * @param integer $errorLevel The error level - one of the E_* constants
+        * @param int $errorLevel The error level - one of the E_* constants
         * @param string $errorMessage The error message
         * @param string $errorFile Name of the file the error occurred in
-        * @param integer $errorLine Line number where the error occurred
-        * @return void
+        * @param int $errorLine Line number where the error occurred
+        * @return bool
         * @throws \TYPO3\CMS\Core\Error\Exception with the data passed to this method if the error is registered as exceptionalError
         */
        public function handleError($errorLevel, $errorMessage, $errorFile, $errorLine);
index d2f84d6..3cbf363 100644 (file)
@@ -40,7 +40,7 @@ interface ExceptionHandlerInterface
        /**
         * Handles the given exception
         *
-        * @param Exception $exception: The exception object
+        * @param \Exception $exception: The exception object
         * @return void
         */
        public function handleException(\Exception $exception);
@@ -48,7 +48,7 @@ interface ExceptionHandlerInterface
        /**
         * Formats and echoes the exception as XHTML.
         *
-        * @param Exception $exception The exception object
+        * @param \Exception $exception The exception object
         * @return void
         */
        public function echoExceptionWeb(\Exception $exception);
@@ -56,7 +56,7 @@ interface ExceptionHandlerInterface
        /**
         * Formats and echoes the exception for the command line
         *
-        * @param Exception $exception The exception object
+        * @param \Exception $exception The exception object
         * @return void
         */
        public function echoExceptionCLI(\Exception $exception);
index 1d24292..411c1eb 100644 (file)
@@ -28,7 +28,7 @@ namespace TYPO3\CMS\Core\Error\Http;
  *
  * @author Steffen Gebert <steffen.gebert@typo3.org>
  */
-abstract class AbstractClientErrorException extends \TYPO3\CMS\Core\Error\Http\StatusException {
+abstract class AbstractClientErrorException extends StatusException {
 
 
 }
index 42e3ad3..4402035 100644 (file)
@@ -28,7 +28,7 @@ namespace TYPO3\CMS\Core\Error\Http;
  *
  * @author Steffen Gebert <steffen.gebert@typo3.org>
  */
-abstract class AbstractServerErrorException extends \TYPO3\CMS\Core\Error\Http\StatusException {
+abstract class AbstractServerErrorException extends StatusException {
 
 
 }
index abff210..263b3ae 100644 (file)
@@ -28,7 +28,7 @@ namespace TYPO3\CMS\Core\Error\Http;
  *
  * @author Steffen Gebert <steffen.gebert@typo3.org>
  */
-class BadRequestException extends \TYPO3\CMS\Core\Error\Http\AbstractClientErrorException {
+class BadRequestException extends AbstractClientErrorException {
 
        /**
         * @var array HTTP Status Header lines
@@ -49,7 +49,7 @@ class BadRequestException extends \TYPO3\CMS\Core\Error\Http\AbstractClientError
         * Constructor for this Status Exception
         *
         * @param string $message Error Message
-        * @param integer $code Exception Code
+        * @param int $code Exception Code
         */
        public function __construct($message = NULL, $code = 0) {
                if (!empty($message)) {
index f47d75c..6225122 100644 (file)
@@ -28,7 +28,7 @@ namespace TYPO3\CMS\Core\Error\Http;
  *
  * @author Steffen Gebert <steffen.gebert@typo3.org>
  */
-class ForbiddenException extends \TYPO3\CMS\Core\Error\Http\AbstractClientErrorException {
+class ForbiddenException extends AbstractClientErrorException {
 
        /**
         * @var array HTTP Status Header lines
@@ -49,7 +49,7 @@ class ForbiddenException extends \TYPO3\CMS\Core\Error\Http\AbstractClientErrorE
         * Constructor for this Status Exception
         *
         * @param string $message Error Message
-        * @param integer $code Exception Code
+        * @param int $code Exception Code
         */
        public function __construct($message = NULL, $code = 0) {
                if (!empty($message)) {
index dccf66c..7593f24 100644 (file)
@@ -28,7 +28,7 @@ namespace TYPO3\CMS\Core\Error\Http;
  *
  * @author Steffen Gebert <steffen.gebert@typo3.org>
  */
-class PageNotFoundException extends \TYPO3\CMS\Core\Error\Http\AbstractClientErrorException {
+class PageNotFoundException extends AbstractClientErrorException {
 
        /**
         * @var array HTTP Status Header lines
@@ -49,7 +49,7 @@ class PageNotFoundException extends \TYPO3\CMS\Core\Error\Http\AbstractClientErr
         * Constructor for this Status Exception
         *
         * @param string $message Error Message
-        * @param integer $code Exception Code
+        * @param int $code Exception Code
         */
        public function __construct($message = NULL, $code = 0) {
                if (!empty($message)) {
index b6f8ab2..ccad85a 100644 (file)
@@ -28,7 +28,7 @@ namespace TYPO3\CMS\Core\Error\Http;
  *
  * @author Steffen Gebert <steffen.gebert@typo3.org>
  */
-class ServiceUnavailableException extends \TYPO3\CMS\Core\Error\Http\AbstractServerErrorException {
+class ServiceUnavailableException extends AbstractServerErrorException {
 
        /**
         * @var array HTTP Status Header lines
@@ -49,7 +49,7 @@ class ServiceUnavailableException extends \TYPO3\CMS\Core\Error\Http\AbstractSer
         * Constructor for this Status Exception
         *
         * @param string $message Error Message
-        * @param integer $code Exception Code
+        * @param int $code Exception Code
         */
        public function __construct($message = NULL, $code = 0) {
                if (!empty($message)) {
index 8102ada..b523c1a 100644 (file)
@@ -46,7 +46,7 @@ class StatusException extends \TYPO3\CMS\Core\Error\Exception {
         * @param string|array $statusHeaders HTTP Status header line(s)
         * @param string $title Title of the error message
         * @param string $message Error Message
-        * @param integer $code Exception Code
+        * @param int $code Exception Code
         */
        public function __construct($statusHeaders, $message, $title = '', $code = 0) {
                if (is_array($statusHeaders)) {
index d0e2b45..d5641c5 100644 (file)
@@ -28,7 +28,7 @@ namespace TYPO3\CMS\Core\Error\Http;
  *
  * @author Steffen Gebert <steffen.gebert@typo3.org>
  */
-class UnauthorizedException extends \TYPO3\CMS\Core\Error\Http\AbstractClientErrorException {
+class UnauthorizedException extends AbstractClientErrorException {
 
        /**
         * @var array HTTP Status Header lines
@@ -49,7 +49,7 @@ class UnauthorizedException extends \TYPO3\CMS\Core\Error\Http\AbstractClientErr
         * Constructor for this Status Exception
         *
         * @param string $message Error Message
-        * @param integer $code Exception Code
+        * @param int $code Exception Code
         */
        public function __construct($message = NULL, $code = 0) {
                if (!empty($message)) {
index e9d0db6..ab08a08 100644 (file)
@@ -30,7 +30,7 @@ namespace TYPO3\CMS\Core\Error;
  *
  * @author Ingo Renner <ingo@typo3.org>
  */
-class ProductionExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractExceptionHandler {
+class ProductionExceptionHandler extends AbstractExceptionHandler {
 
        /**
         * Default title for error messages
@@ -58,20 +58,24 @@ class ProductionExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractException
        /**
         * Echoes an exception for the web.
         *
-        * @param Exception $exception The exception
+        * @param \Exception $exception The exception
         * @return void
         */
        public function echoExceptionWeb(\Exception $exception) {
                $this->sendStatusHeaders($exception);
                $this->writeLogEntries($exception, self::CONTEXT_WEB);
-               $messageObj = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Messaging\\ErrorpageMessage', $this->getMessage($exception), $this->getTitle($exception));
+               $messageObj = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(
+                       'TYPO3\\CMS\\Core\\Messaging\\ErrorpageMessage',
+                       $this->getMessage($exception),
+                       $this->getTitle($exception)
+               );
                $messageObj->output();
        }
 
        /**
         * Echoes an exception for the command line.
         *
-        * @param Exception $exception The exception
+        * @param \Exception $exception The exception
         * @return void
         */
        public function echoExceptionCLI(\Exception $exception) {
@@ -82,12 +86,12 @@ class ProductionExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractException
        /**
         * Determines, whether Exception details should be outputted
         *
-        * @param Exception $exception The exception
+        * @param \Exception $exception The exception
         * @return boolean
         */
        protected function discloseExceptionInformation(\Exception $exception) {
                // Show client error messages 40x in every case
-               if ($exception instanceof \TYPO3\CMS\Core\Error\Http\AbstractClientErrorException) {
+               if ($exception instanceof Http\AbstractClientErrorException) {
                        return TRUE;
                }
                // Only show errors in FE, if a BE user is authenticated
@@ -100,11 +104,11 @@ class ProductionExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractException
        /**
         * Returns the title for the error message
         *
-        * @param Exception $exception Exception causing the error
+        * @param \Exception $exception Exception causing the error
         * @return string
         */
        protected function getTitle(\Exception $exception) {
-               if ($this->discloseExceptionInformation($exception) && method_exists($exception, 'getTitle') && strlen($exception->getTitle()) > 0) {
+               if ($this->discloseExceptionInformation($exception) && method_exists($exception, 'getTitle') && $exception->getTitle() !== '') {
                        return htmlspecialchars($exception->getTitle());
                } else {
                        return $this->defaultTitle;
@@ -114,14 +118,15 @@ class ProductionExceptionHandler extends \TYPO3\CMS\Core\Error\AbstractException
        /**
         * Returns the message for the error message
         *
-        * @param Exception $exception Exception causing the error
+        * @param \Exception $exception Exception causing the error
         * @return string
         */
        protected function getMessage(\Exception $exception) {
                if ($this->discloseExceptionInformation($exception)) {
                        // Exception has an error code given
                        if ($exception->getCode() > 0) {
-                               $moreInformationLink = '<p>More information regarding this error might be available <a href="' . TYPO3_URL_EXCEPTION . $exception->getCode() . '" target="_blank">online</a>.</p>';
+                               $moreInformationLink = '<p>More information regarding this error might be available <a href="'
+                                       . TYPO3_URL_EXCEPTION . $exception->getCode() . '" target="_blank">online</a>.</p>';
                        } else {
                                $moreInformationLink = '';
                        }