← Back to team overview

openerp-community team mailing list archive

Re: [Merge] lp:~therp-nl/server-env-tools/6.1-mass_editing-fix_dataloss into lp:~openerp-community/server-env-tools/6.1-mass_editing

 


Il 17/03/2014 17:38, Yannick Vaucher @ Camptocamp ha scritto:
Review: Needs Fixing

Instead of dict.update({split_key: m2m_list})

You can simply write

dict['split_key'] = m2m_list

It might be a bit more readable

If you use dict['split_key'], are you sure 'split_key' key exists in dict? If you use update, python create the key for you if not exist or update it if exists yet.


References