Confirm deletion of flags

diff --git a/imp/js/src/flagmanagement.js b/imp/js/src/flagmanagement.js
index 92ef8e7..4acce0b 100644 (file)
--- a/imp/js/src/flagmanagement.js
+++ b/imp/js/src/flagmanagement.js
@@ -6,7 +6,7 @@
  */
 
 var ImpFlagmanagement = {
-    // Variables set by other code: new_prompt
+    // Variables set by other code: confirm_delete, new_prompt
 
     addFlag: function()
     {
@@ -45,7 +45,9 @@ var ImpFlagmanagement = {
             }
 
             if (elt.hasClassName('flagdelete')) {
-                this._sendData('delete', elt.previous('INPUT').readAttribute('id'));
+                if (window.confirm(this.confirm_delete)) {
+                    this._sendData('delete', elt.previous('INPUT').readAttribute('id'));
+                }
                 e.stop();
                 return;
             }

diff --git a/imp/lib/prefs.php b/imp/lib/prefs.php
index 711b6e8..0a17348 100644 (file)
--- a/imp/lib/prefs.php
+++ b/imp/lib/prefs.php
@@ -255,7 +255,8 @@ case 'flags':
     Horde::addScriptFile('flagmanagement.js', 'imp', true);
 
     Horde::addInlineScript(array(
-        'ImpFlagmanagement.new_prompt = ' . Horde_Serialize::serialize(_("Please enter the label for the new flag:"), Horde_Serialize::JSON, Horde_Nls::getCharset())
+        'ImpFlagmanagement.new_prompt = ' . Horde_Serialize::serialize(_("Please enter the label for the new flag:"), Horde_Serialize::JSON, Horde_Nls::getCharset()),
+        'ImpFlagmanagement.confirm_delete = ' . Horde_Serialize::serialize(_("Are you sure you want to delete this flag?"), Horde_Serialize::JSON, Horde_Nls::getCharset())
     ));
     break;
 }