From 9ec4dbe917a8d443effe629afcf3df7caa8a123f Mon Sep 17 00:00:00 2001
From: Rob Cutmore <robcutmore@gmail.com>
Date: Thu, 9 Jun 2016 07:59:54 -0400
Subject: [PATCH] Fix campaign group removal (fixes #249)

Every time a group was added, the event handler for removing group was
set for all rows in group table. This change ensures the event handler
is only set for the newly added group.
---
 static/js/app/campaigns.js | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/static/js/app/campaigns.js b/static/js/app/campaigns.js
index f9bb84b0..d98afd24 100644
--- a/static/js/app/campaigns.js
+++ b/static/js/app/campaigns.js
@@ -302,18 +302,23 @@ $(document).ready(function() {
             errorFlash("Error fetching campaigns")
         })
     $("#groupForm").submit(function() {
-            groupTable.row.add([
+            // Add row to group table.
+            var newRow = groupTable.row.add([
                 $("#groupSelect").val(),
                 '<span style="cursor:pointer;"><i class="fa fa-trash-o"></i></span>'
-            ]).draw()
-            $("#groupTable").on("click", "span>i.fa-trash-o", function() {
+            ]).draw().node();
+
+            // Set event handler for removing row from group table.
+            $(newRow).on("click", "span>i.fa-trash-o", function() {
                 groupTable.row($(this).parents('tr'))
                     .remove()
                     .draw();
-            })
+            });
+
+            // Clear user input.
             $("#groupSelect").val("");
             return false;
-        })
+        });
         // Create the group typeahead objects
     groupTable = $("#groupTable").DataTable({
         columnDefs: [{