Fix bug: duplicate log messages when polling

otto-de · Jun 21, 2018 · 32adf30 · 32adf30
1 parent 9b26ebe
commit 32adf30
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/edison-jobs/src/main/resources/static/internal/js/logLoader.js b/edison-jobs/src/main/resources/static/internal/js/logLoader.js
@@ -16,10 +16,10 @@ function getLog(logIndex) {
             jobStatus.html("<span>UNKNOWN</span>");
         },
         success: function (data, textStatus, xhr) {
-            var newLogIndex = data.messages.length - 1;
+            var numberOfMessages = data.messages.length;
             var logWindow = $('.logWindow');
 
-            while (logIndex <= newLogIndex) {
+            while (logIndex < numberOfMessages) {
                 if (logIndex === 0) {
                     logWindow.empty();
                 }
@@ -37,7 +37,7 @@ function getLog(logIndex) {
             //Schedule further polling if still runnin'
             if (data.state === 'Running') {
                 setTimeout(function () {
-                    getLog(newLogIndex)
+                    getLog(logIndex)
                 }, 2000);
             } else {
                 var jobStatus = $('#job-status');