Merge lp:~camptocamp/openerp-fiscal-rules/7.0-onchange_shop_id-1277125 into lp:openerp-fiscal-rules

[Guewen Baconnier @ Camptocamp]

[FIX] onchange_shop_id should call super to initialize all the fields properly

Fixes lp:1277125

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

OK, nice catch.

BTW, instead of changing the signature of the onchange_shop_id, in the future (v8?) I suggest we use web_context_tunnel here. In any case the sale.order shop_id is t be replaced by the sale team and the warehouse in v8, but you get the idea.

review: Approve

[Leonardo Pistone (lepistone)]

ok!

review: Approve (code review)

[Pedro Manuel Baeza (pedro.baeza)]

LGTM

review: Approve (code review)

=== modified file 'account_fiscal_position_rule_sale/sale.py'
--- account_fiscal_position_rule_sale/sale.py	2013-12-16 15:22:28 +0000
+++ account_fiscal_position_rule_sale/sale.py	2014-02-06 15:24:08 +0000
@@ -89,7 +89,8 @@
         if not context:
             context = {}
 
-        result = {'value': {}}
+        result = super(sale_order, self).onchange_shop_id(
+            cr, uid, ids, shop_id, context=context)
         if not shop_id or not partner_id:
             return result