openerp-mexico-localization

Merge lp://staging/~arielfigue/openerp-mexico-localization/trunk into lp://staging/openerp-mexico-localization

trunk
Merge into trunk

Proposed by Ariel E. Figueroa - http://www.humanytek.com on 2012-05-02

Status:	Rejected
Rejected by:	Moisés López - http://www.vauxoo.com on 2013-02-27
Proposed branch:	lp://staging/~arielfigue/openerp-mexico-localization/trunk
Merge into:	lp://staging/openerp-mexico-localization
Diff against target:	300 lines (+175/-62) 7 files modified partner_credit_limit/__terp__.py (+0/-46) partner_credit_limit/i18n/es_MX.po (+2/-2) partner_credit_limit/invoice.py (+10/-14) sfs_reference_calculation/__init__.py (+24/-0) sfs_reference_calculation/__openerp__.py (+42/-0) sfs_reference_calculation/partner.py (+70/-0) sfs_reference_calculation/partner_view.xml (+27/-0)
To merge this branch:	bzr merge lp://staging/~arielfigue/openerp-mexico-localization/trunk
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Moisés López - http://www.vauxoo.com			Disapprove on 2013-02-27
Ariel E. Figueroa - http://www.humanytek.com			Pending
Review via email: mp+104421@code.staging.launchpad.net

Revision history for this message

Ariel E. Figueroa - http://www.humanytek.com (arielfigue) wrote on 2012-05-02:

Este modulo crea un dato alfanumerico de referencia para pagos basado en el algoritmo 10 alfanumerico que utiliza Banco Santander

lp://staging/~arielfigue/openerp-mexico-localization/trunk updated on 2012-05-02

162. By Ariel E. Figueroa - http://www.humanytek.com on 2012-05-02: [IMP] Cambio de prefijo

Revision history for this message

Moisés López - http://www.vauxoo.com (moylop260) wrote on 2012-05-02:

Hola Ariel,
Excelente aporte
Este desarrollo es parte de la nueva ley del SAT para el pago referenciado?
O se trata de otra cosa distinta.

Gracias por aportar a la comunidad, y devolver un poco de lo que ya no brinda la misma.

Puedes dejar el branch en donde ahora lo subiste.
Y tambien lo podemos agregar al proyecto de localización, para lo último se tienen que cubrir ciertas guidelines, por si deseas hacerlo.

1) Busca un nombre de tu módulo alusivo a la solución que presenta, asi como el prefijo del módulo que heredas. O si es particular al Pais, puedes utilizar l10n_mx
Evitar usar nombres o abreviaturas de la empresa, esto va en el __openerp__
Nota: El proyecto de addons-extra si puedes conservar el prefijo que quieras.

2) Evitar tener código duro.
(no se si la palabra HESA se refiera al nuestro partner amigo o sea parte del algoritmo)

Revision history for this message

Ariel E. Figueroa - http://www.humanytek.com (arielfigue) wrote on 2012-05-02:

Moy:

Entiendo perfectamente bien tu comentario y si creo que lo mejor sera cumplir con estas guidelines para proponerlo dentro del proyecto de localizacion, trabajaremos en ello el dia de hoy y estaremos creando la nueva propuesta de merge cumpliendo con lo que se requiere para la utilizacion.

Respecto a tu pregunta de si es parte de la nueva ley del SAT, no, no se hizo considerando aspectos de ley, se hizo considerando aspectos de exigencia bancaria para la identificacion de pagos en una cuenta de banco.

Saludos
AEF

Revision history for this message

Ariel E. Figueroa - http://www.humanytek.com (arielfigue) wrote on 2012-05-02:

Moy, por lo que estuve averiguando este algoritmo es un algoritmo estandar que el banco santander usa para identificar depositos pero no solamente en mexico sino en cualquier pais a donde el banco este establecido, en este caso creo que no aplica lo de un modulo de localizacion o si?

Saludos
AEF

Revision history for this message

Moisés López - http://www.vauxoo.com (moylop260) wrote on 2012-05-02:

Aah excelente.
Coincidió el nombre de Pago Referenciado con un tema que tienen actualmente el SAT de México.
Como bien dices, no aplica para prefijo de localización l10n_mx aunque si puede estar en el proyecto de localización.

Revision history for this message

Ariel E. Figueroa - http://www.humanytek.com (arielfigue) wrote on 2012-05-02:

Ok siendo asi en las proximas horas estaremos haciendo la propuesta de merge cumpliendo guidelines.

Saludos
AEF

Revision history for this message

Moisés López - http://www.vauxoo.com (moylop260) wrote on 2012-05-04:

Ariel,
Ahora como guideline, y para conservar la calidad de los branch automatizados por medio de runbot, también se necesitan los archivos Yaml Test, con los casos de pruebas unitarias.

En este caso, puedes crear partner's y después obtener el dato calculado y compararlo con el que se genero para que pase el test.

Oliver Dony me lo solicito en el branch que estamos mejorando el algoritmo btree, y seguro lo pedirá en los proyectos de localización, mira:
http://vauxoo.com/node/34