From 51a95575f7e2cd2c5a098de990363cee78faf649 Mon Sep 17 00:00:00 2001
From: Henry <akhenry@gmail.com>
Date: Fri, 7 Oct 2016 14:51:59 -0700
Subject: [PATCH] [Time Conductor] Refactored time of interest as optional
 generic behavior of MctTable

---
 .../table/res/templates/historical-table.html |  1 +
 .../table/res/templates/mct-table.html        |  2 +-
 .../controllers/HistoricalTableController.js  | 14 +++---
 .../src/controllers/MCTTableController.js     | 50 ++++++++++++++++---
 .../controllers/TelemetryTableController.js   | 31 ++----------
 .../features/table/src/directives/MCTTable.js |  4 +-
 6 files changed, 58 insertions(+), 44 deletions(-)

diff --git a/platform/features/table/res/templates/historical-table.html b/platform/features/table/res/templates/historical-table.html
index 168886e59d..f0176d1649 100644
--- a/platform/features/table/res/templates/historical-table.html
+++ b/platform/features/table/res/templates/historical-table.html
@@ -2,6 +2,7 @@
      ng-class="{'loading': loading}">
     <mct-table
         headers="headers"
+        time-columns="timeColumns"
         rows="rows"
         enableFilter="true"
         enableSort="true"
diff --git a/platform/features/table/res/templates/mct-table.html b/platform/features/table/res/templates/mct-table.html
index 054ec54e94..9a7cdc1347 100644
--- a/platform/features/table/res/templates/mct-table.html
+++ b/platform/features/table/res/templates/mct-table.html
@@ -52,7 +52,7 @@
         <!--ng-class="{ 'l-toi pinned': false }"-->
         <!--ng-click="dummyUnpin()" -->
             <tr ng-repeat="visibleRow in visibleRows track by visibleRow.rowIndex"
-                class="{{visibleRow.contents.cssClass}}"
+                ng-class="{ 'l-toi pinned': visibleRow.rowIndex === toiRowIndex }"
                 ng-click="table.onRowClick($event, visibleRow.contents)"
             ng-style="{
                 top: visibleRow.offsetY + 'px',
diff --git a/platform/features/table/src/controllers/HistoricalTableController.js b/platform/features/table/src/controllers/HistoricalTableController.js
index 90e13db704..d8af8060de 100644
--- a/platform/features/table/src/controllers/HistoricalTableController.js
+++ b/platform/features/table/src/controllers/HistoricalTableController.js
@@ -36,7 +36,7 @@ define(
          * @param telemetryFormatter
          * @constructor
          */
-        function HistoricalTableController($scope, telemetryHandler, telemetryFormatter, $timeout, conductor) {
+        function HistoricalTableController($scope, telemetryHandler, telemetryFormatter, $timeout) {
             var self = this;
 
             this.$timeout = $timeout;
@@ -49,7 +49,7 @@ define(
                 }
             });
 
-            TableController.call(this, $scope, telemetryHandler, telemetryFormatter, conductor);
+            TableController.call(this, $scope, telemetryHandler, telemetryFormatter);
         }
 
         HistoricalTableController.prototype = Object.create(TableController.prototype);
@@ -59,7 +59,6 @@ define(
          * @private
          */
         HistoricalTableController.prototype.doneProcessing = function (rowData) {
-            //Set table rows to formatted data;
             this.$scope.rows = rowData;
             this.$scope.loading = false;
         };
@@ -110,9 +109,8 @@ define(
 
             //Process rows in a batch with size not exceeding a maximum length
             for (; i < end; i++) {
-                var datum = this.handle.makeDatum(telemetryObject, series, i);
-                this.data.push(datum);
-                rowData.push(this.table.getRowValues(telemetryObject, datum));
+                rowData.push(this.table.getRowValues(telemetryObject,
+                    this.handle.makeDatum(telemetryObject, series, i)));
             }
 
             //Done processing all rows for this object.
@@ -125,7 +123,7 @@ define(
             // before continuing processing
             this.timeoutHandle = this.$timeout(this.processTelemetryObjects.bind(this, objects, offset, end, rowData));
         };
-        
+
         /**
         * Populates historical data on scope when it becomes available from
         * the telemetry API
@@ -134,7 +132,7 @@ define(
             if (this.timeoutHandle) {
                 this.$timeout.cancel(this.timeoutHandle);
             }
-            this.data = [];
+
             this.timeoutHandle = this.$timeout(this.processTelemetryObjects.bind(this, this.handle.getTelemetryObjects(), 0, 0, []));
         };
 
diff --git a/platform/features/table/src/controllers/MCTTableController.js b/platform/features/table/src/controllers/MCTTableController.js
index 3db5d8267a..77ee1ffe67 100644
--- a/platform/features/table/src/controllers/MCTTableController.js
+++ b/platform/features/table/src/controllers/MCTTableController.js
@@ -12,7 +12,7 @@ define(
          * @param element
          * @constructor
          */
-        function MCTTableController($scope, $timeout, element, exportService) {
+        function MCTTableController($scope, $timeout, element, exportService, formatService, conductor) {
             var self = this;
 
             this.$scope = $scope;
@@ -24,6 +24,9 @@ define(
             this.thead = element.find('thead');
             this.tbody = element.find('tbody');
             this.$scope.sizingRow = {};
+            this.conductor = conductor;
+            this.toiFormatter = undefined;
+            this.formatService = formatService;
 
             //Bind all class functions to 'this'
             Object.keys(MCTTableController.prototype).filter(function (key) {
@@ -77,6 +80,7 @@ define(
                     $scope.sortDirection = undefined;
                 }
                 self.setRows($scope.rows);
+                self.setTimeOfInterest(self.conductor.timeOfInterest());
             };
 
             /*
@@ -99,8 +103,29 @@ define(
              */
             $scope.resize = this.setElementSizes.bind(this);
 
+
+            // Time conductor integration
+            if (this.$scope.timeColumns) {
+                this.conductor.on('timeSystem', this.changeTimeSystem);
+                this.conductor.on('timeOfInterest', this.setTimeOfInterest);
+                $scope.$on('$destroy', function () {
+                    this.conductor.off('timeSystem', this.changeTimeSystem);
+                    this.conductor.off('timeOfInterest', this.setTimeOfInterest);
+                }.bind(this));
+
+                // If time system defined, set initially
+                if (conductor.timeSystem()) {
+                    this.changeTimeSystem(conductor.timeSystem());
+                }
+            }
+
         }
 
+        MCTTableController.prototype.changeTimeSystem = function () {
+            var format = this.conductor.timeSystem().formats()[0];
+            this.toiFormatter = this.formatService.getFormat(format);
+        };
+
         /**
          * If auto-scroll is enabled, this function will scroll to the
          * bottom of the page
@@ -308,7 +333,7 @@ define(
                 return min; // Element is not in array, min gives direction
             }
 
-            switch (this.sortComparator(searchElement[this.$scope.sortColumn].text,
+            switch (this.sortComparator(searchElement,
                 searchArray[sampleAt][this.$scope.sortColumn].text)) {
                 case -1:
                     return this.binarySearch(searchArray, searchElement, min,
@@ -332,7 +357,7 @@ define(
                 index = array.length;
             } else {
                 //Sort is enabled, perform binary search to find insertion point
-                index = this.binarySearch(array, element, 0, array.length - 1);
+                index = this.binarySearch(array, element[this.$scope.sortColumn].text, 0, array.length - 1);
             }
             if (index === -1) {
                 array.unshift(element);
@@ -496,11 +521,6 @@ define(
             this.resize(newRows).then(this.setVisibleRows.bind(this));
         };
 
-        MCTTableController.prototype.onRowClick = function (event, row) {
-            var index = this.$scope.rows.indexOf(row);
-            this.$scope.onRowClick({event: event, rowIndex:index, sortColumn: this.$scope.sortColumn, sortDirection: this.$scope.sortDirection});
-        };
-
         /**
          * Applies user defined filters to rows. These filters are based on
          * the text entered in the search areas in each column.
@@ -538,6 +558,20 @@ define(
             return rowsToFilter.filter(matchRow.bind(null, filters));
         };
 
+        /**
+         * Update rows with new data.  If filtering is enabled, rows
+         * will be sorted before display.
+         */
+        MCTTableController.prototype.setTimeOfInterest = function (newTOI) {
+            if (this.$scope.timeColumns.indexOf(this.$scope.sortColumn) !== -1) {
+                var formattedTOI = this.toiFormatter.format(newTOI);
+                var rowsLength = this.$scope.displayRows.length;
+                // searchElement, min, max
+                this.$scope.toiRowIndex = this.binarySearch(this.$scope.displayRows, formattedTOI, 0, rowsLength);
+            } else {
+                this.$scope.toiRowIndex = -1;
+            }
+        };
 
         return MCTTableController;
     }
diff --git a/platform/features/table/src/controllers/TelemetryTableController.js b/platform/features/table/src/controllers/TelemetryTableController.js
index 3845026f8e..1f105fb75d 100644
--- a/platform/features/table/src/controllers/TelemetryTableController.js
+++ b/platform/features/table/src/controllers/TelemetryTableController.js
@@ -26,10 +26,9 @@
  */
 define(
     [
-        '../TableConfiguration',
-        '../DomainColumn'
+        '../TableConfiguration'
     ],
-    function (TableConfiguration, DomainColumn) {
+    function (TableConfiguration) {
 
         /**
          * The TableController is responsible for getting data onto the page
@@ -57,7 +56,6 @@ define(
                 telemetryFormatter);
             this.changeListeners = [];
             this.conductor = conductor;
-            this.data = [];
 
             $scope.rows = [];
 
@@ -67,28 +65,13 @@ define(
                 self.registerChangeListeners();
             });
 
+            this.destroy = this.destroy.bind(this);
+
             // Unsubscribe when the plot is destroyed
             this.$scope.$on("$destroy", this.destroy);
+            this.$scope.timeColumns = ['Time'];
         }
 
-        TelemetryTableController.prototype.onRowClick = function (event, rowIndex, sortBy, sortOrder) {
-            var datum = this.data[rowIndex];
-
-            if (event.altKey) {
-                console.log("selected: " + this.$scope.rows[rowIndex]);
-                //Is column one that we can use to set time of interest?
-                var domainColumn = this.table.columns.filter(function (column) {
-                    return column instanceof DomainColumn &&
-                        column.getTitle() === sortBy;
-                })[0];
-                if (domainColumn) {
-                    var timeOfInterest = datum[domainColumn.domainMetadata.key];
-                    this.conductor.timeOfInterest(timeOfInterest);
-                }
-            }
-
-        };
-
         /**
          * @private
          */
@@ -210,10 +193,6 @@ define(
             });
         };
 
-        TelemetryTableController.prototype.changeTimeOfInterest = function (toi) {
-
-        }
-
         return TelemetryTableController;
     }
 );
diff --git a/platform/features/table/src/directives/MCTTable.js b/platform/features/table/src/directives/MCTTable.js
index d138c8ac97..529b22d024 100644
--- a/platform/features/table/src/directives/MCTTable.js
+++ b/platform/features/table/src/directives/MCTTable.js
@@ -86,6 +86,8 @@ define(
                     '$timeout',
                     '$element',
                     'exportService',
+                    'formatService',
+                    'timeConductor',
                     MCTTableController
                 ],
                 controllerAs: "table",
@@ -95,7 +97,7 @@ define(
                     enableFilter: "=?",
                     enableSort: "=?",
                     autoScroll: "=?",
-                    onRowClick: "&"
+                    timeColumns: "=?"
                 }
             };
         }