Review: Needs Fixing

Hello,

I have a remark about "for user in self.browse(cr, uid, ids, context=context)": 
we are talking about partners, not users :) . It is important to use variable 
names that clearly represent their content, because it allows to understand the 
code when reading it.

Thibault.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-social-partner-kjo/+merge/103414
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-openchatter.

_______________________________________________
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