Fix rows count when messages search fails (#1490266)

pull/268/head
Aleksander Machniak 10 years ago
parent d12b999403
commit 10934dcd2a

@ -32,6 +32,7 @@ CHANGELOG Roundcube Webmail
- Fix storing signatures with big images in MySQL database (#1490306) - Fix storing signatures with big images in MySQL database (#1490306)
- Fix Opera browser detection in javascript (#1490307) - Fix Opera browser detection in javascript (#1490307)
- Fix so search filter, scope and fields are reset on folder change - Fix so search filter, scope and fields are reset on folder change
- Fix rows count when messages search fails (#1490266)
RELEASE 1.1.0 RELEASE 1.1.0
------------- -------------

@ -151,12 +151,14 @@ $_SESSION['search_scope'] = $scope;
// Get the headers // Get the headers
if (!$result->incomplete) { if (!$result->incomplete) {
$result_h = $RCMAIL->storage->list_messages($mbox, 1, $sort_column, rcmail_sort_order()); $result_h = $RCMAIL->storage->list_messages($mbox, 1, $sort_column, rcmail_sort_order());
$count = $RCMAIL->storage->count($mbox, $RCMAIL->storage->get_threading() ? 'THREADS' : 'ALL');
} }
// Make sure we got the headers // Make sure we got the headers
if (!empty($result_h)) { if (!empty($result_h)) {
$count = $RCMAIL->storage->count($mbox, $RCMAIL->storage->get_threading() ? 'THREADS' : 'ALL');
rcmail_js_message_list($result_h, false); rcmail_js_message_list($result_h, false);
if ($search_str) { if ($search_str) {
$OUTPUT->show_message('searchsuccessful', 'confirmation', array('nr' => $RCMAIL->storage->count(NULL, 'ALL'))); $OUTPUT->show_message('searchsuccessful', 'confirmation', array('nr' => $RCMAIL->storage->count(NULL, 'ALL')));
} }
@ -172,6 +174,7 @@ if (!empty($result_h)) {
} }
// handle IMAP errors (e.g. #1486905) // handle IMAP errors (e.g. #1486905)
else if ($err_code = $RCMAIL->storage->get_error_code()) { else if ($err_code = $RCMAIL->storage->get_error_code()) {
$count = 0;
$RCMAIL->display_server_error(); $RCMAIL->display_server_error();
} }
// advice the client to re-send the (cross-folder) search request // advice the client to re-send the (cross-folder) search request
@ -180,6 +183,7 @@ else if ($result->incomplete) {
$OUTPUT->command('continue_search', $search_request); $OUTPUT->command('continue_search', $search_request);
} }
else { else {
$count = 0;
$OUTPUT->show_message('searchnomatch', 'notice'); $OUTPUT->show_message('searchnomatch', 'notice');
$OUTPUT->set_env('multifolder_listing', (bool)$result->multi); $OUTPUT->set_env('multifolder_listing', (bool)$result->multi);
if ($result->multi && $scope == 'all') { if ($result->multi && $scope == 'all') {

Loading…
Cancel
Save