Changeset df0da2e0 in github

Timestamp:

Feb 2, 2008 2:41:32 PM (5 years ago)

Author:

thomascube <thomas@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

adc3e41

Parents:

39508cb

Message:

Simplified fix for remembering searches when viewing message + Don't ask for MDN confirmations on drafted messages (#1484691)

Location:

program

Files:

• include/rcube_imap.inc (modified) (1 diff)
• steps/mail/show.inc (modified) (3 diffs)

program/include/rcube_imap.inc

@@ -762 +762 @@
 
     $mailbox = $mbox_name ? $this->_mod_mailbox($mbox_name) : $this->mailbox;
-    $key = "$mbox:".$this->sort_field.":".$this->sort_order.".msgi";
+    $key = "{$mailbox}:{$this->sort_field}:{$this->sort_order}:{$this->search_string}.msgi";
+
+    // we have a saved search result. get index from there
+    if (!isset($this->cache[$key]) && $this->search_string && $mailbox == $this->mailbox)
+    {
+      $this->cache[$key] = $a_msg_headers = array();
+      $this->_fetch_headers($mailbox, join(',', $this->search_set), $a_msg_headers, NULL);
+
+      foreach (iil_SortHeaders($a_msg_headers, $this->sort_field, $this->sort_order) as $i => $msg)
+        $this->cache[$key][] = $msg->uid;
+    }
 
     // have stored it in RAM

program/steps/mail/show.inc

@@ -75 +75 @@
   
   // check for unset disposition notification
-  if ($MESSAGE['headers']->mdn_to && !$MESSAGE['headers']->mdn_sent)
+  if ($MESSAGE['headers']->mdn_to && !$MESSAGE['headers']->mdn_sent && $IMAP->get_mailbox_name() != $CONFIG['drafts_mbox'])
   {
     rcube_add_label('mdnrequest');
@@ -81 +81 @@
   }
 
-  // set message set for search result
-  if (!empty($_REQUEST['_search']) && isset($_SESSION['search'][$_REQUEST['_search']]))
-    {
-    $IMAP->set_search_set($_SESSION['search'][$_REQUEST['_search']]);
-    $OUTPUT->set_env('search_request', $_REQUEST['_search']);
-    }
 
   $next = $prev = $first = $last = -1;
   // get previous, first, next and last message UID
   if ((!($_SESSION['sort_col'] == 'date' && $_SESSION['sort_order'] == 'DESC') &&
-      $IMAP->get_capability('sort') ) && !(!empty($_REQUEST['_search']) &&
-      isset($_SESSION['search'][$_REQUEST['_search']])) )
+      $IMAP->get_capability('sort')) || !empty($_REQUEST['_search']))
     {
     // Only if we use custom sorting
@@ -102 +95 @@
     $next = isset($a_msg_index[$MESSAGE['index']+1]) ? $a_msg_index[$MESSAGE['index']+1] : -1 ;
     $last = count($a_msg_index)>0 ? $a_msg_index[count($a_msg_index)-1] : -1;
-    }
-  elseif (!empty($_REQUEST['_search']) && isset($_SESSION['search'][$_REQUEST['_search']]))
-    {
-      $search_data = $_SESSION['search'][$_REQUEST['_search']];
-      $result = $IMAP->search(NULL, $search_data[0], $search_data[1],$search_data[3]);
-      $result = array_reverse($result);
-      foreach ($result as $key=>$rid)
-        {
-        $result[$key] = $IMAP->get_uid($rid);
-        if ($MESSAGE['UID'] == $result[$key])
-          $seq = $key;
-        }
-      $prev = isset($result[$seq-1]) ? $result[$seq-1] : -1 ;
-      $first = count($result)>0 ? $result[0] : -1;
-      $next = isset($result[$seq+1]) ? $result[$seq+1] : -1 ;
-      $last = count($result)>0 ? $result[count($result)-1] : -1;
-      $MESSAGE['index'] = $seq;
     }
   else