diff options
| author | Yannick Tivisse <yti@odoo.com> | 2016-09-27 13:09:12 +0200 |
|---|---|---|
| committer | Yannick Tivisse <yti@odoo.com> | 2016-09-27 13:09:12 +0200 |
| commit | a04522ac57944d5f49af8cda5dd480bc3e73e504 (patch) | |
| tree | 571a2ace76cbce9c3139a3834327ed8b9260a7ac | |
| parent | a22d206ac4362c4593ce7c7f3dfe68f5b9c2e43c (diff) | |
Revert "[IMP] stock: Remove domain on source/destination on Operation Details"
This reverts commit 7e6ad941cf4ce4b67cfaceb3fef7c740e5020f77.
The domain makes clear that the location to be chosen in is among the children of the location zones chosen in the picking. That way, if the stock would be calculated in the past (e.g. also for the stock valuation report), the system is going to think it is maybe still internal, while it went out, ..., so those domains are crucial.
| -rw-r--r-- | addons/stock/views/stock_pack_operation_views.xml | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/addons/stock/views/stock_pack_operation_views.xml b/addons/stock/views/stock_pack_operation_views.xml index 5f7e46850e6..e982fac90bf 100644 --- a/addons/stock/views/stock_pack_operation_views.xml +++ b/addons/stock/views/stock_pack_operation_views.xml @@ -21,8 +21,8 @@ </div> </group> <group string="Locations" groups="stock.group_stock_multi_locations"> - <field name="location_id"/> - <field name="location_dest_id"/> + <field name="location_id" domain="[('id', 'child_of', picking_source_location_id)]"/> + <field name="location_dest_id" domain="[('id', 'child_of', picking_destination_location_id)]"/> </group> <group string="Packages" groups="stock.group_tracking_lot"> <field name="package_id" attrs="{'required': [('product_id', '=', False)]}" domain="[('location_id', '=', location_id)]" context="{'location_id': location_id}"/> |