Skip to content

Commit

Permalink
Merge pull request #507 from jpmorganchase/d3fc-build-fix-2
Browse files Browse the repository at this point in the history
Fixed merge error in new plugin API access to `view.config`
  • Loading branch information
texodus authored Apr 1, 2019
2 parents 9c9aec8 + 90c8172 commit c6381c9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/perspective-viewer-d3fc/src/js/plugin/plugin.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,14 +35,14 @@ function drawChart(chart) {
// FIXME: super tight coupling to private viewer methods
const aggregates = this._get_view_aggregates();
const hidden = this._get_view_hidden(aggregates);
const filter = this._view._config.filter;

const [tschema, json, config] = await Promise.all([this._table.schema(), view.to_json(), view.get_config()]);
if (task.cancelled) {
return;
}
const row_pivots = config.row_pivot;
const col_pivots = config.column_pivot;
const filter = config.filter;

const filtered = row_pivots.length > 0 ? json.filter(col => col.__ROW_PATH__ && col.__ROW_PATH__.length == row_pivots.length) : json;
const dataMap = !row_pivots.length ? (col, i) => ({...removeHiddenData(col, hidden), __ROW_PATH__: [i]}) : col => removeHiddenData(col, hidden);
Expand Down

0 comments on commit c6381c9

Please sign in to comment.