Code review comment for lp:~pedro.baeza/ocb-addons/7.0-bug-1253701
Revision history for this message
| Stefan Rijnhart (Opener) (stefan-opener) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Hi Pedro,
thank you for this fix! Changing the rule seems like a good idea. Seems to me that the limits to multi-company BOM are any company-specific routes.
However, putting this incidental security rule in an updating data block would be a deviation from the OpenERP standard to have security rules in noupdate so that they can be changed by users and consultants. I don't think it is a good idea to introduce such inconsistency.