mirror of
https://github.com/phpbb/phpbb.git
synced 2025-06-28 06:08:52 +00:00
changes to how errors are processed by the messenger
git-svn-id: file:///svn/phpbb/trunk@6715 89ea8834-ac86-4346-8a33-228a782c2dd0
This commit is contained in:
parent
daeef71d81
commit
676fd08c47
2 changed files with 23 additions and 11 deletions
|
@ -249,7 +249,7 @@ class messenger
|
||||||
*/
|
*/
|
||||||
function error($type, $msg)
|
function error($type, $msg)
|
||||||
{
|
{
|
||||||
global $user, $phpEx, $phpbb_root_path;
|
global $user, $phpEx, $phpbb_root_path, $config;
|
||||||
|
|
||||||
// Session doesn't exist, create it
|
// Session doesn't exist, create it
|
||||||
if (!isset($user->session_id) || $user->session_id === '')
|
if (!isset($user->session_id) || $user->session_id === '')
|
||||||
|
@ -257,7 +257,22 @@ class messenger
|
||||||
$user->session_begin();
|
$user->session_begin();
|
||||||
}
|
}
|
||||||
|
|
||||||
add_log('critical', 'LOG_ERROR_' . $type, $msg);
|
$calling_page = (!empty($_SERVER['PHP_SELF'])) ? $_SERVER['PHP_SELF'] : $_ENV['PHP_SELF'];
|
||||||
|
|
||||||
|
$message = '';
|
||||||
|
switch ($type)
|
||||||
|
{
|
||||||
|
case 'EMAIL':
|
||||||
|
$message = '<strong>EMAIL/' . (($config['smtp_delivery']) ? 'SMTP' : 'PHP/' . $config['email_function_name'] . '()') . '</strong>';
|
||||||
|
break;
|
||||||
|
|
||||||
|
default:
|
||||||
|
$message = "<strong>$type</strong>";
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
|
$message .= '<br /><em>' . htmlspecialchars($calling_page) . '<em><br /><br />' . $msg . '<br />';
|
||||||
|
add_log('critical', 'LOG_ERROR_' . $type, $message);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -388,9 +403,7 @@ class messenger
|
||||||
|
|
||||||
if (!$result)
|
if (!$result)
|
||||||
{
|
{
|
||||||
$message = '<u>EMAIL ERROR</u> [ ' . (($config['smtp_delivery']) ? 'SMTP' : 'PHP') . ' ]<br /><br />' . $err_msg . '<br /><br /><u>CALLING PAGE</u><br /><br />' . ((!empty($_SERVER['PHP_SELF'])) ? $_SERVER['PHP_SELF'] : $_ENV['PHP_SELF']) . '<br />';
|
$this->error('EMAIL', $err_msg);
|
||||||
|
|
||||||
$this->error('EMAIL', $message);
|
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -613,8 +626,7 @@ class queue
|
||||||
{
|
{
|
||||||
@unlink($this->cache_file . '.lock');
|
@unlink($this->cache_file . '.lock');
|
||||||
|
|
||||||
$message = 'Method: [ ' . (($config['smtp_delivery']) ? 'SMTP' : 'PHP') . ' ]<br /><br />' . $err_msg . '<br /><br /><u>CALLING PAGE</u><br /><br />' . ((!empty($_SERVER['PHP_SELF'])) ? $_SERVER['PHP_SELF'] : $_ENV['PHP_SELF']);
|
messenger::error('EMAIL', $err_msg);
|
||||||
messenger::error('EMAIL', $message);
|
|
||||||
continue 2;
|
continue 2;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
|
@ -624,8 +636,7 @@ class queue
|
||||||
{
|
{
|
||||||
if ($this->jabber->send_message($address, 'normal', NULL, array('body' => $msg)) === false)
|
if ($this->jabber->send_message($address, 'normal', NULL, array('body' => $msg)) === false)
|
||||||
{
|
{
|
||||||
$message = 'Method: [ JABBER ]<br /><br />' . $this->jabber->get_log() . '<br /><br /><u>CALLING PAGE</u><br /><br />' . ((!empty($_SERVER['PHP_SELF'])) ? $_SERVER['PHP_SELF'] : $_ENV['PHP_SELF']);
|
messenger::error('JABBER', $this->jabber_get_log());
|
||||||
messenger::error('JABBER', $message);
|
|
||||||
continue 3;
|
continue 3;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -976,7 +987,7 @@ class smtp_class
|
||||||
{
|
{
|
||||||
if ($this->backtrace)
|
if ($this->backtrace)
|
||||||
{
|
{
|
||||||
$this->backtrace_log[] = $message;
|
$this->backtrace_log[] = htmlspecialchars($message, ENT_COMPAT, 'UTF-8');
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1033,7 +1044,7 @@ class smtp_class
|
||||||
|
|
||||||
if ($this->backtrace)
|
if ($this->backtrace)
|
||||||
{
|
{
|
||||||
$message = '<h1>Backtrace</h1><p>' . implode('<br />', array_map('htmlspecialchars', $this->backtrace_log)) . '</p>';
|
$message = '<h1>Backtrace</h1><p>' . implode('<br />', $this->backtrace_log) . '</p>';
|
||||||
$err_msg .= $message;
|
$err_msg .= $message;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -321,6 +321,7 @@ $lang = array_merge($lang, array(
|
||||||
'PASS_TYPE_ANY_EXPLAIN' => 'Must be between %1$d and %2$d characters.',
|
'PASS_TYPE_ANY_EXPLAIN' => 'Must be between %1$d and %2$d characters.',
|
||||||
'PASS_TYPE_CASE_EXPLAIN' => 'Password must be between %1$d and %2$d chars long and must be mixed case',
|
'PASS_TYPE_CASE_EXPLAIN' => 'Password must be between %1$d and %2$d chars long and must be mixed case',
|
||||||
'PASS_TYPE_SYMBOL_EXPLAIN' => 'Password must be between %1$d and %2$d chars long and must contain symbols',
|
'PASS_TYPE_SYMBOL_EXPLAIN' => 'Password must be between %1$d and %2$d chars long and must contain symbols',
|
||||||
|
'PASSWORD' => 'Password',
|
||||||
'PASSWORD_ACTIVATED' => 'Your new password has been activated',
|
'PASSWORD_ACTIVATED' => 'Your new password has been activated',
|
||||||
'PASSWORD_UPDATED' => 'Your password has been sent successfully to your original email address.',
|
'PASSWORD_UPDATED' => 'Your password has been sent successfully to your original email address.',
|
||||||
'PERMISSIONS_RESTORED' => 'Successfully restored original permissions.',
|
'PERMISSIONS_RESTORED' => 'Successfully restored original permissions.',
|
||||||
|
|
Loading…
Add table
Reference in a new issue