Merge lp://staging/~vauxoo/openerp-mexico-localization/report_PAC_SF_dev_rodo into lp://staging/openerp-mexico-localization
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: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Rodolfo Lopez (community) | Needs Resubmitting | ||
Moisés López - http://www.vauxoo.com | Needs Fixing | ||
Review via email:
|
This proposal supersedes a proposal from 2012-08-28.
Description of the change
[FIX][l10n_
[FIX][l10n_
[FIX][l10n_
[FIX][l10n_
[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
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.