Merge lp:~openerp-dev/openerp-web/7.0-opw-593605-cpa into lp:openerp-web/7.0

[Chirag Patel (OpenERP)]

Hello,

Group_by passed in wrong model.

Demo: Project
1) Group by 'Parent'
2) select list view.
3) Open any record.
4) Click on 'Timesheets' or 'Tasks' button.

Observed :

assert groupby_def and groupby_def._classic_write, "Fields in 'groupby' must be regular database-persisted fields (no function or related fields), or function fields with store=True"

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

Thanks

[Bern Schneider (mia-11ngola)]

Hi,

Thanks for this patch, this resolve the bug but all group by in dashbord are lost with this patch!!

Is there more solutions about that?

Thx,
Vitasoa

Unmerged revisions

3974. By Chirag Patel <email address hidden> on 2013-06-13

[FIX] Fixed group_by passed in wrong model.

=== modified file 'addons/web/static/src/js/views.js'
--- addons/web/static/src/js/views.js	2013-05-13 11:22:02 +0000
+++ addons/web/static/src/js/views.js	2013-06-17 12:42:28 +0000
@@ -776,7 +776,7 @@
                 self.dataset.model, results.context, results.domain);
             var groupby = results.group_by.length
                         ? results.group_by
-                        : action_context.group_by;
+                        : (self.dataset.model === action_context.active_model) ? action_context.group_by : [];
             if (_.isString(groupby)) {
                 groupby = [groupby];
             }