Review: Needs Fixing
Hello.
1. No need to check: if context is None:context = {}, if we don't manipulate
the context inside the method.
2. Use:
record = self.browse(cr, uid, ids[0], context=context)
Instead of this:
11 + record = self.browse(cr, uid, ids, context=context)
12 + record = record and record[0]
Thanks,
Divyesh
--
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-addons_23-mdi-hr-rpr/+merge/135129
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-addons/trunk-addons_23-mdi.
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : openerp-dev-gtk@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
