Merge lp:~openerp-community/openobject-addons/7.0-587621-opw-orphan-voucher into lp:~openerp-dev/openobject-addons/7.0-587621-opw-skh

[Niels Huylebroeck]

Instead of simply unhooking any draft vouchers from the bank.statement.line we clean them up when you change the amount of the bank.statement.line.

[Naresh(OpenERP) (nch-openerp)]

Hello Niels,

Well, this does not seem to be an issue that needs an OPW cover to get in to stable, rather it seems an improvement. Secondly, according to the current stable source the related vouchers are set to cancelled state when you cancel the bank statement and later if you confirm the bank statement again the same voucher will be in posted state. Which is so far good.

Now comes your case(@Niels) for which you have proposed a merge proposal https://code.launchpad.net/~openerp-community/openobject-addons/7.0-587621-opw-orphan-voucher/+merge/154932 where in you unlink the isolated vouchers that are result of changing the original amount on the bank statement line. But here if you look closely as soon as you change the original amount on the bank statement line the button on the line will turn *red* if the balance does not match that means you need to take care you are reconciling it with the correct amount. To Fix this you need to click on the button to and change the *Allocation* value to the actual value you want to reconcile. As soon as you correct that you see the button turns *Green* means everything is correctly setup and now you can proceed with reconciliation. In this case the previous voucher will be in cancelled state and a new entry will be created and reconciled(posted).

Keeping the history of vouchers (altered ones) seems good. instead of setting them to draft and later unlinking them.

However, we its still open to be discussed.

Thanks,

review: Needs Information

Unmerged revisions

8909. By Niels Huylebroeck on 2013-03-22

[IMP] account_voucher: cleaning up the orphan voucher when changing the amount on the bank.statement.line

=== modified file 'account_voucher/account_voucher.py'
--- account_voucher/account_voucher.py	2013-03-21 13:38:48 +0000
+++ account_voucher/account_voucher.py	2013-03-22 12:20:26 +0000
@@ -1561,6 +1561,14 @@
         return res
 
     def onchange_amount(self, cr, uid, ids, amount, context=None):
+        # If the line already has a voucher_id and the state of it is 'draft'
+        # it means this bank statement was cancelled before and we should delete
+        # the voucher AND clear the field voucher_id.
+        if ids:
+            for line in self.browse(cr, uid, ids, context=context):
+                if line.voucher_id and line.voucher_id.state == 'draft':
+                    # Instead of orphaning this draft voucher, we will delete it
+                    line.voucher_id.unlink()
         return {'value' :  {'voucher_id' : False}}
 
     def _amount_reconciled(self, cursor, user, ids, name, args, context=None):