alecpl
2010-10-14 2d1d68bc7e1683bca869fcd1eca517ef209097d8
- Display IMAP errors for LIST/THREAD/SEARCH commands, fixes #1486905


6 files modified
25 ■■■■ changed files
CHANGELOG 1 ●●●● patch | view | raw | blame | history
program/include/rcube_imap_generic.php 2 ●●● patch | view | raw | blame | history
program/localization/en_US/messages.inc 1 ●●●● patch | view | raw | blame | history
program/localization/pl_PL/messages.inc 1 ●●●● patch | view | raw | blame | history
program/steps/mail/list.inc 15 ●●●● patch | view | raw | blame | history
program/steps/mail/search.inc 5 ●●●●● patch | view | raw | blame | history
CHANGELOG
@@ -29,6 +29,7 @@
- Add SASL proxy authentication for SMTP (#1486693)
- Improve displaying of UI messages (#1486977)
- Fix double e-mail filed in identity form (#1487054)
- Display IMAP errors for LIST/THREAD/SEARCH commands (#1486905)
RELEASE 0.4.2
-------------
program/include/rcube_imap_generic.php
@@ -278,7 +278,7 @@
            $str = trim($matches[2]);
            if ($res == 'OK') {
                return self::ERROR_OK;
                return $this->errornum = self::ERROR_OK;
            } else if ($res == 'NO') {
                $this->errornum = self::ERROR_NO;
            } else if ($res == 'BAD') {
program/localization/en_US/messages.inc
@@ -23,6 +23,7 @@
$messages['sessionerror'] = 'Your session is invalid or expired';
$messages['imaperror'] = 'Connection to IMAP server failed';
$messages['servererror'] = 'Server Error!';
$messages['servererrormsg'] = 'Server Error: $msg';
$messages['invalidrequest'] = 'Invalid request! No data was saved.';
$messages['nomessagesfound'] = 'No messages found in this mailbox';
$messages['loggedout'] = 'You have successfully terminated the session. Good bye!';
program/localization/pl_PL/messages.inc
@@ -28,6 +28,7 @@
$messages['sessionerror'] = 'Błąd sesji bądź sesja wygasła!';
$messages['imaperror'] = 'Błąd połączenia z serwerem!';
$messages['servererror'] = 'Błąd serwera!';
$messages['servererrormsg'] = 'Błąd serwera: $msg';
$messages['invalidrequest'] = 'Błędne żądanie! Nie zapisano danych.';
$messages['nomessagesfound'] = 'Brak wiadomości w skrzynce.';
$messages['loggedout'] = 'Użytkownik wylogował się poprawnie.';
program/steps/mail/list.inc
@@ -103,10 +103,17 @@
  if ($search_request)
    $OUTPUT->show_message('searchsuccessful', 'confirmation', array('nr' => $count));
}
else if ($search_request)
  $OUTPUT->show_message('searchnomatch', 'notice');
else
  $OUTPUT->show_message('nomessagesfound', 'notice');
else {
  // handle IMAP errors (e.g. #1486905)
  if ($err_code = $IMAP->get_error_code()) {
    $err_str = $IMAP->get_error_str();
    $OUTPUT->show_message('servererrormsg', 'error', array('msg' => $err_str));
  }
  else if ($search_request)
    $OUTPUT->show_message('searchnomatch', 'notice');
  else
    $OUTPUT->show_message('nomessagesfound', 'notice');
}
// send response
$OUTPUT->send();
program/steps/mail/search.inc
@@ -122,6 +122,11 @@
  if ($search_str)
    $OUTPUT->show_message('searchsuccessful', 'confirmation', array('nr' => $IMAP->messagecount(NULL, 'ALL')));
}
// handle IMAP errors (e.g. #1486905)
else  if ($err_code = $IMAP->get_error_code()) {
  $err_str = $IMAP->get_error_str();
  $OUTPUT->show_message('servererrormsg', 'error', array('msg' => $err_str));
}
else {
  $OUTPUT->show_message('searchnomatch', 'notice');
}