Code review comment for lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-lga
Revision history for this message
| Lionel Sausin - Initiatives/Numérigraphe (ls-initiatives) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
I see 2 problems with replacing onchange_location with a domain:
- if the inventory is not exhaustive, the content of location_id is unspecified (probably the default, but we can't be sure)
- if users insist on entering a wrong location, openerp will propose to create it which is going to be confusing