Fix deleting user flag
authorMichael M Slusarz <slusarz@curecanti.org>
Wed, 24 Nov 2010 07:08:49 +0000 (00:08 -0700)
committerMichael M Slusarz <slusarz@curecanti.org>
Wed, 24 Nov 2010 07:08:49 +0000 (00:08 -0700)
imp/lib/Imap/Flags.php
imp/lib/Prefs/Ui.php

index 2ed8a83..80784c4 100644 (file)
@@ -246,8 +246,9 @@ class IMP_Imap_Flags
         if (isset($this->_flags[$label]) &&
             $this->_flags[$label]['l'] &&
             !empty($this->_flags[$label]['d'])) {
+            $user_flag = isset($this->_userflags[$label]);
             unset($this->_flags[$label], $this->_userflags[$label]);
-            $this->_save();
+            $this->_save($user_flag);
             return true;
         }
 
index 7bd54e5..6d915d9 100644 (file)
@@ -994,8 +994,8 @@ class IMP_Prefs_Ui
 
             switch ($ui->vars->flag_action) {
             case 'delete':
-                if ($ui->vars->flag_data == ('bg_' . $md5)) {
-                    $imp_flags->deleteFlag($key);
+                if (($ui->vars->flag_data == ('bg_' . $md5)) &&
+                    $imp_flags->deleteFlag($key)) {
                     $GLOBALS['notification']->push(sprintf(_("Deleted flag \"%s\"."), $val['l']), 'horde.success');
                 }
                 break;