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

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

Luis,
En vez de usar 'N/A' utiliza False, para que valides en el if con False

'noExterior': address_invoice_parent.l10n_mx_street4 and \
141 address_invoice_parent.l10n_mx_street4.replace(
142 '\n\r', ' ').replace('\r\n', ' ').replace('\n', ' ').\
143 replace('\r', ' ') or 'N/A', # "Numero Exterior"

Cambio en la línea 143: replace('\r', ' ') or False, # "Numero Interior"
(Cambié n/a por False, y cambié el comentario a Numero interior.

Para que el if lo hagas así
if invoice_data['Emisor']['DomicilioFiscal'].get('colonia') == False:

review: Needs Fixing

« Back to merge proposal