Keep current folder context when fetching message parts,

to make sure that proper folder is used in case when
current folder has been changed in meantime.
pull/23/head
Aleksander Machniak 12 years ago
parent 764641d4a3
commit 10562d8a77

@ -52,7 +52,8 @@ class rcube_message
private $opt = array(); private $opt = array();
private $parse_alternative = false; private $parse_alternative = false;
public $uid = null; public $uid;
public $folder;
public $headers; public $headers;
public $parts = array(); public $parts = array();
public $mime_parts = array(); public $mime_parts = array();
@ -68,17 +69,22 @@ class rcube_message
* *
* Provide a uid, and parse message structure. * Provide a uid, and parse message structure.
* *
* @param string $uid The message UID. * @param string $uid The message UID.
* @param string $folder Folder name
* *
* @see self::$app, self::$storage, self::$opt, self::$parts * @see self::$app, self::$storage, self::$opt, self::$parts
*/ */
function __construct($uid) function __construct($uid, $folder = null)
{ {
$this->uid = $uid; $this->uid = $uid;
$this->app = rcube::get_instance(); $this->app = rcube::get_instance();
$this->storage = $this->app->get_storage(); $this->storage = $this->app->get_storage();
$this->folder = strlen($folder) ? $folder : $this->storage->get_folder();
$this->storage->set_options(array('all_headers' => true)); $this->storage->set_options(array('all_headers' => true));
// Set current folder
$this->storage->set_folder($this->folder);
$this->headers = $this->storage->get_message($uid); $this->headers = $this->storage->get_message($uid);
if (!$this->headers) if (!$this->headers)
@ -179,10 +185,12 @@ class rcube_message
} }
return $fp ? true : $part->body; return $fp ? true : $part->body;
} }
// get from IMAP // get from IMAP
$this->storage->set_folder($this->folder);
return $this->storage->get_message_part($this->uid, $mime_id, $part, NULL, $fp, $skip_charset_conv); return $this->storage->get_message_part($this->uid, $mime_id, $part, NULL, $fp, $skip_charset_conv);
} else }
return null;
} }
@ -637,8 +645,10 @@ class rcube_message
function tnef_decode(&$part) function tnef_decode(&$part)
{ {
// @TODO: attachment may be huge, hadle it via file // @TODO: attachment may be huge, hadle it via file
if (!isset($part->body)) if (!isset($part->body)) {
$this->storage->set_folder($this->folder);
$part->body = $this->storage->get_message_part($this->uid, $part->mime_id, $part); $part->body = $this->storage->get_message_part($this->uid, $part->mime_id, $part);
}
$parts = array(); $parts = array();
$tnef = new tnef_decoder; $tnef = new tnef_decoder;
@ -673,8 +683,10 @@ class rcube_message
function uu_decode(&$part) function uu_decode(&$part)
{ {
// @TODO: messages may be huge, hadle body via file // @TODO: messages may be huge, hadle body via file
if (!isset($part->body)) if (!isset($part->body)) {
$this->storage->set_folder($this->folder);
$part->body = $this->storage->get_message_part($this->uid, $part->mime_id, $part); $part->body = $this->storage->get_message_part($this->uid, $part->mime_id, $part);
}
$parts = array(); $parts = array();
// FIXME: line length is max.65? // FIXME: line length is max.65?

@ -195,7 +195,7 @@ abstract class rcube_storage
*/ */
public function set_folder($folder) public function set_folder($folder)
{ {
if ($this->folder == $folder) { if ($this->folder === $folder) {
return; return;
} }

Loading…
Cancel
Save