enable deletion of supply items from an issue #19

Author: jkraemer

app/helpers/supply_items_helper.rb

@@ -9,12 +9,13 @@ def issue_supply_item_form_tags(issue_supply_items)
   def issue_supply_item_form_tag(issue_supply_item, label: false)
     supply_item = issue_supply_item.supply_item
     id = dom_id supply_item
-    content_tag :p, id: "#{id}_wrap" do
+    content_tag :p, id: "#{id}_wrap", class: 'issue_supply_item_wrap' do
       tags = [
         content_tag(:label, supply_item.name, for: id, class: "supply-item"),
         text_field_tag('issue[issue_supply_items_attributes][][quantity]',
                        issue_supply_item.quantity, id: id, size: 3),
         content_tag(:span, supply_item.unit_name, class: 'unit'),
+        link_to('', '#', class: 'icon icon-del'),
         hidden_field_tag('issue[issue_supply_items_attributes][][supply_item_id]',  supply_item.id)
       ]
       unless issue_supply_item.new_record?

app/views/issue_supply_items/create.js.erb

@@ -8,6 +8,7 @@
       quantity = 0;
     }
     wrap.find('input[type=text]').val(quantity + <%= i.quantity %>);
+    wrap.show(); // it may have been hidden by a previous click on delete
   } else {
     $('#supply_items_inputs').append('<%= j issue_supply_item_form_tag(i) %>');
   }

assets/javascripts/supply.js

@@ -35,8 +35,8 @@ function observeResourceItemSearchfield(url) {
   });
 }
 
+// issue resource item deletion
 $(document).on('click', '.issue_resource_item_wrap a.icon-del', function(e){
-
   var wrapper = $(this).parent();
   if(wrapper.find('input[name="issue[issue_resource_items_attributes][][id]"]').val() == '') {
     wrapper.remove();
@@ -47,4 +47,14 @@ $(document).on('click', '.issue_resource_item_wrap a.icon-del', function(e){
   return false;
 });
 
+// issue supply item deletion
+$(document).on('click', '.issue_supply_item_wrap a.icon-del', function(e){
+  var wrapper = $(this).parent();
+  // set quantity to zero
+  wrapper.find('input[name="issue[issue_supply_items_attributes][][quantity]"]').val('0');
+  // hide the whole item
+  wrapper.hide();
+  return false;
+});
+