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

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.