[ADD] Nuevo modulo para pagos con tarjeta de credito (sermepa-redsys)

[sergio-teruel]

No description provided.

[rlizana]

Genial @sergio-incaser, corrige los errores PEP8.

[coveralls]

Coverage remained the same when pulling c1b9560 on incaser:8.0 into fbb90ab on OCA:8.0.

[rlizana]

@sergio-incaser para evitar problemas en la configuración los siguientes campos deberían de ser required:

redsys_merchant_code
redsys_merchant_titular
redsys_merchant_name
redsys_merchant_description

por lo demás 👍

[sergio-teruel]

Si, tienes razón, parece ser que no funciona el atributo "required_if_provider", creo que esto es para que el campo sea requerido cuando selecciono este proveedor de pago, pero tampoco en los proveedores "oficiales"

[rafaelbn]

Hola Sergio, Roberto:

¿Me podríais informar como está el módulo? Vamos a tener necesidad de usarlo y si necesita correciones nos ponemos con ello.

Me decís por favor

Gracias
Rafa

[rlizana]

@rafaelbn el modulo es totalmente funcional, yo lo he probado y funciona correctamente, el problema que @sergio-incaser comenta es que el atributo required_if_provider parece que no funciona correctamente, pero lo mismo ocurre para los modulos oficiales, no he tenido tiempo de mirar esa parte.

[rafaelbn]

+1
Probado con TPV contratado al BBVA @sermepa.es / redsys.es
Tenemos las pruebas de un pedido confirmado, erróneo y en proceso usando el TPV.

[pedrobaeza]

Poner una descripción más amplia y que no sea spanglish, jeje. En este caso, podría ser en español, ya que el módulo es para una audiencia española. Pon un título con ===, lo que hace (esa frase puede valer), y una pequeña explicación de los parámetros necesarios para configurarlo.

Por último, extrae por favor esta descripción a un archivo README.rst, que Odoo lo soporta y además es legible directamente en GitHub.

[pedrobaeza]

Para los campos requeridos, aquí se puede ver que se comprueba al guardar con una constraint:
https://github.com/odoo/odoo/blob/8.0/addons/payment/models/payment_acquirer.py#L93
Por lo que sí debe funcionar, pero no poniendo el campo en morado, si no como digo a la hora de guardar.

[pedrobaeza]

Para el icono del módulo, siempre utilizamos un icono cuadrado de 128x128, para que así la alineación en la vista kanban sea correcta (aunque Odoo se lo salte en alguno de sus módulos). Además, es conveniente que lleve transparencia, para así poder utilizarlo en vistas que no tengan el fondo blanco. Cambia por favor ese icono a por ejemplo éste redimensionado a 128: http://www.cobraronline.es/images/gateways/redsys.png.

También para el icono que sale en la solapa sería conveniente buscar un icon con transparencia, como éste: http://www.redsys.es/wps/contenthandler/dav/fs-type1/themes/RedSys_ThemePublic/img/logo.png

[coveralls]

Coverage remained the same when pulling 5408a2a on incaser:8.0 into fbb90ab on OCA:8.0.

[pedrobaeza]

Ya no es necesario repetir esta descripción. La coge del archivo. Elimínala por favor.

[coveralls]

Coverage remained the same when pulling 32f50e2 on incaser:8.0 into fbb90ab on OCA:8.0.

[coveralls]

Coverage remained the same when pulling abba856 on incaser:8.0 into fbb90ab on OCA:8.0.

[pedrobaeza]

Gracias por los cambios y la contribución, Sergio!

👍

[rafaelbn]

Hola hemos encontrado errores en el módulo.

[Endika]

Hola, hemos detectado que el modulo de redsys no captura los errores que se puedan dar en el pago.
O te lleva a la URL_OK o URL_KO. Y en caso de error no captura el error para poder mostrarlo.

[pedrobaeza]

Publicadlos como issues y poned con @ al creador del módulo.

[rafaelbn]

Hola @sergio-incaser , hola @rlizana !

¿lleváis mucho tiempo con instancias de odoo v8 en producción? Hemos detectado que el módulo con crear una vista qweb de error para mostrar al usuario en caso de fallo como dice @Endika .

Vamos a agregar funcionalidad y nos gustaría comentarlo con vosotros.

Gracias
Rafa

[sergio-teruel]

Todas las mejoras son bienvenidas.... adelante con ello.

[rlizana]

@rafaelbn exactamente donde esta el fallo?

Tanto en URL_OK como en URL_KO tienes los valores de la respuesta por petición GET, y entre ellos el DS_Response que indica el código de respuesta (o de error en su caso)

Lo que si tengo pendiente es subir una mejora con un método que le pasas el código de DS_Reponse y te devuelve el mensaje. Me falta confirmar con REDSYS si dependiendo del Banco devuelven el codigo con un prefijo o no.

[rafaelbn]

Hola @rlizana ,

El módulo no te crea en una vista en Odoo para llevar al cliente después de un URL_KO, además en esta vista lo idóneo sería pintar el mensaje de error que nos devuelve REDSYS.

¿No te parece?