azkaban-developers

diff --git a/src/java/azkaban/webapp/servlet/velocity/flowexecutionpanel.vm b/src/java/azkaban/webapp/servlet/velocity/flowexecutionpanel.vm
index 6895f48..f1d45a3 100644
--- a/src/java/azkaban/webapp/servlet/velocity/flowexecutionpanel.vm
+++ b/src/java/azkaban/webapp/servlet/velocity/flowexecutionpanel.vm
@@ -165,7 +165,7 @@
 												<tbody>
 													<tr id="addRow" class="addRow">
 														<td id="addRow-col" colspan="2">
-															<button type="button" class="btn btn-success btn-xs">Add Row</button>
+															<button type="button" class="btn btn-success btn-xs" id="add-btn">Add Row</button>
 														</td>
 													</tr>
 												</tbody>

diff --git a/src/java/azkaban/webapp/servlet/velocity/jobpage.vm b/src/java/azkaban/webapp/servlet/velocity/jobpage.vm
index e212556..61fd82e 100644
--- a/src/java/azkaban/webapp/servlet/velocity/jobpage.vm
+++ b/src/java/azkaban/webapp/servlet/velocity/jobpage.vm
@@ -196,7 +196,7 @@
 								<tbody>
 									<tr id="addRow">
 										<td id="addRow-col" colspan="2">
-											<button type="button" class="btn btn-xs btn-success">Add Row</button>
+											<button type="button" class="btn btn-xs btn-success" id="add-btn">Add Row</button>
 										</td>
 									</tr>
 								</tbody>

diff --git a/src/java/azkaban/webapp/servlet/velocity/slapanel.vm b/src/java/azkaban/webapp/servlet/velocity/slapanel.vm
index a4ed58b..e57f737 100644
--- a/src/java/azkaban/webapp/servlet/velocity/slapanel.vm
+++ b/src/java/azkaban/webapp/servlet/velocity/slapanel.vm
@@ -44,7 +44,7 @@
 									<tr id="addRow">
 										<td id="addRow-col" colspan="5">
 											<span class="addIcon"></span>
-											<button type="button" class="btn btn-xs btn-success">Add Row</button>
+											<button type="button" class="btn btn-xs btn-success" id="add-btn">Add Row</button>
 										</td>
 									</tr>
 								</tbody>

diff --git a/src/web/js/azkaban/view/flow-execute-dialog.js b/src/web/js/azkaban/view/flow-execute-dialog.js
index fa1717e..ce590ca 100644
--- a/src/web/js/azkaban/view/flow-execute-dialog.js
+++ b/src/web/js/azkaban/view/flow-execute-dialog.js
@@ -61,10 +61,10 @@ azkaban.FlowExecuteDialogView = Backbone.View.extend({
 		var editRows = $(".editRow");
 		for (var i = 0; i < editRows.length; ++i) {
 			var row = editRows[i];
-			var td = $(row).find('td');
+			var td = $(row).find('span');
 			var key = $(td[0]).text();
 			var val = $(td[1]).text();
-			
+
 			if (key && key.length > 0) {
 				flowOverride[key] = val;
 			}
@@ -103,6 +103,7 @@ azkaban.FlowExecuteDialogView = Backbone.View.extend({
 		
 		return executingData;
 	},
+
 	changeFlowInfo: function() {
 		var successEmails = this.model.get("successEmails");
 		var failureEmails = this.model.get("failureEmails");
@@ -285,6 +286,7 @@ azkaban.FlowExecuteDialogView = Backbone.View.extend({
 		};
 		$.get(requestURL, requestData, successHandler, "json");
 	},
+
 	assignInitialStatus: function(data, statusData) {
 		// Copies statuses over from the previous execution if it exists.
 		var statusNodeMap = statusData.nodeMap;
@@ -300,6 +302,7 @@ azkaban.FlowExecuteDialogView = Backbone.View.extend({
 			}
 		}
 	},
+
 	handleExecuteFlow: function(evt) {
 		console.log("click schedule button.");
 		var executeURL = contextURL + "/executor";
@@ -311,9 +314,9 @@ azkaban.FlowExecuteDialogView = Backbone.View.extend({
 var editTableView;
 azkaban.EditTableView = Backbone.View.extend({
 	events: {
-		"click table .addRow": "handleAddRow",
+		"click table #add-btn": "handleAddRow",
 		"click table .editable": "handleEditColumn",
-		"click table .removeIcon": "handleRemoveColumn"
+		"click table .remove-btn": "handleRemoveColumn"
 	},
 
 	initialize: function(setting) {
@@ -391,7 +394,7 @@ azkaban.EditTableView = Backbone.View.extend({
 			}
 		});
 	},
-	
+
 	handleRemoveColumn: function(evt) {
 		var curTarget = evt.currentTarget;
 		// Should be the table
@@ -485,7 +488,7 @@ var executableGraphModel;
  * Disable jobs that need to be disabled
  */
 var disableFinishedJobs = function(data) {
-	for (var i=0; i < data.nodes.length; ++i) {
+	for (var i = 0; i < data.nodes.length; ++i) {
 		var node = data.nodes[i];
 		
 		if (node.status == "DISABLED" || node.status == "SKIPPED") {
@@ -522,7 +525,7 @@ var disableAll = function() {
 }
 
 var recurseTree = function(data, disabled, recurse) {
-	for (var i=0; i < data.nodes.length; ++i) {
+	for (var i = 0; i < data.nodes.length; ++i) {
 		var node = data.nodes[i];
 		node.disabled = disabled;
 		

diff --git a/src/web/js/azkaban/view/job-edit.js b/src/web/js/azkaban/view/job-edit.js
index ba037b0..c36ccfb 100644
--- a/src/web/js/azkaban/view/job-edit.js
+++ b/src/web/js/azkaban/view/job-edit.js
@@ -23,7 +23,7 @@ azkaban.JobEditView = Backbone.View.extend({
 		"click #set-btn": "handleSet",	
 		"click #cancel-btn": "handleCancel",
 		"click #close-btn": "handleCancel",
-		"click #addRow": "handleAddRow",
+		"click #add-btn": "handleAddRow",
 		"click table .editable": "handleEditColumn",
 		"click table .remove-btn": "handleRemoveColumn"
 	},

diff --git a/src/web/js/azkaban/view/schedule-sla.js b/src/web/js/azkaban/view/schedule-sla.js
index f2f316a..0fecd0b 100644
--- a/src/web/js/azkaban/view/schedule-sla.js
+++ b/src/web/js/azkaban/view/schedule-sla.js
@@ -21,7 +21,7 @@ azkaban.ChangeSlaView = Backbone.View.extend({
 		"click": "closeEditingTarget",
 		"click #set-sla-btn": "handleSetSla",	
 		"click #remove-sla-btn": "handleRemoveSla",
-		"click #addRow": "handleAddRow"
+		"click #add-btn": "handleAddRow"
 	},
 	
 	initialize: function(setting) {