From f6b44e1f7745ae7bd4aa775564478f876d14229c Mon Sep 17 00:00:00 2001 From: Marc Florisson Date: Wed, 13 Aug 2014 10:36:38 +0200 Subject: fix confirmation message for multiple line delete, Mantis 25468 --- .../javascripts/multiple_selection.js.coffee | 37 +++++++++++----------- 1 file changed, 19 insertions(+), 18 deletions(-) (limited to 'app/assets/javascripts') diff --git a/app/assets/javascripts/multiple_selection.js.coffee b/app/assets/javascripts/multiple_selection.js.coffee index 1ff473c4d..86b260fb7 100644 --- a/app/assets/javascripts/multiple_selection.js.coffee +++ b/app/assets/javascripts/multiple_selection.js.coffee @@ -1,5 +1,5 @@ jQuery -> - enable_multiple_selection = (event) -> + enable_multiple_selection = (event) -> event.preventDefault() $('#multiple_selection_menu div.enabled').show() $('#multiple_selection_menu div.disabled').hide() @@ -7,7 +7,7 @@ jQuery -> $('#multiple_selection_menu a.enable').click(enable_multiple_selection) - disable_multiple_selection = (event) -> + disable_multiple_selection = (event) -> event.preventDefault() $('#multiple_selection_menu div.enabled').hide() $('#multiple_selection_menu div.disabled').show() @@ -15,13 +15,13 @@ jQuery -> $('#multiple_selection_menu a.disable').click(disable_multiple_selection) - select_all = (event) -> + select_all = (event) -> event.preventDefault() $('input[type=checkbox].multiple_selection').prop("checked", true) $('#multiple_selection_menu a.select_all').click(select_all) - deselect_all = (event) -> + deselect_all = (event) -> event.preventDefault() $('input[type=checkbox].multiple_selection').prop("checked", false) @@ -30,24 +30,25 @@ jQuery -> handle_multiple_action = (event) -> event.preventDefault() link = $(event.target) + r = confirm( link.attr( "confirmation-text" ) ); + if (r == true) + href = link.attr("href") + method = link.data('multiple-method') + csrf_token = $('meta[name=csrf-token]').attr('content') + csrf_param = $('meta[name=csrf-param]').attr('content') + form = $('
') + target = link.attr('target') - href = link.attr("href") - method = link.data('multiple-method') - csrf_token = $('meta[name=csrf-token]').attr('content') - csrf_param = $('meta[name=csrf-param]').attr('content') - form = $('
') - target = link.attr('target') + metadata_input = '' - metadata_input = '' + if csrf_param? and csrf_token? + metadata_input += '' - if csrf_param? and csrf_token? - metadata_input += '' + form.append($(input).clone()) for input in $('input[type=checkbox].multiple_selection:checked') - form.append($(input).clone()) for input in $('input[type=checkbox].multiple_selection:checked') - - form.attr('target', target) if target? + form.attr('target', target) if target? - form.hide().append(metadata_input).appendTo('body') - form.submit() + form.hide().append(metadata_input).appendTo('body') + form.submit() $('#multiple_selection_menu .actions a.remove').click(handle_multiple_action) -- cgit v1.2.3