Code review comment for lp://staging/~vauxoo/openerp-mexico-localization/7.0_l10n_mx_payment_method_dev_carlos

Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

Juan Carlos,
Ya quedó bien la propuesta, ahora sí analizándola, me doy cuenta del verdadero problema del tema técnico de la vista.

La vista <record model="ir.ui.view" id="view_partner_form_inh_xml22">
Hereda a la vista base_vat.view_partner_form"
Que esta a su vez, viene heredada de base.view_partner_form, la cual, esta última es la verdadera vista padre, la cual es la que se debería estar heredando desde este módulo.

Conclusión:
Dejar el __openerp__.py con depends ["account"] (solo sin "base_vat")
Cambiar la vista actual a la cual se está heredando <field name="inherit_id" ref="base_vat.view_partner_form"/>
ahora cambiar mejor a la vista padre <field name="inherit_id" ref="base.view_partner_form"/>

NOTA: No estoy seguro del nombre de la vista padre, pero la puedes ver en server/openerp/addons/base/

review: Needs Fixing

« Back to merge proposal