Commit 4a527b9a authored by Penny Leach's avatar Penny Leach
Browse files

tidied up the default exception handler a bit - now uses get_string and

smarty where available.
parent 9060aaaf
......@@ -44,4 +44,9 @@ $string['datarootnotwritable'] = 'Your defined data root directory, %s, is not w
$string['dbconnfailed'] = 'Failed to connect to database, error message was %s';
$string['unrecoverableerror'] = 'A nonrecoverable error occured. This probably means that you have encountered a bug in the system';
$string['unrecoverableerrortitle'] = 'Mahara - Site Unavailable';
......@@ -379,12 +379,16 @@ function exception (Exception $e) {
// if $e not Exception
// get language string based on class name
// rather than by switch on class name
$outputmessage =<<<EOF
<p>Sorry, an unrecoverable error occured. Eventually, this page will look nice
and say something that won't make the viewer think that they broke something,
but since at the moment the viewer is a developer, allow me to laugh at you :)</p>
<p><a href="#" onclick="history.go(-1)">back</a></p>
if (function_exists('get_string')) {
$outputmessage = get_string('unrecoverableerror', 'error');
$outputtitle = get_string('unrecoverableerrortitle', 'error');
else {
// sensible english defaults
$outputmessage = 'A nonrecoverable error occured. '
. 'This probably means you have encountered a bug in the system';
$outputtitle = 'Mahara - Site Unavailable';
switch (get_class($e)) {
case 'ConfigSanityException':
$outputmessage = $message = get_string('configsanityexception', 'error', $e->getMessage());
......@@ -395,10 +399,17 @@ EOF;
log_message($message, LOG_LEVEL_WARN, true, true, $e->getFile(), $e->getLine(), $e->getTrace());
if (function_exists('smarty')) {
$smarty = smarty();
$smarty->assign('title', $outputtitle);
$smarty->assign('message', $outputmessage);
else {
echo <<<EOF
<title>Mahara - Site Unavailable</title>
<style type="text/css">
#reason {
margin: 0 3em;
......@@ -411,12 +422,13 @@ EOF;
echo insert_messages();
echo <<<EOF
{include file="header.tpl" title="$title" heading="$title"}
<div class="content">
{include file="footer.tpl"}
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