From 2244e02017a3da4f8a4d3f78ea888723b63df509 Mon Sep 17 00:00:00 2001 From: Mathew Foscarini Date: Wed, 30 Apr 2014 16:25:01 -0400 Subject: [PATCH] remove constructor return value. Not sure why this was here, but constructors shouldn't return a value. Nothing in the docs says that a controller's implementation of appError should return a value either. So I figure this was a mistake. --- lib/Cake/Error/ExceptionRenderer.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/Cake/Error/ExceptionRenderer.php b/lib/Cake/Error/ExceptionRenderer.php index b55236d6d..944e1b104 100644 --- a/lib/Cake/Error/ExceptionRenderer.php +++ b/lib/Cake/Error/ExceptionRenderer.php @@ -92,7 +92,8 @@ class ExceptionRenderer { $this->controller = $this->_getController($exception); if (method_exists($this->controller, 'appError')) { - return $this->controller->appError($exception); + $this->controller->appError($exception); + return; } $method = $template = Inflector::variable(str_replace('Exception', '', get_class($exception))); $code = $exception->getCode();