Ravi Gohil (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-server/6.0-opw-441976-rgo into 
lp:openobject-server/6.0.

Requested reviews:
  Naresh(OpenERP) (nch-openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-441976-rgo/+merge/94936

Hello,

When an Integer 'ids' is supplied as parameter of read() method, a traceback is 
caused because a dict will be return by the execution of next statement(when 
passed 'ids' as integer) that is 'res=super(osv.osv, self).read(cr, user, ids, 
fields, context, load)' whereas a list of dict is expected to execute the code 
within next looping statement.

I modified the code accordingly. Kindly review it.

Thanks.


-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-441976-rgo/+merge/94936
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/6.0-opw-441976-rgo.
=== modified file 'bin/addons/base/res/res_currency.py'
--- bin/addons/base/res/res_currency.py	2011-10-12 14:15:50 +0000
+++ bin/addons/base/res/res_currency.py	2012-02-28 11:00:26 +0000
@@ -78,6 +78,8 @@
     _constraints = [(_check_rounding, "The rounding factor cannot be 0 !", ['rounding'])]
 
     def read(self, cr, user, ids, fields=None, context=None, load='_classic_read'):
+        if not isinstance(ids, list):
+            ids = [ids]
         res=super(osv.osv, self).read(cr, user, ids, fields, context, load)
         for r in res:
             if r.__contains__('rate_ids'):

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to