Merge lp:~openerp-dev/openobject-addons/6.1-opw-575460-pso into lp:openobject-addons/6.1

[Priyesh (OpenERP)]

Hello,

To reproduce the issue:

Install point_of_sale module.
Go to the Accounting/Configuration/Financial Accounting/Journals/Journals.
Open the Bank statements.

It will raise an error like this:
NotImplementedError: Iteration is not allowed on browse_record(account.journal, 15)

I fixed this issue. Kindly review the branch and let me know your inputs on it.

Regards,
Priyesh

[lp:~openerp-dev/openobject-addons/6.1-opw-575460-pso]

6829. By Priyesh (OpenERP) on 2012-06-05

[IMP] account: Added one more condition on ids(case:575460)

[Vinay Rana (OpenERP) (vra-openerp)]

The problem is happen with gtk client only, the propose patch will fix the issue.
As the ids should be check before passing this in to browse method directly.

review: Approve

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug was qualified as Already Fixed on Trunk (means that it was already fixed and merged in Trunk). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6829. By Priyesh (OpenERP) on 2012-06-05

[IMP] account: Added one more condition on ids(case:575460)

6828. By Priyesh (OpenERP) on 2012-06-04

[FIX] account: Checked type of ids and set it as list for future process(case:575460)

=== modified file 'account/account.py'
--- account/account.py	2012-03-22 12:24:19 +0000
+++ account/account.py	2012-06-05 06:21:21 +0000
@@ -824,6 +824,10 @@
 
         @return: Returns a list of tupples containing id, name
         """
+        if not ids:
+            return []
+        if isinstance(ids, (int, long)):
+            ids = [ids]
         result = self.browse(cr, user, ids, context=context)
         res = []
         for rs in result: