Code review comment for lp:~camptocamp/openerp-humanitarian-ngo/purchase-wkfl-fix_copy_origin-rde

Revision history for this message
Leonardo Pistone (lepistone) wrote :

Romain,

I suggest doing that another way, that I find more clear.

if default is None:
    default = {}

initial_origin = default.get('origin')

...super...

if initial_origin:
   ...write...

thanks!

review: Needs Fixing (code review)

« Back to merge proposal