From 6cb2df64ddb7b89e0bca9fa368cd27a44a8dbc72 Mon Sep 17 00:00:00 2001 From: Ingo Schommer Date: Sun, 16 Sep 2007 02:18:18 +0000 Subject: [PATCH] elofgren: Added delete confirmation for items in Newsletter left tree. As part of fix for gsoc trac ticekt: #51 Warning is needed before items are deleted from security & newsletter tree (merged from branches/gsoc) git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@42040 467b73ca-7a2a-4603-9d3b-597d59a354a9 --- javascript/NewsletterAdmin_left.js | 39 +++++++++++++++--------------- 1 file changed, 20 insertions(+), 19 deletions(-) diff --git a/javascript/NewsletterAdmin_left.js b/javascript/NewsletterAdmin_left.js index 3c0b5eec..21059ec5 100755 --- a/javascript/NewsletterAdmin_left.js +++ b/javascript/NewsletterAdmin_left.js @@ -131,25 +131,26 @@ deletedraft = { } if(csvIDs) { - $('deletedrafts_options').elements.csvIDs.value = csvIDs; - - Ajax.SubmitForm('deletedrafts_options', null, { - onSuccess : function(response) { - Ajax.Evaluator(response); - - var sel; - if((sel = $('sitetree').selected) && sel.parentNode) sel.addNodeClass('current'); -// else $('Form_EditForm').innerHTML = ""; - - treeactions.closeSelection($('deletedrafts')); - }, - onFailure : function(response) { - errorMessage('Error deleting drafts', response); - } - }); - - $('deletedrafts').getElementsByTagName('button')[0].onclick(); - + if(confirm("Do you really want to these Newsletter items?")) { + $('deletedrafts_options').elements.csvIDs.value = csvIDs; + + Ajax.SubmitForm('deletedrafts_options', null, { + onSuccess : function(response) { + Ajax.Evaluator(response); + + var sel; + if((sel = $('sitetree').selected) && sel.parentNode) sel.addNodeClass('current'); + // else $('Form_EditForm').innerHTML = ""; + + treeactions.closeSelection($('deletedrafts')); + }, + onFailure : function(response) { + errorMessage('Error deleting drafts', response); + } + }); + + $('deletedrafts').getElementsByTagName('button')[0].onclick(); + } } else { alert("Please select at least 1 page."); }