REGRESSION - El modulo bank_management en el wizard de generar pago perdio la posibilidad de Cancelar un cheque

Bug #920629 reported by hbto [Vauxoo] http://www.vauxoo.com
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bank and Payment Management Latin America
Fix Committed
High
Javier Duran

Bug Description

con los ultimos cambios introducidos, bank_management perdio la posibilidad de comprobar si un cheque se imprimio correctamente o no, el hecho de que el modulo ahora no imprima directamente los cheques ahora,
no implica que el modulo ahora no haga seguimiento de si el cheque fue impreso correctamente o no,
el cheque se seguira imprimiendo sea en la maquina de al lado, a maquina de escribir, en boligrafo, o en lapiz, pero el cheque SE SIGUE IMPRIMIENDO,

ESTA CARACTERISTICA NO SE PUEDE OBVIAR,

DEBE VOLVER,

POR FAVOR, REVISEN EL CODIGO CUANDO SE HAGA ESTO,
PREGUNTEN, HABLEN LLAMEN, CONVERSEN, COMUNIQUENSE,

Changed in bank-management:
importance: Undecided → High
Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

Javier.

Te he delegado el cambio.

Por favor cuando estamos haciendo un "Refactor" de una funcionalidad ES UN REFACTOR no esta listo hasta que la funcionalidad COMPLETA no está culminada, si le quitas funcionalidades DEBES analizar el impacto, sino tienes forma de hacerlo inmediato, se envía un correo y se espera a que la propuesta de merge sea APROBADA.

Es decir.

Si estamos refactorizando algo que hace A+B+C y decidimos quitar B para que lo haba otro modulo, el modulo orignal debe quedar A+C y B en el otro modulo, no A y B+C en todo caso mucho menos si para que A sea "A" y no "a" requerirá del otro módulo.

Eso fue lo que logro con el cambio.

Saludos.

PS: Conceptos importantes:_ Para revisión por el equipo despues de resuelto este detalle.

http://es.wikipedia.org/wiki/Refactorización

Changed in bank-management:
status: New → Confirmed
assignee: nobody → Javier Duran - Vauxoo (javieredm)
Javier Duran (javieredm)
Changed in bank-management:
status: Confirmed → Triaged
status: Triaged → Fix Committed
Revision history for this message
Javier Duran (javieredm) wrote :

Se unió a la rama principal:

revno: 34
revision-id: javier@squezee-vir-20120123230410-82hgl0bzlvqe0u4a

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.