Commit 7f1f1b77 by Alexander Makarov

- Moved handling fatal errors into handleFatalError() method.

- Refactored checking for fatal error. - Suppressed errors in exception handler in case of fatal errors.
parent 97eda21a
...@@ -150,10 +150,22 @@ class Application extends Module ...@@ -150,10 +150,22 @@ class Application extends Module
$this->afterRequest(); $this->afterRequest();
} }
$this->handleFatalError();
if ($exit) {
exit($status);
}
}
/**
* Handles fatal PHP errors
*/
public function handleFatalError()
{
if(YII_ENABLE_ERROR_HANDLER) { if(YII_ENABLE_ERROR_HANDLER) {
$error = error_get_last(); $error = error_get_last();
if(isset($error['type']) && in_array($error['type'], ErrorException::getFatalCodes())) { if(ErrorException::isFatalErorr($error)) {
unset($this->_memoryReserve); unset($this->_memoryReserve);
$exception = new ErrorException($error['message'], $error['type'], $error['type'], $error['file'], $error['line']); $exception = new ErrorException($error['message'], $error['type'], $error['type'], $error['file'], $error['line']);
...@@ -184,17 +196,13 @@ class Application extends Module ...@@ -184,17 +196,13 @@ class Application extends Module
$this->logException($exception); $this->logException($exception);
if (($handler = $this->getErrorHandler()) !== null) { if (($handler = $this->getErrorHandler()) !== null) {
$handler->handle($exception); @$handler->handle($exception);
} else { } else {
$this->renderException($exception); $this->renderException($exception);
} }
$status = 1; exit(1);
}
} }
if ($exit) {
exit($status);
} }
} }
......
...@@ -47,9 +47,15 @@ class ErrorException extends Exception ...@@ -47,9 +47,15 @@ class ErrorException extends Exception
return $this->severity; return $this->severity;
} }
public static function getFatalCodes() /**
* Returns if error is one of fatal type
*
* @param array $error error got from error_get_last()
* @return bool if error is one of fatal type
*/
public static function isFatalErorr($error)
{ {
return array(E_ERROR, E_PARSE, E_CORE_ERROR, E_CORE_WARNING, E_COMPILE_ERROR, E_COMPILE_WARNING); return isset($error['type']) && in_array($error['type'], array(E_ERROR, E_PARSE, E_CORE_ERROR, E_CORE_WARNING, E_COMPILE_ERROR, E_COMPILE_WARNING));
} }
/** /**
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment