Merge pull request #83 from jpmorganchase/fix-viewer-aggregates

Fix setting the aggregate attribute on perspective-viewer
finos · Apr 3, 2018 · c5c151c · c5c151c
2 parents 3425dfe + 88e9000
commit c5c151c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/perspective-viewer/src/js/view.js b/packages/perspective-viewer/src/js/view.js
@@ -748,15 +748,16 @@ registerElement(template, {
      */
     aggregates: {
         set: function () {
-            let show = JSON.stringify(this.getAttribute('aggregates'));
-            let lis = Array.prototype.slice.call(this.querySelectorAll("#side_panel perspective-row"));
+            let show = JSON.parse(this.getAttribute('aggregates'));
+            let lis = Array.prototype.slice.call(this.querySelectorAll("#active_columns perspective-row"));
             lis.map((x, ix) => {
                 let agg = show[x.getAttribute('name')];
                 if (agg) {
                     x.setAttribute('aggregate', agg);
                 }
             });
             this.dispatchEvent(new Event('config-update'));
+            this._update();
         }
     },