[Tickets #7745] Re: The folder "sent-mail" already exists

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/7745
------------------------------------------------------------------------------
  Ticket             | 7745
  Updated By         | rsalmon at mbpgroup.com
  Summary            | The folder "sent-mail" already exists
  Queue              | IMP
  Version            | 4.3
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


rsalmon at mbpgroup.com (2009-01-07 09:41) wrote:

I've modified Folder.php and Tree.php :
--- Folder.php.org	2009-01-07 15:31:15.000000000 +0100
+++ Folder.php	2009-01-07 15:28:36.000000000 +0100
@@ -401,12 +401,12 @@
                  imap_subscribe($imp_imap->stream(), $server . $folder_new);
              }
          }
-
+	$GLOBALS['notification']->push(__FILE__ . ' line ' . __LINE__ . ' '  
. print_r($imaptree->_parent,true));
          if (!empty($deleted)) {
              $imaptree->rename($deleted, $inserted);
              $this->_onDelete($deleted);
          }
-
+	$GLOBALS['notification']->push(__FILE__ . ' line ' . __LINE__ . ' '  
. print_r($imaptree->_parent,true));
          return true;
      }

--- Tree.php.org	2009-01-07 15:31:16.000000000 +0100
+++ Tree.php	2009-01-07 15:29:18.000000000 +0100
@@ -1691,7 +1691,8 @@
       */
      function rename($old, $new)
      {
-        foreach ($old as $key => $val) {
+ 	$GLOBALS['notification']->push(__FILE__ . ' line ' . __LINE__ . ' '  
. print_r($this->_parent,true));
+       foreach ($old as $key => $val) {
              $polled = (isset($this->_tree[$val])) ?  
$this->isPolled($this->_tree[$val]) : false;
              if ($this->delete($val)) {
                  $this->insert($new[$key]);
@@ -1700,7 +1701,8 @@
                  }
              }
          }
-    }
+ 	$GLOBALS['notification']->push(__FILE__ . ' line ' . __LINE__ . ' '  
. print_r($this->_parent,true));
+   }


This is what I get :

/var/www/html/horde/imp/lib/Folder.php line 404 Array
(
     [%] => Array
         (
             [0] => INBOX
             [1] => INBOX.Brouillon
             [2] => INBOX.Sent
             [3] => INBOX.Sent-07-2008
             [4] => INBOX.Sent-08-2008
             [5] => INBOX.Sent-09-2008
             [6] => INBOX.Sent-10-2008
             [7] => INBOX.Sent-nov-2008
             [8] => INBOX.ShareRonan
             [9] => INBOX.Spam
             [10] => INBOX.Trash
             [11] => Dossiers partagés%
             [12] => Dossiers virtuels%
         )

     [Dossiers virtuels%] => Array
         (
             [0] => **search_ff9dsrqr8m0c8kcw4084
         )

)

/var/www/html/horde/imp/lib/IMAP/Tree.php line 1694 Array
(
     [%] => Array
         (
             [0] => INBOX
             [1] => INBOX.Brouillon
             [2] => INBOX.Sent
             [3] => INBOX.Sent-07-2008
             [4] => INBOX.Sent-08-2008
             [5] => INBOX.Sent-09-2008
             [6] => INBOX.Sent-10-2008
             [7] => INBOX.Sent-nov-2008
             [8] => INBOX.ShareRonan
             [9] => INBOX.Spam
             [10] => INBOX.Trash
             [11] => Dossiers partagés%
             [12] => Dossiers virtuels%
         )

     [Dossiers virtuels%] => Array
         (
             [0] => **search_ff9dsrqr8m0c8kcw4084
         )

)

/var/www/html/horde/imp/lib/IMAP/Tree.php line 1704 Array
(
     [%] => Array
         (
             [0] => INBOX
             [1] => INBOX.Brouillon
             [2] => INBOX.Sent-07-2008
             [3] => INBOX.Sent-08-2008
             [4] => INBOX.Sent-09-2008
             [5] => INBOX.Sent-10-2008
             [6] => INBOX.Sent-nov-2008
             [7] => INBOX.ShareRonan
             [8] => INBOX.Spam
             [9] => INBOX.Trash
             [10] => Dossiers partagés%
             [11] => Dossiers virtuels%
             [12] => INBOX.Sent-12-2008
         )

     [Dossiers virtuels%] => Array
         (
             [0] => **search_ff9dsrqr8m0c8kcw4084
         )

)

/var/www/html/horde/imp/lib/Folder.php line 409 Array
(
     [%] => Array
         (
             [0] => INBOX
             [1] => INBOX.Brouillon
             [2] => INBOX.Sent
             [3] => INBOX.Sent-07-2008
             [4] => INBOX.Sent-08-2008
             [5] => INBOX.Sent-09-2008
             [6] => INBOX.Sent-10-2008
             [7] => INBOX.Sent-12-2008
             [8] => INBOX.Sent-nov-2008
             [9] => INBOX.ShareRonan
             [10] => INBOX.Spam
             [11] => INBOX.Trash
             [12] => Dossiers partagés%
             [13] => Dossiers virtuels%
         )

     [Dossiers virtuels%] => Array
         (
             [0] => **search_ff9dsrqr8m0c8kcw4084
         )

)

I can't find out why folder INBOX.Sent is gone from the tree at the  
end of function IMP_Tree::rename (which is the expected behaviour) and  
is back at the end of function IMP_Folder::rename!