Merge lp:~pexego/openerp-spain/openerp_spain_javier into lp:~openerp-spain-team/openerp-spain/6.1
Proposed by
Javier
Status: | Merged |
---|---|
Merged at revision: | 381 |
Proposed branch: | lp:~pexego/openerp-spain/openerp_spain_javier |
Merge into: | lp:~openerp-spain-team/openerp-spain/6.1 |
Diff against target: |
24 lines (+2/-1) 2 files modified
l10n_es_partner/__openerp__.py (+1/-0) l10n_es_partner/partner_es_view.xml (+1/-1) |
To merge this branch: | bzr merge lp:~pexego/openerp-spain/openerp_spain_javier |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Pedro Manuel Baeza | Approve | ||
Review via email: mp+200116@code.launchpad.net |
Description of the change
Se corriege una herencia erronea del módulo l10n_es_partner
To post a comment you must log in.
Buenas, Javier, gracias por detectar y reparar el problema. No obstante, ya que lo reparamos, vamos a hacerlo bien, cambiando la sustitución del campo por una sustitución de atributos, de tal forma que si existen otras propiedades asociadas al campo, no se pierdan en la herencia.
De tal forma que quedaría así:
<field name="acc_number" position= "attributes" > change" >onchange_ banco(acc_ number, acc_country_ id)"<attribute/ >
<attribute name="on_
</field>
El context no hace falta pasarlo explícitamente como argumento, ya que lo pasa automáticamente el ORM. Paso a realizar el MP con ese cambio.
Un saludo.