Merge lp:~openerp-dev/openobject-addons/trunk-bug-888135-bde into lp:openobject-addons

Proposed by Bharat Devnani (Open ERP)
Status: Merged
Merged at revision: 5688
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-888135-bde
Merge into: lp:openobject-addons
Diff against target: 14 lines (+4/-0)
1 file modified
account/account_bank.py (+4/-0)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-888135-bde
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+82248@code.launchpad.net

Description of the change

Hello Sir,

I have applied the patch given by Yannick Vaucher.

Thanks & Regards,
Devnani Bharat R.

To post a comment you must log in.
Revision history for this message
Vo Minh Thu (thu) wrote :

Thanks. In the future, please try to write such code to sanitize the params right at the beginning of the function's body instead of later. (In this case, I will do it when merging in trunk.)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account/account_bank.py'
2--- account/account_bank.py 2011-11-07 12:43:37 +0000
3+++ account/account_bank.py 2011-11-15 06:51:31 +0000
4@@ -44,6 +44,10 @@
5 def post_write(self, cr, uid, ids, context={}):
6 obj_acc = self.pool.get('account.account')
7 obj_data = self.pool.get('ir.model.data')
8+
9+ if isinstance(ids, (int, long)):
10+ ids = [ids]
11+
12 for bank in self.browse(cr, uid, ids, context):
13 if bank.company_id and not bank.journal_id:
14 # Find the code and parent of the bank account to create

Subscribers

People subscribed via source and target branches

to all changes: