Merge lp://staging/~vauxoo/openerp-mexico-localization/7.0_street3_street4_dev_luis into lp://staging/openerp-mexico-localization/7.0
Proposed by
Luis Torres - http://www.vauxoo.com
Status: | Merged |
---|---|
Merged at revision: | 217 |
Proposed branch: | lp://staging/~vauxoo/openerp-mexico-localization/7.0_street3_street4_dev_luis |
Merge into: | lp://staging/openerp-mexico-localization/7.0 |
Diff against target: |
293 lines (+33/-137) 5 files modified
l10n_mx_company_multi_address/invoice.py (+0/-115) l10n_mx_facturae/invoice.py (+18/-6) l10n_mx_facturae_pac/invoice.py (+0/-1) l10n_mx_partner_address/i18n/es.po (+9/-9) l10n_mx_partner_address/partner_address.py (+6/-6) |
To merge this branch: | bzr merge lp://staging/~vauxoo/openerp-mexico-localization/7.0_street3_street4_dev_luis |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Luis Torres - http://www.vauxoo.com (community) | Needs Resubmitting | ||
Moisés López - http://www.vauxoo.com | Needs Fixing | ||
Julio Serna-http://www.vauxoo.com | Pending | ||
Review via email:
|
Description of the change
Se ordenaron los números de las direcciones, se tomaban los interiores por los exteriores.
Y se agrego la validacion que si no hay colonia o localidad en las direcciones no coloque estos datos en el xml
To post a comment you must log in.
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 \ invoice_ parent. l10n_mx_ street4. replace(
141 address_
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í data['Emisor' ]['DomicilioFis cal'].get( 'colonia' ) == False:
if invoice_