openerp-mexico-localization

Merge lp://staging/~vauxoo/openerp-mexico-localization/report_PAC_SF_dev_rodo into lp://staging/openerp-mexico-localization

report_PAC_SF_dev_rodo
Merge into trunk

Proposed by Rodolfo Lopez on 2012-08-29

Status:	Merged
Merged at revision:	193
Proposed branch:	lp://staging/~vauxoo/openerp-mexico-localization/report_PAC_SF_dev_rodo
Merge into:	lp://staging/openerp-mexico-localization
Diff against target:	600 lines (+198/-193) 6 files modified l10n_mx_facturae/report/invoice_facturae_pdf.py (+2/-2) l10n_mx_facturae/report/invoice_facturae_pdf.rml (+2/-2) l10n_mx_facturae_pac_sf/__openerp__.py (+1/-1) l10n_mx_facturae_pac_sf/report/invoice_facturae_pac_sf_pdf.py (+2/-2) l10n_mx_facturae_pac_sf/report/invoice_facturae_pac_sf_pdf.rml (+174/-173) l10n_mx_invoice_tax_ref/invoice.py (+17/-13)
To merge this branch:	bzr merge lp://staging/~vauxoo/openerp-mexico-localization/report_PAC_SF_dev_rodo
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Rodolfo Lopez (community)		Needs Resubmitting on 2012-08-29
Moisés López - http://www.vauxoo.com	2012-08-29	Needs Fixing on 2012-08-29
Review via email: mp+121728@code.staging.launchpad.net

This proposal supersedes a proposal from 2012-08-28.

Description of the change

[FIX][l10n_mx_facturae_pac_sf] se modifico el reporte PAC SF para poder mostrar el IVA 0%, usando como dependence el modulo account_invoice_tax del repositorio bzr branch lp:addons-vauxoo.
[FIX][l10n_mx_invoice_tax_ref] se modifico invoice.py para actualizar la funcion _get_tax_data para traer la categoria de los impuestos o el nombre en su defecto
[FIX][l10n_mx_facturae_pac_sf] se modifico validación de campo tax_id existente en account.invoice.tax
[FIX][l10n_mx_invoice_tax_ref] se modifico validación de campo tax_id existente en account.invoice.tax

[FIX] Se modifico la funcion _get_tax_data en factura párrafo Rastrear Mejor y mas Eficiente los Impuestos, ESTO origino Cambios en los Reportes invoice_ facturae_ pac_sf_ pdf y invoice_ facturae_ pdf

Revision history for this message

Moisés López - http://www.vauxoo.com (moylop260) wrote on 2012-08-28: Posted in a previous version of this proposal

Mis observaciones:

*En el archivo ../addons/l10n_mx_invoice_tax_ref/invoice.py
Haces referencia a la variable "tax_category_id"
Pero no hay una dependencia al módulo que lo contiene en su __openerp__.py.

*Los datos que se obtienen en los campos de ../addons/l10n_mx_invoice_tax_ref/invoice.py no los estás utilizando en los reportes SXW & RML, ya que en el código se vuelven a obtener innecesariamente desde la fuente original.

*Pruébalo con casos retroactivos, es decir, que ocurre con los movimientos que estában antes de instalar este módulo, y se intenta imprimir o generar el XML.
Ya que la validación "if 'tax_id' in self._columns:" valida la retroactividad si existe el campo, pero además, debe de tener contenido.

*Una vez que se hayan aplicado los cambios en el módulo ../addons/l10n_mx_invoice_tax_ref, es probable que ya no sea necesario cambiar nada en los rml en cada uno de los reportes de facturación electrónica.

*Si se llegara a tener la necesidad de cambiar el .py de alguno de los reportes de facturae, tienen que ser en el del pac_sf, facturae normal y cbb (en todos).

NOTA: Es probable que con el depends en ../addons/l10n_mx_invoice_tax_ref/__openerp__.py ya no se necesite validar si la columna existe, pero sí hay que validar el valor.

review: Needs Fixing

Revision history for this message

Rodolfo Lopez (el-rodo-1) wrote on 2012-08-29: