From cb16936dfc2e5d4192ff867d4650e8c19e7c6024 Mon Sep 17 00:00:00 2001
From: Cedric Snauwaert <csn@openerp.com>
Date: Tue, 18 Jun 2013 14:48:15 +0200
Subject: [PATCH] [FIX]purchase_requisition: rename state "send to supplier"
 into "tender confirmed", remove constraint from draft to confirmed and add
 readonly attribute on all fields when tender has been confirmed.

bzr revid: csn@openerp.com-20130618124815-u3hchxnt74n0tzsi
---
 .../purchase_requisition.py                   |  7 +----
 .../purchase_requisition_view.xml             | 30 +++++++++----------
 2 files changed, 16 insertions(+), 21 deletions(-)

diff --git a/addons/purchase_requisition/purchase_requisition.py b/addons/purchase_requisition/purchase_requisition.py
index 1d988ba59d9..b159c0307d0 100644
--- a/addons/purchase_requisition/purchase_requisition.py
+++ b/addons/purchase_requisition/purchase_requisition.py
@@ -58,7 +58,7 @@ class purchase_requisition(osv.osv):
         'line_ids' : fields.one2many('purchase.requisition.line','requisition_id','Products to Purchase',states={'done': [('readonly', True)]}),
         'move_dest_id': fields.many2one('stock.move', 'Reservation Destination', ondelete='set null'),
         'warehouse_id': fields.many2one('stock.warehouse', 'Warehouse'),        
-        'state': fields.selection([('draft','Draft Tender'),('in_progress','Sent to Suppliers'),('open','Choosing Lines'),('done','PO Created'),('cancel','Cancelled')],
+        'state': fields.selection([('draft','Draft Tender'),('in_progress','Tender Confirmed'),('open','Choosing Lines'),('done','PO Created'),('cancel','Cancelled')],
             'Status', track_visibility='onchange', required=True),
         'multiple_rfq_per_supplier': fields.boolean('Multiple RFQ per supplier'),
         'account_analytic_id':fields.many2one('account.analytic.account', 'Analytic Account'),
@@ -93,11 +93,6 @@ class purchase_requisition(osv.osv):
         return self.write(cr, uid, ids, {'state': 'cancel'})
 
     def tender_in_progress(self, cr, uid, ids, context=None):
-        #check if all quotations are not in a draft state before going to that state
-        for purchase in self.browse(cr, uid, ids, context=context):
-            for purchase_id in purchase.purchase_ids:
-                if purchase_id.state == 'draft':
-                    raise osv.except_osv(_('Warning!'), _('You still have some quotation(s) in draft state.'))
         return self.write(cr, uid, ids, {'state':'in_progress'} ,context=context)
 
     def tender_open(self, cr, uid, ids, context=None):
diff --git a/addons/purchase_requisition/purchase_requisition_view.xml b/addons/purchase_requisition/purchase_requisition_view.xml
index 28b2fc1b8f3..6cd0f73bf7b 100644
--- a/addons/purchase_requisition/purchase_requisition_view.xml
+++ b/addons/purchase_requisition/purchase_requisition_view.xml
@@ -30,7 +30,7 @@
         <field name="arch" type="xml">
             <form string="Request for Tender" version="7.0">
             <header>              
-                <button name="sent_suppliers" states="draft" string="Send to Suppliers" class="oe_highlight"/>
+                <button name="sent_suppliers" states="draft" string="Confirm Tender" class="oe_highlight"/>
                 <button name="open_bid" states="in_progress" string="Choose supplier(s)" class="oe_highlight"/>
                 <button name="tender_reset" states="cancel" string="Reset to Draft" type="object" />
                 <button name="open_product_line" states="open" string="Choose product lines" type="object" class="oe_highlight" groups="purchase.group_advance_bidding"/>
@@ -47,27 +47,27 @@
                     <label for="name" class="oe_inline"/>
                 </div>
                 <h1>
-                    <field name="name" class="oe_inline"/>
+                    <field name="name" class="oe_inline" attrs="{'readonly': [('state','not in',('draft'))]}"/>
                 </h1>
                 <group>
                     <group>
-                        <field name="user_id"/>
-                        <field name="exclusive"/>
-                        <field name="account_analytic_id" colspan="2" groups="purchase.group_analytic_accounting"/>
-                        <field name="multiple_rfq_per_supplier"/>
+                        <field name="user_id" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="exclusive" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="account_analytic_id" colspan="2" groups="purchase.group_analytic_accounting" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="multiple_rfq_per_supplier" attrs="{'readonly': [('state','not in',('draft'))]}"/>
                     </group>
                     <group>
-                        <field name="date_start"/>
-                        <field name="date_end"/>
-                        <field name="schedule_date"/>
-                        <field name="origin" placeholder="e.g. OP0025"/>
-                        <field name="warehouse_id" widget="selection" groups="stock.group_locations"/>
-                        <field name="company_id" groups="base.group_multi_company" widget="selection"/>
+                        <field name="date_start" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="date_end" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="schedule_date" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="origin" placeholder="e.g. OP0025" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="warehouse_id" widget="selection" groups="stock.group_locations" attrs="{'readonly': [('state','not in',('draft'))]}"/>
+                        <field name="company_id" groups="base.group_multi_company" widget="selection" attrs="{'readonly': [('state','not in',('draft'))]}"/>
                     </group>
                 </group>
                 <notebook>
                     <page string="Products">
-                        <field name="line_ids">
+                        <field name="line_ids" attrs="{'readonly': [('state','not in',('draft'))]}">
                             <tree string="Products" editable="bottom">
                                 <field name="product_id" on_change="onchange_product_id(product_id,product_uom_id,parent.account_analytic_id,account_analytic_id,parent.schedule_date,schedule_date)"/>
                                 <field name="product_qty"/>
@@ -90,7 +90,7 @@
                         <div class="oe_right oe_button_box" style="margin-top: 10px">
                             <button name="%(action_purchase_requisition_partner)d" type="action"
                                 string="Request a Quotation" icon="gtk-execute"
-                                attrs="{'invisible': ['|', ('line_ids','=',[]),('state', 'in', ('open','done'))]}"/>
+                                attrs="{'invisible': ['|', ('line_ids','=',[]),('state', 'in', ('draft' ,'open','done'))]}"/>
                         </div>
                         <separator string="Requests for Quotation"/>
                         <field name="purchase_ids" readonly="1">
@@ -110,7 +110,7 @@
                             </tree>
                         </field>
                         <separator string="Terms and Conditions"/>
-                        <field name="description"/>
+                        <field name="description" attrs="{'readonly': [('state','not in',('draft'))]}"/>
                     </page>
                 </notebook>
             </sheet>