summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/views/pivot
diff options
context:
space:
mode:
authorJigar Patel <jat@openerp.com>2017-07-19 14:18:12 +0530
committerGéry Debongnie <ged@odoo.com>2017-07-19 10:50:21 +0200
commitb63e22f50ea8cce8deaa7dcc0d1c5649d2d2880b (patch)
tree97de8a98ea4e12f1882298343f01b00eb5e7b880 /addons/web/static/src/js/views/pivot
parentefd158523438cebd96f9f2e7243873f222a97fea (diff)
[FIX] web: Groupby not working on Pivot and Graph view
The groupby property was wrongly set in some cases: if a graph/pivot view is defined as a groupby on field F, but there is another groupby key G in the context, it will take F instead of G (at initial load).
Diffstat (limited to 'addons/web/static/src/js/views/pivot')
-rw-r--r--addons/web/static/src/js/views/pivot/pivot_model.js5
1 files changed, 3 insertions, 2 deletions
diff --git a/addons/web/static/src/js/views/pivot/pivot_model.js b/addons/web/static/src/js/views/pivot/pivot_model.js
index 49c00663d47..d74af7fee22 100644
--- a/addons/web/static/src/js/views/pivot/pivot_model.js
+++ b/addons/web/static/src/js/views/pivot/pivot_model.js
@@ -229,11 +229,12 @@ var PivotModel = AbstractModel.extend({
this.data = {
domain: params.domain,
context: _.extend({}, session.user_context, params.context),
- groupedBy: params.rowGroupBys,
+ groupedBy: params.groupedBy,
colGroupBys: params.colGroupBys,
measures: params.measures,
sorted_column: {},
};
+ this.defaultGroupedBy = params.groupedBy;
return this._loadData();
},
/**
@@ -250,7 +251,7 @@ var PivotModel = AbstractModel.extend({
this.data.domain = this.initialDomain;
}
if ('groupBy' in params) {
- this.data.groupedBy = params.groupBy;
+ this.data.groupedBy = params.groupBy.length ? params.groupBy : this.defaultGroupedBy;
}
if (!this.data.has_data) {
return this._loadData();