Merge lp:~therp-nl/banking-addons/ba70-lp1246642 into lp:banking-addons

[Stefan Rijnhart (Opener)]

[Holger Brunn (Therp) (hbrunn)]

review: Approve (code review)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

review: Approve

[Stefan Rijnhart (Opener) (stefan-opener)]

I'm retracting this proposal as the server bug should be fixed by this commit: http://bazaar.launchpad.net/~openerp/openobject-server/7.0/revision/5136

Unmerged revisions

203. By Stefan Rijnhart (Opener) on 2013-10-31

[FIX] AccessError on confirming bank statements of a child company
 (Workaround for lp:1246642)

=== modified file 'account_banking/banking_import_transaction.py'
--- account_banking/banking_import_transaction.py	2013-10-14 12:33:57 +0000
+++ account_banking/banking_import_transaction.py	2013-10-31 10:13:29 +0000
@@ -1602,7 +1602,13 @@
             if st_line.import_transaction_id:
                 import_transaction_obj.confirm(
                     cr, uid, st_line.import_transaction_id.id, context)
-            st_line.refresh()
+            # Workaround for
+            # https://bugs.launchpad.net/openobject-server/+bug/1238042
+            # Need to rebrowse instead of refresh, because any parent company
+            # will be in the browse record's id cache after the first pass, and
+            # accessing the actual company will then trigger a read attempt on
+            # the inaccessible parent company too, leading to an access error.
+            st_line = self.browse(cr, uid, st_line.id, context=context)
             st_line_number = statement_pool.get_next_st_line_number(
                 cr, uid, st_number, st_line, context)
             company_currency_id = st.journal_id.company_id.currency_id.id