Merge lp:~openerp-dev/openerp-web/7.0-opw-595221-fka into lp:openerp-web/7.0

[Foram Katharotiya (OpenERP)]

Hello,

To reproduce:
Go to Sale -> Customers and group by Country
choose customer and click on button "Quoation and Sales"

Fixed the issue of AssertionError: Fields in 'groupby' must be regular database-persisted fields (no function or related fields), or function fields with store=True.

Thanks,
FKA

[Csaba TOTH (tsabi)]

This is already fixed in MP 174106: https://code.launchpad.net/~openerp-dev/openerp-web/7.0-opw-594838-pna/+merge/174106 and in MP 169774: https://code.launchpad.net/~openerp-dev/openerp-web/7.0-opw-593605-cpa/+merge/169774

review: Disapprove (duplicate)

[Bern Schneider (mia-11ngola)]

Hello,

This patch resolve this bug but in dashboard, all filters are lost!!

Is there more solutions about that??

Thx,
Vitasoa

Unmerged revisions

4000. By Foram Katharotiya (OpenERP) on 2013-07-16

[FIX] AssertionError: Fields in 'groupby' must be regular database-persisted fields (no function or related fields), or function fields with store=True

=== modified file 'addons/web/static/src/js/views.js'
--- addons/web/static/src/js/views.js	2013-06-21 12:44:49 +0000
+++ addons/web/static/src/js/views.js	2013-07-18 07:45:35 +0000
@@ -788,9 +788,12 @@
         }).done(function (results) {
             self.dataset._model = new instance.web.Model(
                 self.dataset.model, results.context, results.domain);
+            var default_groupby = (self.dataset.model === action_context.active_model)
+                                ? action_context.group_by 
+                                : [];
             var groupby = results.group_by.length
                         ? results.group_by
-                        : action_context.group_by;
+                        : default_groupby;
             if (_.isString(groupby)) {
                 groupby = [groupby];
             }