Revert "[IMP] stock: Remove domain on source/destination on Operation Details"

This reverts commit 7e6ad941.

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.

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}"/>