tryton-ar / account_voucher_ar

GNU General Public License v3.0
2 stars 4 forks source link

Desfasaje en la numeración de comprobantes #9

Open lukio opened 7 years ago

lukio commented 7 years ago

Tengo un cliente que lo esta utilizando desde la web. Me parece que al clickear Pagar, falla al crear el movimiento, pero igualmente se incrementa el número de la secuencia y le asigna al recibo en estado borrador un número de comprobante.

Por lo que veo en el código, el voucher crea el número previamente a crear los movimientos. Si el movimiento falla, igualmente el número ya ha sido asignado.

@classmethod
@ModelView.button
@Workflow.transition('posted')
def post(cls, vouchers):
    cls.check_already_reconciled(vouchers)
    cls.check_amount_invoices(vouchers) 
    for voucher in vouchers:
        voucher.set_number()
        move_lines = voucher.prepare_move_lines()
        voucher.create_move(move_lines)

Sugieron pasar la linea voucher.set_number() debajo de voucher.create_move(move_lines)

El tema es que el método prepare_move_lines() utiliza el número de recibo para agregarlo a la descripción de la linea del movimiento.

smarro commented 7 years ago

Hola Lukio, que raro el error en la creación del movimiento... tenés el traceback de ese error ? Por otro lado, al dar error debería hacer un rollback de toda la transacción y no aumentar la numeración... lo voy a probar....

lukio commented 7 years ago

Hola seba! Me ha sucedido con un cliente en producción. No lo he reproducido en una instancia local. Voy a realizar unas pruebas locales y te cuento. Estoy de acuerdo que debería hacer un rollback, pero lo cierto es que en la db del cliente la numeración de los recibos quedo salteada :/