Hi Dmitrijs, Thanks for your action! To be honest there is a problem with the merged branch [1] and browse object caching. When the same browse object is read again, the old information, which is cached, is being returned. As the move_line browse object argument is potentially being reused later on, this could lead to problems.
Of course, the original code is wrong in writing an object id to a browse object place holder, but we should consider writing a reference to a newly created browse object of the reconciliation to move_line.reconciliation_id. Should this merge be reverted for the above, note that this branch also fixes a separate problem with handling of an existing partial reconciliation. Cheers, Stefan. [1] http://bazaar.launchpad.net/~therp-nl/banking-addons/fix-lp903929/revision/71 -- You received this bug notification because you are a member of Banking Addons Team, which is the registrant for Banking Addons. https://bugs.launchpad.net/bugs/903929 Title: Bank import: browse record is written to Status in Banking Addons: Fix Released Bug description: in account_banking/wizard/bank_import.py, in _get_move_info() the browse record 'move_line' is being written to. To manage notifications about this bug go to: https://bugs.launchpad.net/banking-addons/+bug/903929/+subscriptions -- Mailing list: https://launchpad.net/~credativ Post to : [email protected] Unsubscribe : https://launchpad.net/~credativ More help : https://help.launchpad.net/ListHelp
