From f6aac389bfc29e0f6d1f2d43aa900a2ba017bc2f Mon Sep 17 00:00:00 2001 From: alecpl <alec@alec.pl> Date: Wed, 13 Oct 2010 15:15:51 -0400 Subject: [PATCH] - Minimize session data size by storing only last search result and by removing search result from session after listing with empty search filter --- program/steps/mail/list.inc | 15 ++++++++++++--- 1 files changed, 12 insertions(+), 3 deletions(-) diff --git a/program/steps/mail/list.inc b/program/steps/mail/list.inc index b5ee1b6..f5fc44b 100644 --- a/program/steps/mail/list.inc +++ b/program/steps/mail/list.inc @@ -58,7 +58,8 @@ { $search_request = md5($mbox_name.$_SESSION['search_filter']); $IMAP->search($mbox_name, $_SESSION['search_filter'], RCMAIL_CHARSET, $sort_col); - $_SESSION['search'][$search_request] = $IMAP->get_search_set(); + $_SESSION['search'] = $IMAP->get_search_set(); + $_SESSION['search_request'] = $search_request; $OUTPUT->set_env('search_request', $search_request); } @@ -67,8 +68,16 @@ $a_headers = $IMAP->list_headers($mbox_name, NULL, $sort_col, $sort_order); // update search set (possible change of threading mode) -if (!empty($_REQUEST['_search']) && isset($_SESSION['search'][$_REQUEST['_search']])) - $_SESSION['search'][$_REQUEST['_search']] = $IMAP->get_search_set(); +if (!empty($_REQUEST['_search']) && isset($_SESSION['search']) + && $_SESSION['search_request'] == $_REQUEST['_search'] +) { + $_SESSION['search'] = $IMAP->get_search_set(); +} +// remove old search data +else if (empty($_REQUEST['_search']) && isset($_SESSION['search'])) { + $RCMAIL->session->remove('search'); +} + // empty result? we'll skip UNSEEN counting in rcmail_send_unread_count() if (empty($search_request) && empty($a_headers)) { -- Gitblit v1.9.1