diff options
author | diosmosis <benakamoorthi@fastmail.fm> | 2014-08-20 11:12:06 +0400 |
---|---|---|
committer | diosmosis <benakamoorthi@fastmail.fm> | 2014-08-20 11:22:09 +0400 |
commit | ca0bdf191bf1ab85525169242cbfef754855dfec (patch) | |
tree | 5e33c8be0f7b171243317db824886b78eb31708d /core/ReportRenderer.php | |
parent | 09246eb179ceb4202c5f404179088db0056e8370 (diff) |
Fix bug where certain factory methods assume new operator will throw on class not found (fatal error occurs). Use class_exists to check and then throw exception.
Diffstat (limited to 'core/ReportRenderer.php')
-rw-r--r-- | core/ReportRenderer.php | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/core/ReportRenderer.php b/core/ReportRenderer.php index f58dd8456e..3367491428 100644 --- a/core/ReportRenderer.php +++ b/core/ReportRenderer.php @@ -51,10 +51,7 @@ abstract class ReportRenderer $name = ucfirst(strtolower($rendererType)); $className = 'Piwik\ReportRenderer\\' . $name; - try { - return new $className; - } catch (Exception $e) { - + if (!class_exists($className)) { @header('Content-Type: text/html; charset=utf-8'); throw new Exception( @@ -64,6 +61,8 @@ abstract class ReportRenderer ) ); } + + return new $className; } /** |