Merge lp:~cmsa/openerp.pt-br-localiz/onchange_shop_id_arguments into lp:~openerp-brazil-core-team/openerp.pt-br-localiz/openerp.pt-br-localiz-v7.0

[Carlos Almeida]

Removi a redefinição na view e do método, pois não faz sentido redefinir este método só para chamar o super, se o super não tem estes parâmetros todos e o método não faz nada aqui com esses parâmetros...

[Renato Lima - http://www.akretion.com (renatonlima)]

Este método na localização existe para passar no contexto o campo fiscal_category_id onde ao chamar o super, passa pelo método account_fiscal_position_rule_sale que faz a definição da posição fiscal e depois passa pelo método do módulo sale, então neste caso é importante notar que a inteligencia esta no módulo account_fiscal_position_rule_sale. A importância deste método e que se você mudar o estabelecimento será recalculada a regra de posição fiscal, e deve ser mantido na localização

review: Disapprove

Unmerged revisions

544. By Carlos Almeida on 2013-06-13

[FIX] BUG1190772
Não faz sentido redefinir este método só para chamar o super, se o super não tem estes parâmetros todos e o método não faz nada aqui com esses parâmetros...

=== modified file 'l10n_br_sale/__openerp__.py'
--- l10n_br_sale/__openerp__.py	2013-02-11 15:51:01 +0000
+++ l10n_br_sale/__openerp__.py	2013-06-13 22:50:38 +0000
@@ -24,7 +24,7 @@
     'license': 'AGPL-3',
     'author': 'Akretion, OpenERP Brasil',
     'website': 'http://openerpbrasil.org',
-    'version': '0.6',
+    'version': '0.544',
     'depends': [
         'l10n_br_account',
         'account_fiscal_position_rule_sale',

=== modified file 'l10n_br_sale/sale.py'
--- l10n_br_sale/sale.py	2013-04-16 15:06:39 +0000
+++ l10n_br_sale/sale.py	2013-06-13 22:50:38 +0000
@@ -93,14 +93,6 @@
             partner_id, shop_id, context,
             fiscal_category_id=fiscal_category_id)
 
-    def onchange_shop_id(self, cr, uid, ids, shop_id=None, context=None,
-                         partner_id=None, partner_invoice_id=None,
-                         partner_shipping_id=None,
-                         fiscal_category_id=None, **kwargs):
-        return super(sale_order, self).onchange_shop_id(
-            cr, uid, ids, shop_id, context, partner_id, partner_invoice_id,
-            partner_shipping_id, fiscal_category_id=fiscal_category_id)
-
     def onchange_fiscal_category_id(self, cr, uid, ids, partner_id,
                                     partner_invoice_id=False, shop_id=False,
                                     fiscal_category_id=False, context=None):

=== modified file 'l10n_br_sale/sale_view.xml'
--- l10n_br_sale/sale_view.xml	2013-02-09 13:54:30 +0000
+++ l10n_br_sale/sale_view.xml	2013-06-13 22:50:38 +0000
@@ -42,13 +42,6 @@
 				ref="account_fiscal_position_rule_sale.view_order_form1" />
 			<field name="priority">26</field>
 			<field name="arch" type="xml">
-				<field name="shop_id" position="attributes">
-					<attribute name="on_change">
-						onchange_shop_id(shop_id, context,
-						partner_id, partner_invoice_id, partner_shipping_id,
-						fiscal_category_id)
-					</attribute>
-				</field>
 				<field name="partner_invoice_id" position="attributes">
 					<attribute name="on_change">
 						onchange_address_id(partner_invoice_id,