[Tickets #3030] Trouble using vtrash

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=3030
-----------------------------------------------------------------------
 Ticket             | 3030
 Updated By         | Michael Slusarz <slusarz at mail.curecanti.org>
 Summary            | Trouble using vtrash
 Queue              | IMP
 Version            | HEAD
 State              | Unconfirmed
 Priority           | 1. Low
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


Michael Slusarz <slusarz at mail.curecanti.org> (2005-11-24 22:42) wrote:

>> You wouldn't happen to be using 'thread' sorting, are you?  With
>> thread sorting, we must always display all deleted messages in the
>> folder, no matter if we are using virtual trash or not.
>
> Is there really now way to work around that, even if not in an 
> elegant way? This is really unintuitive and confusing for the users 
> (including myself).

I agree, but unfortunately there is no elegant way of working around this. 
Simple fact is imap_thread() does not support a search parameter so you have
to do threading on the entire mailbox.  And it's not as simple as removing
deleted entries from the thread list - those deleted entries may radically
change the shape/nature of the thread so simply trying to do some kind of
decrement of the level of child messages horribly breaks the threading tree.
 

Another option would be to build our own thread parsing function using
IMAP_Client::.  But that would have the disadvantage of requiring us to open
a second connection to the IMAP server along with all the overhead of having
to load all code associated with IMAP_Client::.

The last option would be the most radical and that would be to disable the
thread sorting (and reverting to subject sorting) if not using a trash
mailbox and hiding deleted messages.