openerp-community-reviewer team mailing list archive

[Romain Deheele - Camptocamp <romain.deheele@xxxxxxxxxxxxxx>]

Romain Deheele - Camptocamp has proposed merging lp:~camptocamp/sale-reports/typo_groups_id-rde into lp:sale-reports.

Requested reviews:
  Sale Core Editors (sale-core-editors)

For more details, see:
https://code.launchpad.net/~camptocamp/sale-reports/typo_groups_id-rde/+merge/206874

Hello,

Sorry, my previous commit has introduced a typo error:
on res_users, group field is named groups_id, not group_id.

Romain
-- 
https://code.launchpad.net/~camptocamp/sale-reports/typo_groups_id-rde/+merge/206874
Your team Sale Core Editors is requested to review the proposed merge of lp:~camptocamp/sale-reports/typo_groups_id-rde into lp:sale-reports.

=== modified file 'sale_order_webkit/report/sale_order.py'
--- sale_order_webkit/report/sale_order.py	2014-02-17 14:34:44 +0000
+++ sale_order_webkit/report/sale_order.py	2014-02-18 09:33:51 +0000
@@ -48,7 +48,7 @@
         except ValueError:
             #group named group_discount_per_so_line doesn't exist
             return False
-        groups = res_users_obj.browse(cr, uid, uid, context=context).group_id
+        groups = res_users_obj.browse(cr, uid, uid, context=context).groups_id
         return any(x for x in groups if x.id == group_id)
 
     def _get_company_vat(self):