Code review comment for lp:~acysos-team/openerp-spain/openerp-spain-7.0-payment-order

Revision history for this message
Ignacio Ibeas (www.acysos.com) (ignacio-acysos) wrote :

Voy respondiendo a algunos puntos:

- La versión de account_payment_extension de la 7.0 no tiene la vista "view_payment_order_form_cancel", por lo que no se puede heredar de ella. En su lugar, habría que heredar de "view_payment_order_form_ext1".

La han debido modificar hace poco, porque yo si que tengo account_paymment_extension para openerp 7.0 con esa vista. Lo corrijo.

- El botón de "Seleccionar facturas a pagar/cobrar" no selecciona las facturas correspondientes. Tal vez este problema está en el account_payment_extension de la 7.0, pero el caso es que no está funcionando. ¿Puedes echarle un vistazo?

Este problema es de account_payment_extension y lo arreglaron, otra vez vuelve a fallar?, voy a verificarlo.

- Poniendo a mano el apunte, cuando se confirma el pago y se quiere exportar, aparece el error:

"UnicodeEncodeError: 'ascii' codec can't encode characters in position 56-57: ordinal not in range(128)"

En principio el cliente no tienen tildes ni nada en el nombre, pero aún así habría que incluso contemplar ese caso.

No replico este error. Puedes indicarme los pasos dados.

Existen otras cuestiones de convenciones comunitarias para la v7 (como eliminar atributos init_xml y update_xml y sustituirlos por el atributo data) o de traducciones, que ya me encargaré yo de realizar cuando hagamos el merge.

Ya he indicado que no es una refactorización y que no debería hacerse hasta que se conozca el cambio de normativa de SEPA. Por eso no se ha dedicado tiempo en este punto.

Saludos

« Back to merge proposal