[imp] Cyrus imap Junk & Sent child folders

[Tomi Orava]

Quoting Jan Schneider <jan at horde.org>:

> Zitat von Tomi Orava <Tomi.Orava at ncircle.nullnet.fi>:
>
>> Hi,
>>
>> Has anyone else noticed that for some wierd reason
>> the Horde IMP >3.x versions seem to have problems in
>> listing the Sent and Junk sub folders properly ?
>>
>> I'm using the latest Horde IMP 5.0.14 along with
>> Horde 4.0.10 together with Cyrus imapd 2.4.11
>> and for some reason horde imp lists my Sent sub folders
>> as follows:

<SNIP>

>
> http://bugs.horde.org/ticket/10683

Thanks for pointing me to the right direction.
Although the design logic is completely alient to
me (what I read from that bug report) I got my problem
fixed for myself with the following patch:

diff --git a/imp/lib/Mailbox.php b/imp/lib/Mailbox.php
index f06f571..1f2fd61 100644
--- a/imp/lib/Mailbox.php
+++ b/imp/lib/Mailbox.php
@@ -1397,8 +1397,8 @@ class IMP_Mailbox implements Serializable
          if (!isset(self::$_temp[self::CACHE_SPECIALMBOXES])) {
              self::$_temp[self::CACHE_SPECIALMBOXES] = array(
                  self::SPECIAL_DRAFTS => self::getPref('drafts_folder'),
-                self::SPECIAL_SENT =>  
$GLOBALS['injector']->getInstance('IMP_Identity')->getAllSentmailFolders(),
-                self::SPECIAL_SPAM => self::getPref('spam_folder'),
+                // self::SPECIAL_SENT =>  
$GLOBALS['injector']->getInstance('IMP_Identity')->getAllSentmailFolders(),
+                // self::SPECIAL_SPAM => self::getPref('spam_folder'),
                  self::SPECIAL_TRASH =>  
$GLOBALS['prefs']->getValue('use_trash') ?  
self::getPref('trash_folder') : null
              );
          }

Of course this is very crude way to handle it, but I myself
consider the removal of these special handlings much better
than the original behaviour as now I can at least see the
folder names correctly.

Regards,
Tomi Orava