bank-management: genera orden de pago emite un comprobante con valor cero

El bug se genera desde el módulo account_voucher_patch, exactamente en la función account_voucher_get (http://bazaar.launchpad.net/~vauxoo/bank-management/trunk/view/head:/account_voucher_patch/account_voucher.py#L78),
que es llamada desde la función action_move_line_create (http://bazaar.launchpad.net/~vauxoo/bank-management/trunk/view/head:/account_voucher_patch/account_voucher.py#L283).
Esta función action_move_line_create existe en el objeto base y se le está haciendo una redeclaración completa, por el contrario account_voucher_get no existe en el objeto original.

Entrando en el problema, en la linea 108 (http://bazaar.launchpad.net/~vauxoo/bank-management/trunk/view/head:/account_voucher_patch/account_voucher.py#L108) se invoca el método create sobre un objeto account.move, ese asiento que se genera es el que queda sin líneas luego de generar el pago. Con sólo comentar ese create se resuelve el problema de los asientos sin líneas, pero persiste un problema mas grave de fondo (IMHO), que es el que exista un módulo que busque parchear las funciones base de account.voucher y que esté desfasado (deprecated), más aún cuando el mismo módulo es una dependencia del bank_management.

Me parece que debemos hacer una revisión exhaustiva de los módulos account_voucher_patch y account_voucher_multipartner para determinar si realmente son necesarios.

Hola Aristobulo,

Coincido contigo en la de realizar una revisión a los modulos especialmente account_voucher_multipartner, en cuanto al modulo account_voucher_patch fue y es una solución temporal al problema que se le había planteado a OpenERP sobre el método action_move_line_create el cual erá excesivamente largo y confuso, además de poco práctico al tener que reescribirlo y heredar las casí 170 líneas del mismo para agregar 4 ó 5 líneas necesarias para su adapatación a la localización venezolana. Y te digo erá ya que se le realizó la propuesta la cual fue aprobada e incorporada a la versión 6.1 de manera oficial. Lamentablemente esta mejorá no fue incorporada a la version 6.0 de manera oficial por lo que decidimos agregar el modulo account_voucher_patch para que recreará un poco la estructura fututra del método y mantener la portabilidad de las modificaciones sin o con el menor cambio posible para la versión 6.1. Abajo te dejo los enlaces del blueprint y la propuesta realizada:

https://blueprints.launchpad.net/openobject-addons/+spec/refactor-action-move-line-create
https://code.launchpad.net/~vauxoo/openobject-addons/REF-account_voucher/+merge/78747

Saludos,

Javier,

entiendo la situación con respecto a account_voucher_patch, ahora, teniendo en cuenta la existencia del asiento en cero y la ejecución innecesaria del create ¿qué propones?
Nosotros comentamos el create y los pagos se siguen generando correctamente y el asiento en cero deja de crearse, pero al hacer esto la función queda inútil, además que es una solución bastante maltrecha. Para qué se intenta crear ese move?

Basicamente remover la línea del create. Fue una omisión de mi parte. Durante el proceso de la propuesta los métodos sufrierón modificaciones a solicitud de la gente de OpeERP para su aprobación los cuale se probarón y realizarón en la versión 6.1 y no ha si en 6.0. El propósito del método es devolver un diccionario con los datos para la creación del asiento (account.move) .