Bug #13074: Make sure mailbox is subscribed before adding to tree

horde · Mar 25, 2014 · f438afb · f438afb
1 parent 8c039ed
commit f438afb
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/imp/lib/Mailbox.php b/imp/lib/Mailbox.php
@@ -783,15 +783,15 @@ public function create(array $opts = array())
 
         $notification->push(sprintf(_("The mailbox \"%s\" was successfully created."), $this->display), 'horde.success');
 
-        /* Update the mailbox tree. */
-        $injector->getInstance('IMP_Ftree')->insert($this->_mbox);
-
         /* Subscribe, if requested. */
         if ((!isset($opts['subscribe']) && $prefs->getValue('subscribe')) ||
             !empty($opts['subscribe'])) {
             $this->subscribe(true);
         }
 
+        /* Update the mailbox tree. */
+        $injector->getInstance('IMP_Ftree')->insert($this->_mbox);
+
         return true;
     }