New web UI, store highest channel number used

AUTHORS

@@ -29,3 +29,4 @@ Contributors:
   Stefan Krüger, added APA102 support to the SPI Plugin
   Tobi Schäfer, for the MacPort files
   Stefan S, improved timing with monotonic clock
+  Christian Mouttet, fix in new web-ui

README.developer

@@ -200,6 +200,9 @@ Javascript
 Javascript is used for the olad web UI. Instructions for building the
 javascript can be found in javascript/README.
 
+The sources for the new UI is located in javascript/new-src. The built artifact
+is copied to olad/www/new/js/app.min.js.
+
 Closure Compiler
 ----------------
 

javascript/new-src/src/controllers/universe.js

@@ -24,6 +24,7 @@ ola.controller('universeCtrl',
       'use strict';
       $scope.dmx = [];
       $scope.Universe = $routeParams.id;
+      $ola.resetHighestChannelNumberUsed();
 
       var interval = $interval(function() {
         $ola.get.Dmx($scope.Universe).then(function(data) {

javascript/new-src/src/factories/ola.js

@@ -22,6 +22,19 @@
 ola.factory('$ola', ['$http', '$window', 'OLA',
   function($http, $window, OLA) {
     'use strict';
+    var highestChannelNumberUsed = 0;
+
+    var updateHighestChannelNumberUsed = function(dmx) {
+      for (var channel = dmx.length; channel > highestChannelNumberUsed;
+           channel--) {
+
+        if ((dmx[channel - 1] > OLA.MIN_CHANNEL_VALUE) &&
+            (highestChannelNumberUsed < channel)) {
+          highestChannelNumberUsed = channel;
+        }
+      }
+    };
+
     // TODO(Dave_o): once olad supports json post data postEncode
     // can go away and the header in post requests too.
     var postEncode = function(data) {
@@ -52,20 +65,28 @@ ola.factory('$ola', ['$http', '$window', 'OLA',
       return i;
     };
     var dmxConvert = function(dmx) {
-      var strip = true;
       var integers = [];
-      for (var i = OLA.MAX_CHANNEL_NUMBER; i >= OLA.MIN_CHANNEL_NUMBER; i--) {
-        var value = channelValueCheck(dmx[i - 1]);
+      for (var channel = OLA.MAX_CHANNEL_NUMBER;
+           channel >= OLA.MIN_CHANNEL_NUMBER;
+           channel--) {
+
+        var value = channelValueCheck(dmx[channel - 1]);
         if (value > OLA.MIN_CHANNEL_VALUE ||
-          !strip ||
-          i === OLA.MIN_CHANNEL_NUMBER) {
-          integers[i - 1] = value;
-          strip = false;
+            channel <= highestChannelNumberUsed) {
+
+          integers[channel - 1] = value;
+
+          if (highestChannelNumberUsed < channel) {
+            highestChannelNumberUsed = channel;
+          }
         }
       }
       return integers.join(',');
     };
     return {
+      resetHighestChannelNumberUsed: function() {
+        highestChannelNumberUsed = 0;
+      },
       get: {
         ItemList: function() {
           return $http.get('/json/universe_plugin_list')
@@ -118,6 +139,7 @@ ola.factory('$ola', ['$http', '$window', 'OLA',
             }
           })
             .then(function(response) {
+              updateHighestChannelNumberUsed(response.data.dmx);
               return response.data;
             });
         },