Merge lp:~openerp-dev/openerp-web/7.0-opw-594838-pna into lp:openerp-web/7.0

[Pinakin Nayi (OpenERP)]

Hello Sir,

I fixed problem with context in views.

Steps to reproduce:
1. Open Sale Order list view.
2. Create a custom filter (Group the sale orders by Status and Order date). Set to be used by default
3. Create a Sale Order with this options set
  Shipping Policy - Deliver each product when available
  Create Invoice - On Delivery Order
4. Confirm sale order
5. Make a partial delivery
6. From Sale order list view open the created sale order
7. When opening delivery order from sale order error is thrown

Back ported from : https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-1101840-bth/+merge/167912

Thanks,
pna

[Csaba TOTH (tsabi)]

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

review: Disapprove (duplicate)

Unmerged revisions

3995. By Pinakin Nayi (OpenERP) on 2013-07-11

[FIX]views : If model and action_context's model are same then passed group by of action_context else []

=== 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-11 05:20:38 +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];
             }