NotesBrowser: confirm deletion dialog

dradis · May 24, 2012 · e848dd0 · e848dd0
1 parent 4bc1f9c
commit e848dd0
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/app/assets/javascripts/dx/dradis.notes.NotesBrowserPanel.js b/app/assets/javascripts/dx/dradis.notes.NotesBrowserPanel.js
@@ -220,8 +220,19 @@ dradis.notes.Grid=Ext.extend(Ext.grid.EditorGridPanel, {
             itemclick: function(item){
               var s1 = item.parentMenu.grid.getStore();
               var s2 = item.parentMenu.grid.store;
-              item.parentMenu.grid.getStore().remove( item.parentMenu.record );
+
-              item.parentMenu.grid.fireEvent('modified');
+              Ext.Msg.show({
+                title: 'Confirm deletion',
+                msg: 'You have chosen to delete these notes, there is no going back. Continue?',
+                buttons: Ext.Msg.OKCANCEL,
+                fn: function(button){
+                  if (button == 'ok') {
+                    item.parentMenu.grid.getStore().remove( item.parentMenu.record );
+                    item.parentMenu.grid.fireEvent('modified');
+                  }
+                },
+                icon: Ext.MessageBox.QUESTION
+              });
             }
           }
       })