Review: Needs Fixing
you can optimize your code by having some variable for  
o.pricelist_id.currency_id.id, o.partner_id.id.

you can also check if len([ value for value in res_partner_pricelist.values() 
if len(value)!= 1]) != 0: condition out side the for loop (Don't forget to use 
if grouped too with above condition) 
And !=0 for len() you can have condition simple like if len(x): 

Warning message while raising should be improved.

Thanks,
Mustufa
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-804265-mtr/+merge/69059
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-804265-mtr.

_______________________________________________
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