Added functionality to set dirty form flag on 'Add' and 'Remove' button.

gocd · Jun 16, 2015 · 9d764cb · 9d764cb
1 parent daf684e
commit 9d764cb
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/server/webapp/WEB-INF/rails.new/app/assets/javascripts/environment_variable_edit.js b/server/webapp/WEB-INF/rails.new/app/assets/javascripts/environment_variable_edit.js
@@ -80,7 +80,7 @@ var EnvironmentVariableEdit = function (options) {
     })
 };
 
-var EnvironmentVariableAddRemove = function(parentElement) {
+var EnvironmentVariableAddRemove = function(parentElement, options) {
   var $ = jQuery;
 
   var addButton = $(parentElement).find('.add_item');
@@ -93,12 +93,19 @@ var EnvironmentVariableAddRemove = function(parentElement) {
   addButton.on('click.environmentVariableTemplateCopy', function(evt){
     evt.preventDefault();
     addRowTo.append(template);
+    if(options.onAdd){
+      options.onAdd(addButton);
+    }
   });
 
   $(parentElement).on('click.environmentVariableTemplateCopy', '.delete_parent', function(evt){
     evt.preventDefault();
     var deleteLink = $(evt.target);
     deleteLink.parents('.environment-variable-edit-row').remove();
+    if(options.onRemove){
+      options.onRemove(deleteLink);
+    }
+
   });
 
 };
diff --git a/server/webapp/WEB-INF/rails.new/app/views/admin/shared/_name_value.html.erb b/server/webapp/WEB-INF/rails.new/app/views/admin/shared/_name_value.html.erb
@@ -89,7 +89,14 @@
   (function ($) {
     $(function () {
       EnvironmentVariableEdit({tableRowSelector: '.environment-variable-edit-row'});
-      EnvironmentVariableAddRemove($('#<%=template_row_id_prefix-%>'));
+      EnvironmentVariableAddRemove($('#<%=template_row_id_prefix-%>'), {
+        onAdd: function(addButton){
+          addButton.parents('form.dirtyform').data('dirty', true);
+        },
+        onRemove: function(deleteLink){
+          deleteLink.parents('form.dirtyform').data('dirty', true);
+        }
+      });
     });
   })(jQuery);
 </script>