pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] delivery: Many fields still available for editing after delivery is done

Browse Source 
carrier_id, carrier_tracking_ref and number_of_packages must be in readonly if
the stock.picking is in done state.

opw:683810
This commit is contained in:
Goffin Simon
2016-07-26 16:08:03 +02:00
parent 66ad091491
commit a7b68c288b
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/delivery/views/delivery_view.xml
+5 -5
View File
@@ -174,11 +174,11 @@
<separator string="Carrier Information"/>
<group>
<group name='carrier_data'>
<field name="carrier_id"/>
<field name="carrier_id" attrs="{'readonly': [('state', '=', 'done')]}"/>
<field name="delivery_type" attrs="{'invisible':True}"/>
<label for="carrier_tracking_ref"/>
<div>
<field name="carrier_tracking_ref" class="oe_inline"/>
<field name="carrier_tracking_ref" class="oe_inline" attrs="{'readonly': [('state', '=', 'done')]}"/>
<button type='object' class="fa fa-arrow-right oe_link" name="cancel_shipment" string="Cancel" attrs="{'invisible':['|','|','|',('carrier_tracking_ref','=',False),('delivery_type','in', ['fixed', 'base_on_rule']),('delivery_type','=',False),('state','not in',('done'))]}"/>
</div>
<field name="carrier_price"/>
@@ -189,7 +189,7 @@
<field name="weight" class="oe_inline"/>
<field name="weight_uom_id" nolabel="1" class="oe_inline"/>
</div>
<field name="number_of_packages"/>
<field name="number_of_packages" attrs="{'readonly': [('state', '=', 'done')]}"/>
</group>
</group>
</xpath>
@@ -276,11 +276,11 @@
<field name="arch" type="xml">
<xpath expr='//group[@name="carrier_data"]' position='replace'>
<group name='carrier_data'>
<field name="carrier_id" attrs="{'invisible': [('picking_type_code', '=', 'incoming')]}"/>
<field name="carrier_id" attrs="{'invisible': [('picking_type_code', '=', 'incoming')], 'readonly': [('state', '=', 'done')]}"/>
<field name="delivery_type" attrs="{'invisible':True}"/>
<label for="carrier_tracking_ref"/>
<div>
<field name="carrier_tracking_ref" class="oe_inline"/>
<field name="carrier_tracking_ref" class="oe_inline" attrs="{'readonly': [('state', '=', 'done')]}"/>
<button type='object' class="fa fa-arrow-right oe_link" name="cancel_shipment" string="Cancel" attrs="{'invisible':['|','|','|',('carrier_tracking_ref','=',False),('delivery_type','in', ['fixed', 'base_on_rule']),('delivery_type','=',False),('state','not in',('done'))]}"/>
</div>
<field name="carrier_price"/>