openerp-dev-web team mailing list archive

["Somesh Khare\(OpenERP\)" <skh@xxxxxxxxxxx>]

Somesh Khare(OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/6.0-opw-5802-skh into lp:openobject-addons/6.0.

Requested reviews:
  Priyesh (Open ERP) (pso-openerp)
  Jay Vora (OpenERP) (jvo-openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-5802-skh/+merge/61400

Hello,

There is a issue in the sale module of the stable6.0. When we create sale order and confirm the order, currently there is attrs on invoice_quantity in sale_view.xml as "attrs="{'readonly':[('order_policy','=','prepaid'),('order_policy','=','picking')]}" which will never comes true. 

I have fixed the issue by setting attrs="{'readonly':['|',('order_policy','in',['prepaid','picking']),('state','!=','draft')]}".

Now the issue got fixed in the sale module.

Thanks 
Somesh Khare
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-5802-skh/+merge/61400
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/6.0-opw-5802-skh.

=== modified file 'sale/sale_view.xml'
--- sale/sale_view.xml	2011-01-19 09:51:59 +0000
+++ sale/sale_view.xml	2011-05-18 13:13:41 +0000
@@ -223,7 +223,7 @@
                                 <field name="incoterm" widget="selection"/>
                                 <field name="picking_policy" required="True"/>
                                 <field name="order_policy" on_change="shipping_policy_change(order_policy)"/>
-                                <field name="invoice_quantity" attrs="{'readonly':[('order_policy','=','prepaid'),('order_policy','=','picking')]}"/>
+                                <field name="invoice_quantity" attrs="{'readonly':['|',('order_policy','in',['prepaid','picking']),('state','!=','draft')]}"/>
                             </group>
                             <group colspan="2" col="2">
                                 <separator string="References" colspan="2"/>