openerp-community-reviewer team mailing list archive

[Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>]

Review: Needs Fixing

Thanks for the PEP8! Checking again the code for the final inclusion, I have seen another potential problem: on the line 'if not default.get('ref')...' from the copy, this means that if the original record hasn't got ref, it is assigned a new one, but what we have to do is always assign a new sequence if requirements are met, independently from the old value.

Can you check it, please?

Regards.
-- 
https://code.launchpad.net/~initos.com/partner-contact-management/7.0/+merge/195066
Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management.