From d8d925d366bad7536a92acb75c8fdd8641a0cc80 Mon Sep 17 00:00:00 2001
From: William Henrotin <whe@odoo.com>
Date: Wed, 31 Jan 2018 14:45:25 +0100
Subject: [PATCH] [REF] purchase_requisition: vendor mandatory in blanket order

The vendor is now a mandatory field in the blanket order form
because it's directly linked to purchase orders. So Odoo needs
to know to which supplier the PO is addressed.
---
 .../models/purchase_requisition.py              | 17 +++++++++++++++++
 .../views/purchase_requisition_views.xml        |  3 ++-
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/addons/purchase_requisition/models/purchase_requisition.py b/addons/purchase_requisition/models/purchase_requisition.py
index 38e0172ed41..e8af2ab9ae6 100644
--- a/addons/purchase_requisition/models/purchase_requisition.py
+++ b/addons/purchase_requisition/models/purchase_requisition.py
@@ -66,6 +66,23 @@ class PurchaseRequisition(models.Model):
                               copy=False, default='draft')
     account_analytic_id = fields.Many2one('account.analytic.account', 'Analytic Account')
     picking_type_id = fields.Many2one('stock.picking.type', 'Operation Type', required=True, default=_get_picking_in)
+    is_quantity_copy = fields.Selection(related='type_id.quantity_copy', readonly=True)
+
+    @api.onchange('vendor_id')
+    def _onchange_vendor(self):
+        requisitions = self.env['purchase.requisition'].search([
+            ('vendor_id', '=', self.vendor_id.id),
+            ('state', '=', 'ongoing'),
+            ('type_id.quantity_copy', '=', 'none'),
+        ])
+        if any(requisitions):
+            title = _("Warning for %s") % self.vendor_id.name
+            message = _("%s has already an ongoing blanket order") % self.vendor_id.name
+            warning = {
+                'title': title,
+                'message': message
+            }
+            return {'warning': warning}
 
     @api.multi
     @api.depends('purchase_ids')
diff --git a/addons/purchase_requisition/views/purchase_requisition_views.xml b/addons/purchase_requisition/views/purchase_requisition_views.xml
index 843714382f7..89a0daaa49d 100644
--- a/addons/purchase_requisition/views/purchase_requisition_views.xml
+++ b/addons/purchase_requisition/views/purchase_requisition_views.xml
@@ -149,9 +149,10 @@
                 </div>
                 <group>
                     <group>
+                        <field name="is_quantity_copy" invisible='1'/>
                         <field name="user_id" attrs="{'readonly': [('state','not in',('draft','in_progress','open'))]}"/>
                         <field name="type_id" attrs="{'readonly': [('state','not in',('draft','in_progress','open'))]}" widget="selection"/>
-                        <field name="vendor_id" context="{'search_default_supplier':1, 'default_supplier':1, 'default_customer':0}" domain="[('supplier','=',True)]"/>
+                        <field name="vendor_id" context="{'search_default_supplier':1, 'default_supplier':1, 'default_customer':0}" domain="[('supplier','=',True)]" attrs="{'required': [('is_quantity_copy', '=', 'none')]}"/>
                         <field name="account_analytic_id" colspan="2" groups="analytic.group_analytic_accounting" attrs="{'readonly': [('state','not in',('draft'))]}"/>
                     </group>
                     <group>