[Tickets #5012] wrong trash folder options check in Message.php delete()

[bugs@bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=5012
-----------------------------------------------------------------------
 Ticket             | 5012
 Created By         | stephan at stean.ch
 Summary            | wrong trash folder options check in Message.php delete()
 Queue              | IMP
 Version            | 4.1.3
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Owners             | 
-----------------------------------------------------------------------


stephan at stean.ch (2007-02-19 10:57) wrote:

The following line will return a wrong result
if the user forgot to set the trash folder in
the IMP options:

$trash = IMP::folderPref($prefs->getValue('trash_folder'), true);

$prefs->getValue('trash_folder')
returns nothing, however folderPref will prepend INBOX,
thus the trash folder will be INBOX, The following check
will not fail, however should:

...
$use_trash = $prefs->getValue('use_trash');
        if ($use_trash && !$trash) {
...

and @imap_mail_move will report an error.