azkaban-aplcache

fix bug #1481: avoid duplicate properties when closing/reopening

9/21/2017 2:20:25 PM

Details

diff --git a/azkaban-web-server/src/web/js/azkaban/view/flow-execute-dialog.js b/azkaban-web-server/src/web/js/azkaban/view/flow-execute-dialog.js
index 5d85585..1175ee5 100644
--- a/azkaban-web-server/src/web/js/azkaban/view/flow-execute-dialog.js
+++ b/azkaban-web-server/src/web/js/azkaban/view/flow-execute-dialog.js
@@ -162,7 +162,7 @@ azkaban.FlowExecuteDialogView = Backbone.View.extend({
       $('#queueLevel').val(queueLevel);
     }
 
-    if (flowParams) {
+    if (flowParams && $(".editRow").length == 0) {
       for (var key in flowParams) {
         editTableView.handleAddRow({
           paramkey: key,
diff --git a/azkaban-web-server/src/web/js/azkaban/view/job-edit.js b/azkaban-web-server/src/web/js/azkaban/view/job-edit.js
index 3a02762..eb91420 100644
--- a/azkaban-web-server/src/web/js/azkaban/view/job-edit.js
+++ b/azkaban-web-server/src/web/js/azkaban/view/job-edit.js
@@ -86,12 +86,14 @@ azkaban.JobEditView = Backbone.View.extend({
       mythis.overrideParams = overrideParams;
       mythis.generalParams = generalParams;
 
-      for (var okey in overrideParams) {
-        if (okey != 'type' && okey != 'dependencies') {
-          var row = handleAddRow();
-          var td = $(row).find('span');
-          $(td[0]).text(okey);
-          $(td[1]).text(overrideParams[okey]);
+      if (overrideParams && $(".editRow").length == 0)  {
+        for (var okey in overrideParams) {
+          if (okey != 'type' && okey != 'dependencies') {
+            var row = handleAddRow();
+            var td = $(row).find('span');
+            $(td[0]).text(okey);
+            $(td[1]).text(overrideParams[okey]);
+          }
         }
       }
     };