From 4fe36979f824d0b9b3680e5817bb524ab2eef5df Mon Sep 17 00:00:00 2001
From: Andrii Skrypka <andrijskrypa@ukr.net>
Date: Thu, 21 Nov 2019 20:32:28 +0200
Subject: [PATCH 1/3] [IMP] purchase_request: add button Create RFQ on purchase
 request form view

---
 purchase_request/__manifest__.py                |  2 +-
 .../views/purchase_request_view.xml             |  3 +++
 ...purchase_request_line_make_purchase_order.py | 17 ++++++++++-------
 3 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/purchase_request/__manifest__.py b/purchase_request/__manifest__.py
index d7c8265afba..81a2eb97d8f 100644
--- a/purchase_request/__manifest__.py
+++ b/purchase_request/__manifest__.py
@@ -21,13 +21,13 @@
         "data/purchase_request_sequence.xml",
         "data/purchase_request_data.xml",
         "reports/report_purchase_request.xml",
+        "wizard/purchase_request_line_make_purchase_order_view.xml",
         "views/purchase_request_view.xml",
         "views/purchase_request_line_view.xml",
         "views/purchase_request_report.xml",
         "views/product_template.xml",
         "views/purchase_order_view.xml",
         "views/stock_move_views.xml",
-        "wizard/purchase_request_line_make_purchase_order_view.xml",
     ],
     'demo': [
         "demo/purchase_request_demo.xml",
diff --git a/purchase_request/views/purchase_request_view.xml b/purchase_request/views/purchase_request_view.xml
index fd86a4cd247..c58419175df 100644
--- a/purchase_request/views/purchase_request_view.xml
+++ b/purchase_request/views/purchase_request_view.xml
@@ -23,6 +23,9 @@
                         type="object"
                         class="oe_highlight"
                         groups="purchase_request.group_purchase_request_manager"/>
+                <button name="%(action_purchase_request_line_make_purchase_order)d"
+                        states="approved"
+                        string="Create RFQ" type="action"/>
                 <button name="button_done"
                         states="approved"
                         string="Done"
diff --git a/purchase_request/wizard/purchase_request_line_make_purchase_order.py b/purchase_request/wizard/purchase_request_line_make_purchase_order.py
index 00a4fdbb660..c52d8c0b701 100644
--- a/purchase_request/wizard/purchase_request_line_make_purchase_order.py
+++ b/purchase_request/wizard/purchase_request_line_make_purchase_order.py
@@ -94,17 +94,20 @@ def get_items(self, request_line_ids):
 
     @api.model
     def default_get(self, fields):
-        res = super(PurchaseRequestLineMakePurchaseOrder, self).default_get(
-            fields)
-        request_line_obj = self.env['purchase.request.line']
-        request_line_ids = self.env.context.get('active_ids', False)
+        res = super().default_get(fields)
         active_model = self.env.context.get('active_model', False)
+        request_line_ids = []
+        if active_model == 'purchase.request.line':
+            request_line_ids += self.env.context.get('active_ids', [])
+        elif active_model == 'purchase.request':
+            request_ids = self.env.context.get('active_ids', False)
+            request_line_ids += self.env[active_model].browse(
+                request_ids).mapped('line_ids.id')
         if not request_line_ids:
             return res
-        assert active_model == 'purchase.request.line', \
-            'Bad context propagation'
         res['item_ids'] = self.get_items(request_line_ids)
-        request_lines = request_line_obj.browse(request_line_ids)
+        request_lines = self.env['purchase.request.line'].browse(
+            request_line_ids)
         supplier_ids = request_lines.mapped('supplier_id').ids
         if len(supplier_ids) == 1:
             res['supplier_id'] = supplier_ids[0]

From 2d9ac2cb443d1afeebf8aa175d86b5be01f033d4 Mon Sep 17 00:00:00 2001
From: Andrii Skrypka <andrijskrypa@ukr.net>
Date: Thu, 21 Nov 2019 20:58:39 +0200
Subject: [PATCH 2/3] [FIX] purchase_request: fix test
 test_purchase_request_to_rfq_minimum_order_qty_existing_po

---
 purchase_request/tests/test_purchase_request_to_rfq.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/purchase_request/tests/test_purchase_request_to_rfq.py b/purchase_request/tests/test_purchase_request_to_rfq.py
index c64bd63645b..d4c51c30c4d 100644
--- a/purchase_request/tests/test_purchase_request_to_rfq.py
+++ b/purchase_request/tests/test_purchase_request_to_rfq.py
@@ -264,7 +264,7 @@ def test_purchase_request_to_rfq_minimum_order_qty_existing_po(self):
         }
         wiz_id = self.wiz.with_context(
             active_model="purchase.request.line",
-            active_ids=purchase_request_line1.id).create(vals)
+            active_ids=[purchase_request_line1.id]).create(vals)
         wiz_id.make_purchase_order()
         po = purchase_request_line1.purchase_lines[0].order_id
         # Create Purchase Request
@@ -291,7 +291,7 @@ def test_purchase_request_to_rfq_minimum_order_qty_existing_po(self):
         }
         wiz_id = self.wiz.with_context(
             active_model="purchase.request.line",
-            active_ids=purchase_request_line2.id).create(vals)
+            active_ids=[purchase_request_line2.id]).create(vals)
         wiz_id.make_purchase_order()
         # Check Purchase qty should be 6
         po_line = purchase_request_line2.purchase_lines[0]

From 045eefe9459636b13f4d6705a5622f59c4e413dc Mon Sep 17 00:00:00 2001
From: Andrii Skrypka <andrijskrypa@ukr.net>
Date: Tue, 26 Nov 2019 17:57:29 +0200
Subject: [PATCH 3/3] [ADD] purchase_request: test for wizard default_get

---
 .../tests/test_purchase_request_to_rfq.py     | 37 +++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/purchase_request/tests/test_purchase_request_to_rfq.py b/purchase_request/tests/test_purchase_request_to_rfq.py
index d4c51c30c4d..5c0a08c03e6 100644
--- a/purchase_request/tests/test_purchase_request_to_rfq.py
+++ b/purchase_request/tests/test_purchase_request_to_rfq.py
@@ -27,6 +27,43 @@ def setUp(self):
             'product_tmpl_id': self.service_product.product_tmpl_id.id,
         })
 
+    def test_wizard_default_get(self):
+        # Check that correct create items by purchase.request
+        vals = {
+            'picking_type_id': self.env.ref('stock.picking_type_in').id,
+            'requested_by': SUPERUSER_ID,
+            'line_ids': [[0, 0, {
+                'product_id': self.env.ref('product.product_product_13').id,
+                'product_uom_id': self.env.ref('uom.product_uom_unit').id,
+                'product_qty': 2.0,
+            }]],
+        }
+        purchase_request1 = self.purchase_request.create(vals)
+        purchase_request1.button_approved()
+        vals = {
+            'supplier_id': self.env.ref('base.res_partner_1').id,
+            'line_ids': [[0, 0, {
+                'product_id': self.env.ref('product.product_product_13').id,
+                'product_uom_id': self.env.ref('uom.product_uom_unit').id,
+                'product_qty': 2.0,
+            }]],
+        }
+        purchase_request2 = self.purchase_request.create(vals)
+        vals = {
+            'supplier_id': self.env.ref('base.res_partner_1').id,
+        }
+        purchase_request1.button_approved()
+        purchase_request2.button_approved()
+        wiz_id = self.wiz.with_context(
+            active_model="purchase.request",
+            active_ids=[purchase_request1.id,
+                        purchase_request2.id]).create(vals)
+        (purchase_request1 | purchase_request2).mapped('line_ids')
+        self.assertEquals(
+            (purchase_request1 | purchase_request2).mapped('line_ids'),
+            wiz_id.item_ids.mapped('line_id'),
+            'Should have same purchase request lines')
+
     def test_purchase_request_to_purchase_rfq(self):
         vals = {
             'picking_type_id': self.env.ref('stock.picking_type_in').id,