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

[Francesco Apruzzese]

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.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : openerp-community@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp