From 1bb6e178293b28a046db088550fd8e4d08734556 Mon Sep 17 00:00:00 2001 From: Henry Date: Mon, 11 Apr 2016 13:30:05 -0700 Subject: [PATCH] Minor code change to improve clarity --- .../features/table/src/controllers/MCTTableController.js | 7 +++---- platform/features/table/test/TableConfigurationSpec.js | 3 --- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/platform/features/table/src/controllers/MCTTableController.js b/platform/features/table/src/controllers/MCTTableController.js index 9b65613516..1470a9e47c 100644 --- a/platform/features/table/src/controllers/MCTTableController.js +++ b/platform/features/table/src/controllers/MCTTableController.js @@ -450,8 +450,7 @@ define( * occurred. * @private */ - MCTTableController.prototype.resize = function (rows){ - + MCTTableController.prototype.resize = function (rows) { this.$scope.sizingRow = this.buildLargestRow(rows); return this.$timeout(this.setElementSizes.bind(this)); }; @@ -468,7 +467,7 @@ define( if (this.$scope.enableSort) { displayRows = this.sortRows(displayRows.slice(0)); } - this.$scope.displayRows = displayRows; + return displayRows; }; /** @@ -481,7 +480,7 @@ define( return; } - this.filterAndSort(newRows || []); + this.$scope.displayRows = this.filterAndSort(newRows || []); this.resize(newRows).then(this.setVisibleRows.bind(this)); }; diff --git a/platform/features/table/test/TableConfigurationSpec.js b/platform/features/table/test/TableConfigurationSpec.js index 79583d09f8..bfc6f50edb 100644 --- a/platform/features/table/test/TableConfigurationSpec.js +++ b/platform/features/table/test/TableConfigurationSpec.js @@ -191,9 +191,6 @@ define( expect(mockTelemetryFormatter.formatRangeValue).toHaveBeenCalled(); }); }); - /** - * TODO: Add test for saving column config - */ }); }); }