← Back to team overview

openerp-dev-web team mailing list archive

lp:~openerp-dev/openobject-addons/trunk-sale_fields_disabled_on_state-bugfix-ind-ach into lp:~openerp-dev/openobject-addons/trunk-bugfixes-Ind

 

Anup(OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/trunk-sale_fields_disabled_on_state-bugfix-ind-ach into lp:~openerp-dev/openobject-addons/trunk-bugfixes-Ind.

Requested reviews:
  Husen Daudi (husendaudi)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-sale_fields_disabled_on_state-bugfix-ind-ach/+merge/58289

Hello,

   The fields payment term and fiscal position should be disabled when sale order is confirmed like other configuration field. 
   
   As they are related with invoice we can keep it enable till an invoice is created.

Please share your views.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-sale_fields_disabled_on_state-bugfix-ind-ach/+merge/58289
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bugfixes-Ind.
=== modified file 'sale/sale_view.xml'
--- sale/sale_view.xml	2011-03-28 15:46:01 +0000
+++ sale/sale_view.xml	2011-04-19 12:22:25 +0000
@@ -232,8 +232,8 @@
                             </group>
                             <group colspan="2" col="2" groups="base.group_extended">
                                 <separator string="Conditions" colspan="2"/>
-                                <field name="payment_term" widget="selection"/>
-                                <field name="fiscal_position" widget="selection"/>
+                                <field name="payment_term" widget="selection" attrs="{'readonly':[('state','!=','draft')]}"/>
+                                <field name="fiscal_position" widget="selection" attrs="{'readonly':[('state','!=','draft')]}"/>
                                 <field name="company_id" widget="selection" groups="base.group_multi_company"/>
                             </group>
                             <group colspan="2" col="2" groups="base.group_extended">


Follow ups