Refactor exception handling into dedicated class

Signed-off-by: Florian Pritz <bluewind@xinu.at>

1 files changed, 2 insertions, 138 deletions

diff --git a/index.php b/index.php
index f9a9b66ce..0d75fb299 100644
--- a/index.php
+++ b/index.php
@@ -198,146 +198,10 @@ if (false && defined('ENVIRONMENT'))
 /*
  * Custom error handling
  */
-/* CI uses that name for it's error handling function. It misleading, but
- *  whatever. If I don't use it the framework will override my handler later.
- */
-function _exception_handler($errno, $errstr, $errfile, $errline)
-{
-	if (!(error_reporting() & $errno)) {
-		// This error code is not included in error_reporting
-		return;
-	}
-	throw new ErrorException($errstr, 0, $errno, $errfile, $errline);
-}
-set_error_handler("_exception_handler");
-
-// Source: https://gist.github.com/abtris/1437966
-function getExceptionTraceAsString($exception) {
-	$rtn = "";
-	$count = 0;
-	foreach ($exception->getTrace() as $frame) {
-		$args = "";
-		if (isset($frame['args'])) {
-			$args = array();
-			foreach ($frame['args'] as $arg) {
-				if (is_string($arg)) {
-					$args[] = "'" . $arg . "'";
-				} elseif (is_array($arg)) {
-					$args[] = "Array";
-				} elseif (is_null($arg)) {
-					$args[] = 'NULL';
-				} elseif (is_bool($arg)) {
-					$args[] = ($arg) ? "true" : "false";
-				} elseif (is_object($arg)) {
-					$args[] = get_class($arg);
-				} elseif (is_resource($arg)) {
-					$args[] = get_resource_type($arg);
-				} else {
-					$args[] = $arg;
-				}
-			}
-			$args = join(", ", $args);
-		}
-		$rtn .= sprintf( "#%s %s(%s): %s(%s)\n",
-			$count,
-			isset($frame['file']) ? $frame['file'] : 'unknown file',
-			isset($frame['line']) ? $frame['line'] : 'unknown line',
-			(isset($frame['class']))  ? $frame['class'].$frame['type'].$frame['function'] : $frame['function'],
-			$args );
-		$count++;
-	}
-	return $rtn;
-}
-
-function _log_exception($ex)
-{
-	$exceptions = array($ex);
-	while ($ex->getPrevious() !== null) {
-		$ex = $ex->getPrevious();
-		$exceptions[] = $ex;
-	}
-
-	foreach ($exceptions as $key => $e) {
-		$message = sprintf("Exception %d/%d '%s' with message '%s' in %s:%d\n", $key+1, count($exceptions), get_class($e), $e->getMessage(), $e->getFile(), $e->getLine());
-		if (method_exists($e, "get_error_id")) {
-			$message .= 'Error ID: '.$e->get_error_id()."\n";
-		}
-		if (method_exists($e, "get_data") && $e->get_data() !== NULL) {
-			$message .= 'Data: '.var_export($e->get_data(), true)."\n";
-		}
-		$message .= "Backtrace:\n".getExceptionTraceAsString($e)."\n";
-		error_log($message);
-	}
-}
-
-// The actual exception handler
-function _actual_exception_handler($ex)
-{
-	_log_exception($ex);
 
-	$display_errors = in_array(strtolower(ini_get('display_errors')), array('1', 'on', 'true', 'stdout'));
-	if (php_sapi_name() === 'cli' OR defined('STDIN')) {
-		$display_errors = true;
-	}
-
-	$GLOBALS["is_error_page"] = true;
-	$heading = "Internal Server Error";
-	$message = "<p>An unhandled error occured.</p>\n";
-
-	if ($display_errors) {
-		$exceptions = array($ex);
-		while ($ex->getPrevious() !== null) {
-			$ex = $ex->getPrevious();
-			$exceptions[] = $ex;
-		}
-
-		foreach ($exceptions as $key => $e) {
-			$backtrace = getExceptionTraceAsString($e);
-			$message .= '<div>';
-			$message .= '<b>Exception '.($key+1).' of '.count($exceptions).'</b><br>';
-			$message .= '<b>Fatal error</b>:  Uncaught exception '.htmlspecialchars(get_class($e)).'<br>';
-			$message .= '<b>Message</b>: '.htmlspecialchars($e->getMessage()).'<br>';
-			if (method_exists($e, "get_error_id")) {
-				$message .= '<b>Error ID</b>: '.htmlspecialchars($e->get_error_id()).'<br>';
-			}
-			if (method_exists($e, "get_data") && $e->get_data() !== NULL) {
-				$message .= '<b>Data</b>: <pre>'.htmlspecialchars(var_export($e->get_data(), true)).'</pre><br>';
-			}
-			$message .= '<b>Backtrace:</b><br>';
-			$message .= '<pre>'.htmlspecialchars(str_replace(FCPATH, "./", $backtrace)).'</pre>';
-			$message .= 'thrown in <b>'.htmlspecialchars($e->getFile()).'</b> on line <b>'.htmlspecialchars($e->getLine()).'</b><br>';
-			$message .= '</div>';
-		}
-	} else {
-		$message .="<p>More information can be found in the php error log or by enabling display_errors.</p>";
-	}
-
-	$message = "$message";
-	include APPPATH."/errors/error_general.php";
-}
-set_exception_handler('_actual_exception_handler');
-
-/**
- * Checks for a fatal error, work around for set_error_handler not working on fatal errors.
- */
-function check_for_fatal()
-{
-	$error = error_get_last();
-	if ($error["type"] == E_ERROR) {
-		_actual_exception_handler(new ErrorException(
-			$error["message"], 0, $error["type"], $error["file"], $error["line"]));
-	}
-}
-register_shutdown_function("check_for_fatal");
-
-function _assert_failure($file, $line, $expr, $message = "")
-{
-	_actual_exception_handler(new Exception("assert($expr): Assertion failed in $file at line $line".($message != "" ? " with message: '$message'" : "")));
-	exit(1);
-}
+require APPPATH.'libraries/ExceptionHandler.php';
+\libraries\ExceptionHandler::setup();
 
-assert_options(ASSERT_ACTIVE,   true);
-assert_options(ASSERT_CALLBACK, '_assert_failure');
 /*
  * --------------------------------------------------------------------
  * LOAD THE BOOTSTRAP FILE