> 3) i don't like this syntax. Is there a reason to do it? a good one? if no, i 
> propose that you let it like that to keep the code as much consistent as 
> possible.
> 75    - self.write(cr, uid, [o.id], {'invoice_ids': [(4, inv_id)]})
> 78    + o.write({'invoice_ids': [(4, inv_id)]})

as now we use attributes of self.pool.get() for browse_record too, thats quite 
good, easy and consistent too (if we make practice of using that.)


-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724131-rpa/+merge/82254
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-724131-rpa.

_______________________________________________
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