banking-addons-team team mailing list archive

[Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>]

Review: Needs Fixing

Hi,


You're right, it's sounds strange to change the ids dict ! The purpose here is to call button_cancel on all moves not in draft. This ease the process of the bank statement cancellation.

As you cannot delete moves that are not in draft, if you do not call button_cancel on every moves, then you have to open them and cancel each of them manually before being able to cancel the bank statement.

Thanks,

Joël
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-70-fix-statement-cancel/+merge/147114
Your team Banking Addons Team is subscribed to branch lp:banking-addons/bank-statement-reconcile-70.