[Svn] r3318 - in trunk/roundcubemail: . config program/include

trac at roundcube.net trac at roundcube.net
Thu Mar 4 16:34:32 CET 2010


Author: alec
Date: 2010-03-04 09:34:32 -0600 (Thu, 04 Mar 2010)
New Revision: 3318

Modified:
   trunk/roundcubemail/CHANGELOG
   trunk/roundcubemail/config/main.inc.php.dist
   trunk/roundcubemail/program/include/rcube_imap.php
Log:
- Fix inconsistent behaviour of 'delete_always' option (#1486299)


Modified: trunk/roundcubemail/CHANGELOG
===================================================================
--- trunk/roundcubemail/CHANGELOG	2010-03-04 14:39:47 UTC (rev 3317)
+++ trunk/roundcubemail/CHANGELOG	2010-03-04 15:34:32 UTC (rev 3318)
@@ -1,6 +1,7 @@
 CHANGELOG RoundCube Webmail
 ===========================
 
+- Fix inconsistent behaviour of 'delete_always' option (#1486299)
 - Fix deleting all messages from last list page (#1486293)
 - Flag original messages when sending a draft (#1486203)
 - Changed signature separator when top-posting (#1486330)

Modified: trunk/roundcubemail/config/main.inc.php.dist
===================================================================
--- trunk/roundcubemail/config/main.inc.php.dist	2010-03-04 14:39:47 UTC (rev 3317)
+++ trunk/roundcubemail/config/main.inc.php.dist	2010-03-04 15:34:32 UTC (rev 3318)
@@ -361,13 +361,9 @@
 
 /**
  * 'Delete always'
- * This setting reflects if mail should be always marked as deleted,
- * even if moving to "Trash" fails. This is necessary in some setups
- * because a) people may not have a Trash folder or b) they are over
- * quota (and Trash is included in the quota).
- *
- * This is a failover setting for iil_C_Move when a message is moved
- * to the Trash.
+ * This setting reflects if mail should be always deleted
+ * when moving to Trash fails. This is necessary in some setups
+ * when user is over quota and Trash is included in the quota.
  */
 $rcmail_config['delete_always'] = false;
 

Modified: trunk/roundcubemail/program/include/rcube_imap.php
===================================================================
--- trunk/roundcubemail/program/include/rcube_imap.php	2010-03-04 14:39:47 UTC (rev 3317)
+++ trunk/roundcubemail/program/include/rcube_imap.php	2010-03-04 15:34:32 UTC (rev 3318)
@@ -1793,7 +1793,7 @@
     // move messages
     $iil_move = iil_C_Move($this->conn, join(',', $a_uids), $from_mbox, $to_mbox);
     $moved = !($iil_move === false || $iil_move < 0);
-    
+
     // send expunge command in order to have the moved message
     // really deleted from the source mailbox
     if ($moved) {
@@ -1802,8 +1802,8 @@
       $this->_clear_messagecount($to_mbox);
     }
     // moving failed
-    else if (rcmail::get_instance()->config->get('delete_always', false)) {
-      return iil_C_Delete($this->conn, $from_mbox, join(',', $a_uids));
+    else if ($config->get('delete_always', false) && $tbox == $config->get('trash_mbox')) {
+      return $this->delete_message($a_uids, $fbox);
     }
 
     // remove message ids from search set

_______________________________________________
http://lists.roundcube.net/mailman/listinfo/svn



More information about the Svn mailing list