Merge lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_bugfix_943445-dev-moylop260 into lp://staging/openerp-mexico-localization

Proposed by Moisés López - http://www.vauxoo.com
Status: Merged
Approved by: Moisés López - http://www.vauxoo.com
Approved revision: no longer in the source branch.
Merge reported by: Moisés López - http://www.vauxoo.com
Merged at revision: not available
Proposed branch: lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_bugfix_943445-dev-moylop260
Merge into: lp://staging/openerp-mexico-localization
Diff against target: 17 lines (+5/-1)
1 file modified
l10n_mx_facturae/invoice.py (+5/-1)
To merge this branch: bzr merge lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_bugfix_943445-dev-moylop260
Reviewer Review Type Date Requested Status
Edgar M (community) Approve
David Olvera - http://www.qualtecmty.com.mx (community) Approve
Isaac López Zúñiga Pending
Review via email: mp+95393@code.staging.launchpad.net

Commit message

[MERGE] [FIX] [l10n_mx_facturae] Change field "serie" mode optional

To post a comment you must log in.
Revision history for this message
Edgar M (edgar-med) wrote :

Ya apliqué los cambios correspondientes del patch pero al validar el archivo XML en el validador del SAT está saliendo nuevamente el siguiente error:

Estructura: Error en la linea 26: El valor con longitud = '0' es menor a la lóngitud mínima = '1' para el tipo '#AnonType_serieComprobante'
Error en la linea 26: El valor " del atributo 'serie' en el elemento 'Comprobante' no es válido con respecto a este tipo. 'null'

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

Asegúrate de 2 cosas:
1) Que el cambio lo hayas aplicado, fíjate con
bzr diff
o
bzr log -r -1
Para que veas que sí tenga un cambio

2) Que la factura XML se vuelva a generar.
Las facturas utilizan el attachment cuando ya han sido generadas previamente, para no re-generarlas cada vez.
Aségurate de borrar este attachment, para que te asegures de que te está re-generando la factura xml.
Para ello te puedes ir a "cancelar" la factura y luego a "establecer como borrador"
O te puedes ir directamente a sus adjuntos y eliminarlos, y volverla a generar.

¡Saludos!

Revision history for this message
Edgar M (edgar-med) wrote :

Los cambios si se aplicaron, el problema está en que se sigue generando el archivo XML con el campo Serie="", si la serie no está presente este campo ya no debería estar saliendo en el archivo XML

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

Asegúrate de 2 cosas:
1) Que el cambio se haya aplicado.
Ok, esto está comprobado. Gracias.

Ahora el punto 2)
Las facturas utilizan el attachment cuando ya han sido generadas previamente, para no re-generarlas cada vez.
Aségurate de borrar este attachment, para que te asegures de que te está re-generando la factura xml.
Para ello te puedes ir a "cancelar" la factura y luego a "establecer como borrador"
O te puedes ir directamente a sus adjuntos y eliminarlos, y volverla a generar.

Este punto, ¿que comentario tienes?

El cambio que se hizo, era para que ya no apareciera la serie, y es muy similar a como lo hizo @David, solo hay que validar que el punto 2, no te esté afectando.

¡Saludos!

Revision history for this message
Edgar M (edgar-med) wrote :

Para evitar este problema estoy realizando una nueva factura cada vez y verificando el archivo XML, pero sigue saliendo el campo Serie=""

Saludos

Revision history for this message
David Olvera - http://www.qualtecmty.com.mx (dolvera) wrote :

Ya hice pruebas del cambio propuesto y trabaja adecuadamente
Saludos

review: Approve
Revision history for this message
Edgar M (edgar-med) wrote :

Ok, ya esta listo! para que funcionara tuve que definir nuevamente la secuencia del diario, de esta manera ya no apareció nuevamente la Serie=""

Gracias Moisés y David

116. By Moisés López - http://www.vauxoo.com

[MERGE] [FIX] [l10n_mx_facturae] Change field "serie" mode optional

Revision history for this message
Edgar M (edgar-med) wrote :

Me sigue dando lata, pense que ya estaba bien, pero al momento de validad nuevamente el XML en el SAT vuelve el mismo error. Podrian revisal el archivo XML y comprobar que ya no está el cámpo Serie=""?

Saludos

Revision history for this message
Edgar M (edgar-med) wrote :

Una disculpa David y Moisés, ya encontré mi error!! no había reiniciado mi openerp-server, bueno de los errores es de donde aprendemos mas!

Ya me funcionó el cámbio y ya lo validé tambien con el SAT, muchas gracias a los dos, seguimos en contacto.

Saludos

Edgar

Revision history for this message
Edgar M (edgar-med) :
review: Approve
Revision history for this message
Edgar M (edgar-med) :
review: Approve
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

Justo ahora lo estaba probando, y como bien dice @David ya salió correctamente.

Lo que comentas de que no se estaba reiniciando el servidor, era una tercera consideración que no recordaba. jeje
Pero como bien dices, de los errores es como más se aprende. jeje

Ya está como merged & Fix release.

Gracias por reportar el bug, preguntar al SAT y probar el cambio.

¡Saludos!

Revision history for this message
Edgar M (edgar-med) wrote :

Muchas gracias también a ti Moisés seguimos en contacto.

Saludos

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.