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
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: mp+167403@code.staging.launchpad.net

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.
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
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

Luis,
Es probable que la solución sea alternar los string de los campos 'street3' y 'street4' ¿Puedes validar si esto es posible?

El orden del domicilio de partner aparece calle, colonia y números.
Debería de ser calle, número exterior, número interior y hasta entonces la colonia.

[REF][l10n_mx_address] Agregar al campo street2 mensaje place holder de Colonia.

review: Needs Fixing
222. By Luis Torres - http://www.vauxoo.com

[IMP][l10n_mx_facturae]Change 'N/A' in data xml when don´t have street2 or city2

223. By Luis Torres - http://www.vauxoo.com

[IMP][l10n_mx_partner_address]Added placeholder in street2

224. By Luis Torres - http://www.vauxoo.com

[REF][l10n_mx_facturae]Revert changes on street3 and street4

225. By Luis Torres - http://www.vauxoo.com

[IMP][l10n_mx_partner_address]Change references fields street3 & street4

Revision history for this message
Luis Torres - http://www.vauxoo.com (luis-cleto-) wrote :

Se cambio en el diccionario donde se llenan los datos del xml, que si no hay colonia o localidad estos campos los ponga como false, y al final de llenar el diccionario elimine estos nodos.

Se agrego placeholder en la colonia

y se revertieron los cambios realizados en street3 y street4, y se cambiaron las referencias de estos, ahora street3 es street4 y viceversa.

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

Luis,

Podrías cambiar el orden de las siguientes líneas

284 <field name="street" placeholder="Street..."/>
288 + <field name="street2" placeholder="Colonia..."/>
289 + <field name="l10n_mx_street3" invisible="True" placeholder="No. Exterior..."/>
290 + <field name="l10n_mx_street4" placeholder="No. Interior..."/>

A primero Calle, No. Interior, No. Exterior, Colonia.

Pregunta, porque l10n_mx_street3 está como invisible=True?

review: Needs Fixing
226. By Luis Torres - http://www.vauxoo.com

[IMP][l10n_mx_partner_address]Order streets

227. By Luis Torres - http://www.vauxoo.com

[REF][l10n_mx_partner_address]Remove invisible='True', from street3

Revision history for this message
Luis Torres - http://www.vauxoo.com (luis-cleto-) wrote :

Se ordenaron los streets, y se quito el invisible='true' del numero exterior

review: Needs Resubmitting

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.