Bank and Payment Management Latin America

Merge lp://qastaging/~vauxoo/bank-management/javy-voucher-wkf into lp://qastaging/bank-management/trunk

  1. javy-voucher-wkf
  2. Merge into trunk
Proposed by Javier Duran
Status: Merged
Merged at revision: 41
Proposed branch: lp://qastaging/~vauxoo/bank-management/javy-voucher-wkf
Merge into: lp://qastaging/bank-management/trunk
Diff against target: 56 lines (+19/-2)
3 files modified
bank_management/check/wizard/voucher_pay_support_wizard_pay_order.py (+8/-1)
bank_management/i18n/es.po (+6/-0)
bank_management/i18n/es_VE.po (+5/-1)
To merge this branch: bzr merge lp://qastaging/~vauxoo/bank-management/javy-voucher-wkf
Reviewer Review Type Date Requested Status
Javier Duran (community) Approve
Nhomar - Vauxoo newguideline Needs Fixing
hbto [Vauxoo] http://www.vauxoo.com Pending
Gabriela Quilarque Pending
Review via email: mp+90352@code.qastaging.launchpad.net

Description of the change

Se cambia en el wizard de impresión de cheque la validación de la orden de pago para que se realice a través del flujo de trabajo(workflow) del modelo y no de manera manual.

To post a comment you must log in.
Revision history for this message
Javier Duran (javieredm) wrote :

Chic@s espero confirmación de algun@ para realizar este merged.

review: Approve
Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

Hola Javi.

Para que quede como registro aca.

Por favor evitemos esta mala práctica en openerp para evitar traceback inesperados.

cuando hagas esto:

+ wf_service.trg_validate(uid, 'account.voucher', ids2[0], 'proforma_voucher', cr)
deberíamos hacer esto:

+ len(ids) and 2wf_service.trg_validate(uid, 'account.voucher', ids2[0], 'proforma_voucher', cr)

Para evitar si ids2 viene vacío el traceback:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
IndexError: list index out of range

Y para poderlo tracear de forma controlada.

+ len(ids) and 2wf_service.trg_validate(uid, 'account.voucher', ids2[0], 'proforma_voucher', cr) or self.log(cr, uid, inv_id, "MEnsaje informativo del Objeto XX que no pudo ser Validado", context=ctx)

De tal forma que controlemos que cuando nuestros modelos fallen perder el trazo, y si los ids vienen vacíos esn alguna situación que no determinamos no le genere traceback al cliente.

Saludos.

review: Needs Fixing (newguideline)
Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

Es un comentario, podemos pulirlo, pero tenemos que tener alguna estrategia de evitar tomar el primer elemento de una lista sin saber si en alguna situación esa lista viene vacía.

Saludos.

lp://qastaging/~vauxoo/bank-management/javy-voucher-wkf updated
42. By Javier Duran

[IMP] se coloca validación para evitar errores en el supuesto caso de que
se omitiese en cualquier circunstancia la orden de pago original,
en el asistente de impresión de cheques, en el modulo bank_management

43. By Javier Duran

[FIX] corrección menor, en el modulo bank_management

Revision history for this message
Javier Duran (javieredm) wrote :

Unido a la rama principal:

revno: 41 [merge]
revision-id: javier@squezee-vir-20120127181341-6f5bsc66mhr795ab

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: