fix confirmation message for multiple line delete, Mantis 25468

2 files changed, 24 insertions, 23 deletions

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 = $('<form method="post" action="' + href + '"></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 = $('<form method="post" action="' + href + '"></form>')
-    target = link.attr('target')
+      metadata_input = '<input name="_method" value="' + method + '" type="hidden" />'
 
-    metadata_input = '<input name="_method" value="' + method + '" type="hidden" />'
+      if csrf_param? and csrf_token?
+        metadata_input += '<input name="' + csrf_param + '" value="' + csrf_token + '" type="hidden" />'
 
-    if csrf_param? and csrf_token? 
-      metadata_input += '<input name="' + csrf_param + '" value="' + csrf_token + '" type="hidden" />'
+      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)
diff --git a/app/views/lines/index.html.erb b/app/views/lines/index.html.erb
index 73bb69398..4eb9c29f8 100644
--- a/app/views/lines/index.html.erb
+++ b/app/views/lines/index.html.erb
@@ -2,8 +2,8 @@
 
 <%= search_form_for @q, :url => referential_lines_path(@referential), :html => {:method => :get, class: "form-inline", :id => "search", role: "form"} do |f| %>
 <div class="panel panel-default">
-  <div class="panel-heading">   
-    <div class="input-group col-md-9">   
+  <div class="panel-heading">
+    <div class="input-group col-md-9">
       <%= f.text_field :name_or_number_cont, :placeholder => "#{t('.name_or_number')}", :class => 'form-control' %>
       <div class="input-group-btn">
         <button class="btn btn-default" type="submit"><i class="fa fa-search"></i></button>
@@ -13,8 +13,8 @@
       <i class="fa fa-plus"></i> <%= "#{t('.advanced_search')}" %>
     </a>
   </div>
-  
-  <div id="advanced_search" class="panel-collapse collapse">          
+
+  <div id="advanced_search" class="panel-collapse collapse">
     <div class="panel-body">
       <%= f.select(:network_id_eq, @referential.networks.collect {|n| [ n.name, n.id ] }, {include_blank: "#{t('.all_networks')}"}, { :class => 'form-control' }) %>
       <%= f.select(:company_id_eq, @referential.companies.collect {|c| [ c.name, c.id ] }, {include_blank: "#{t('.all_companies')}"}, { :class => 'form-control' }) %>
@@ -49,7 +49,7 @@
     <a class="disable" href="#"><%= t(".multi_selection_disable") %></a>
 
     <ul class="actions">
-      <%= link_to t(".delete_selected"), referential_lines_path(@referential), "data-multiple-method" => "delete", :class => "remove", :confirm => t("lines.actions.destroy_selection_confirm") %>
+      <%= link_to t(".delete_selected"), referential_lines_path(@referential), "data-multiple-method" => "delete", :class => "remove", "confirmation-text" => t("lines.actions.destroy_selection_confirm") %>
     </ul>
 
     <a class="select_all" href="#"><%= t(".select_all") %></a> | <a class="deselect_all" href="#"><%= t(".deselect_all") %></a>