Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-787052-mma

Revision history for this message
Rucha (Open ERP) (rpa-openerp) wrote :

this seems not a proper fix as these changes are of purchase and in the dependency of this module (multi_company) there is no reference to purchase (though this fix doesn't break anything by traceback), so its good to fix the problem in purchase module itself,

- No need to change unique constraint of PO name (user can configure sequence so that its not conflicting with other names)

also this is not a good to refer company in data/update xml which is loaded in demo as it will give error of reference for the database without demo,

please improve it,
thanks

review: Needs Fixing

« Back to merge proposal