← Back to team overview

openerp-india team mailing list archive

[Bug 928616] Re: [v6.0 & v6.1] [account_voucher] cancel_voucher method is unlinking the whole account_move_reconciliation record

 

Ok, Wonderful,

Just a little advice, why dont you use set() for those list being collected for invoice_ids,
this will reduce [1,1,1,1,3,3,3,3,3,4,4,4,4] to just set([1,3,4]), which is either an iterable,

Thank you very much for you diligent attention.

Regards.

Hbto [Vauxoo]

-- 
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Addons.
https://bugs.launchpad.net/bugs/928616

Title:
  [v6.0 & v6.1] [account_voucher] cancel_voucher method is unlinking the
  whole account_move_reconciliation record

Status in OpenERP Addons (modules):
  Fix Committed

Bug description:
  [v6.0 & v6.1] [account_voucher] cancel_voucher method is unlinking the
  whole account_move_reconciliation (amr) record

  There is a more elegant way of doing this, instead of getting rid of the whole amr, it could be unreconciled and reconciled 
  again so the other account_move_lines involved in the reconciliation wont't be lost.

  Whenever you have many partial payments for invoices and one of those partial payments (account_voucher) is unreconciled
  the forementioned method will get rid of the whole amr, and all the previously aml, involved get lost.

  
  Here you find patch fixes for both v6.0 and v6.1.

  Thanks in advance for your attention.

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/928616/+subscriptions


References