credativ team mailing list archive

["Stefan Rijnhart \(Therp\)" <903929@xxxxxxxxxxxxxxxxxx>]

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