From 62e33d0441cac6f1585ccc3acffc1f1fea42aaad Mon Sep 17 00:00:00 2001
From: Matt Taylor <matt454357@gmail.com>
Date: Mon, 10 Apr 2017 17:27:08 -0600
Subject: [PATCH 1/8] [FIX] Add support for the sale_margin module

Adding configured sales order lines fails when sale_margin module is installed.
The sale_margin module requires the sales order line to include product_uom:
odoo/addons/sale_margin/models/sale_order.py
---
 product_configurator_wizard/wizard/product_configurator.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/product_configurator_wizard/wizard/product_configurator.py b/product_configurator_wizard/wizard/product_configurator.py
index 954e22ba..c96d0bab 100644
--- a/product_configurator_wizard/wizard/product_configurator.py
+++ b/product_configurator_wizard/wizard/product_configurator.py
@@ -792,10 +792,8 @@ def _extra_line_values(self, so, product, new=True):
         created or edited lines."""
         vals = {}
         if new:
-            vals.update({
-                'name': product.display_name,
-                'product_uom': product.uom_id.id,
-            })
+            vals.update({'name': product.display_name})
+            vals.update({'product_uom': product.uom_id.id})
         return vals
 
     @api.multi

From 1c63da6f9517ca5fde25d54dba1d4aeaf61ca86e Mon Sep 17 00:00:00 2001
From: Matt Taylor <matt454357@gmail.com>
Date: Tue, 11 Apr 2017 13:18:00 -0600
Subject: [PATCH 2/8] Pass single dict to update method

---
 product_configurator_wizard/wizard/product_configurator.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/product_configurator_wizard/wizard/product_configurator.py b/product_configurator_wizard/wizard/product_configurator.py
index c96d0bab..954e22ba 100644
--- a/product_configurator_wizard/wizard/product_configurator.py
+++ b/product_configurator_wizard/wizard/product_configurator.py
@@ -792,8 +792,10 @@ def _extra_line_values(self, so, product, new=True):
         created or edited lines."""
         vals = {}
         if new:
-            vals.update({'name': product.display_name})
-            vals.update({'product_uom': product.uom_id.id})
+            vals.update({
+                'name': product.display_name,
+                'product_uom': product.uom_id.id,
+            })
         return vals
 
     @api.multi

From 8fd4cec2a187549a2dcbc9b15a42b2b9f2b07ca4 Mon Sep 17 00:00:00 2001
From: Matt Taylor <matt454357@gmail.com>
Date: Mon, 10 Apr 2017 17:27:08 -0600
Subject: [PATCH 3/8] [FIX] Add support for the sale_margin module

Adding configured sales order lines fails when sale_margin module is installed.
The sale_margin module requires the sales order line to include product_uom:
odoo/addons/sale_margin/models/sale_order.py
---
 product_configurator_wizard/wizard/product_configurator.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/product_configurator_wizard/wizard/product_configurator.py b/product_configurator_wizard/wizard/product_configurator.py
index 954e22ba..c96d0bab 100644
--- a/product_configurator_wizard/wizard/product_configurator.py
+++ b/product_configurator_wizard/wizard/product_configurator.py
@@ -792,10 +792,8 @@ def _extra_line_values(self, so, product, new=True):
         created or edited lines."""
         vals = {}
         if new:
-            vals.update({
-                'name': product.display_name,
-                'product_uom': product.uom_id.id,
-            })
+            vals.update({'name': product.display_name})
+            vals.update({'product_uom': product.uom_id.id})
         return vals
 
     @api.multi

From ad76678311eb0138235d8288ede13498e1f162b7 Mon Sep 17 00:00:00 2001
From: Eric Lemire <el@microcom.ca>
Date: Thu, 23 Mar 2017 11:46:19 -0400
Subject: [PATCH 4/8] Create new module to add product_configurator to purchase

---
 product_configurator/models/product.py        |  2 +
 product_configurator/views/product_view.xml   |  3 +
 .../wizard/product_configurator.py            | 59 +++++++++++++++----
 3 files changed, 51 insertions(+), 13 deletions(-)

diff --git a/product_configurator/models/product.py b/product_configurator/models/product.py
index 3ceb2915..3c283914 100644
--- a/product_configurator/models/product.py
+++ b/product_configurator/models/product.py
@@ -11,6 +11,8 @@ class ProductTemplate(models.Model):
 
     config_ok = fields.Boolean(string='Can be Configured')
 
+    reuse_variant = fields.Boolean(string='Reuse variants if exists')
+
     config_line_ids = fields.One2many(
         comodel_name='product.config.line',
         inverse_name='product_tmpl_id',
diff --git a/product_configurator/views/product_view.xml b/product_configurator/views/product_view.xml
index 1519dece..98dd45fa 100644
--- a/product_configurator/views/product_view.xml
+++ b/product_configurator/views/product_view.xml
@@ -43,6 +43,9 @@
             <!-- TODO: Apply domains so only values from template are available -->
             <xpath expr="//notebook/page[@name='variants']" position="after">
                 <page string="Configurator" attrs="{'invisible': [('config_ok','=',False)]}">
+                <separator colspan="4" string="Configuration Options"/>
+                <field name="reuse_variant"/>
+                <label for="reuse_variant"/>
                 <separator colspan="4" string="Configuration Restrictions"/>
                 <field name="config_line_ids"
                        attrs="{'readonly': [('attribute_line_ids','=',[])]}"
diff --git a/product_configurator_wizard/wizard/product_configurator.py b/product_configurator_wizard/wizard/product_configurator.py
index c96d0bab..fc34d330 100644
--- a/product_configurator_wizard/wizard/product_configurator.py
+++ b/product_configurator_wizard/wizard/product_configurator.py
@@ -2,9 +2,11 @@
 
 from lxml import etree
 
+from datetime import datetime
 from odoo.osv import orm
 from odoo import models, fields, api, _
 from odoo.exceptions import Warning, ValidationError
+from odoo.tools import DEFAULT_SERVER_DATETIME_FORMAT
 
 
 class FreeSelection(fields.Selection):
@@ -234,6 +236,10 @@ def onchange(self, values, field_name, field_onchange):
         comodel_name='sale.order.line',
         readonly=True,
     )
+    purchase_order_line_id = fields.Many2one(
+        comodel_name='purchase.order.line',
+        readonly=True,
+    )
 
     @api.model
     def fields_get(self, allfields=None, attributes=None):
@@ -817,6 +823,10 @@ def action_config_done(self):
                 order_line_vals = self._extra_line_values(
                     self.order_line_id.order_id, self.product_id, new=False)
                 self.order_line_id.write(order_line_vals)
+            if self.purchase_order_line_id:
+                self.purchase_order_line_id.write(self._extra_line_values(self.purchase_order_line_id.order_id,
+                                                                 self.product_id,
+                                                                 new=False))
             self.unlink()
             return
         #
@@ -827,23 +837,46 @@ def action_config_done(self):
         # In the meantime, at least make sure that a validation
         # error legitimately raised in a nested routine
         # is passed through.
-        try:
-            variant = self.product_tmpl_id.create_variant(
-                self.value_ids.ids, custom_vals)
-        except ValidationError:
-            raise
-        except:
-            raise ValidationError(
-                _('Invalid configuration! Please check all '
-                  'required steps and fields.')
-            )
+        domain = [['product_tmpl_id', '=', self.product_tmpl_id.id]]
+
+        template_products = self.env['product.product'].search(domain)
+        product_found = False
 
-        so = self.env['sale.order'].browse(self.env.context.get('active_id'))
+        if self.product_tmpl_id.reuse_variant:
+            for product in template_products:
+                if product.attribute_value_ids == self.value_ids:
+                    product_found = product
+                    break
+
+        if product_found:
+            variant = product_found
+            price = product_found.standard_price
+            uom = product_found.uom_id.id
+        else:
+            try:
+                variant = self.product_tmpl_id.create_variant(
+                    self.value_ids.ids, custom_vals)
+                price = self.product_tmpl_id.standard_price
+                uom = self.product_tmpl_id.uom_id.id
+            except ValidationError:
+                raise
+            except:
+                raise ValidationError(
+                    _('Invalid configuration! Please check all '
+                      'required steps and fields.')
+                )
+
+        if self.env.context.get('active_model') == 'purchase.order':
+            order = self.env['purchase.order'].browse(self.env.context.get('active_id'))
+            line_vals = {'product_id': variant.id, 'date_planned': datetime.today().strftime(DEFAULT_SERVER_DATETIME_FORMAT),
+                         'product_uom': uom, 'price_unit': price, 'product_qty': 1}
+        else:
+            order = self.env['sale.order'].browse(self.env.context.get('active_id'))
+            line_vals = {'product_id': variant.id}
 
-        line_vals = {'product_id': variant.id}
         line_vals.update(self._extra_line_values(so, variant, new=True))
 
-        so.write({
+        order.write({
             'order_line': [(0, 0, line_vals)]
         })
 

From 8cf80aef23ab39b3b8dc377da0aa4f65cbab9ab7 Mon Sep 17 00:00:00 2001
From: Eric Lemire <el@microcom.ca>
Date: Thu, 23 Mar 2017 13:11:39 -0400
Subject: [PATCH 5/8] Move some code so that the Wizard mode can run without
 purchase module

---
 .../models/product_configurator.py            | 40 +++++++++++++++++++
 .../wizard/product_configurator.py            | 10 +----
 2 files changed, 41 insertions(+), 9 deletions(-)
 create mode 100644 product_configurator_purchase/models/product_configurator.py

diff --git a/product_configurator_purchase/models/product_configurator.py b/product_configurator_purchase/models/product_configurator.py
new file mode 100644
index 00000000..7fec5940
--- /dev/null
+++ b/product_configurator_purchase/models/product_configurator.py
@@ -0,0 +1,40 @@
+# -*- coding: utf-8 -*-
+
+from odoo import models, fields, api
+
+
+class ProductConfigurator(models.Model):
+    _inherit = 'product.configurator'
+
+    purchase_order_line_id = fields.Many2one(
+        comodel_name='purchase.order.line',
+        readonly=True,
+    )
+
+    @api.multi
+    def action_config_done(self):
+        """This fuction are copied from product_configurator_wizard.ProductConfigurator
+           We only add one "if" required for purchase_order
+           Parse values and execute final code before closing the wizard"""
+        custom_vals = {
+            l.attribute_id.id:
+                l.value or l.attachment_ids for l in self.custom_value_ids
+        }
+
+        if self.product_id:
+            remove_cv_links = map(lambda cv: (2, cv), self.product_id.value_custom_ids.ids)
+            new_cv_links = self.product_id.product_tmpl_id.encode_custom_values(custom_vals)
+            self.product_id.write({
+                'attribute_value_ids': [(6, 0, self.value_ids.ids)],
+                'value_custom_ids':  remove_cv_links + new_cv_links,
+            })
+            if self.order_line_id:
+                self.order_line_id.write(self._extra_line_values(self.order_line_id.order_id,
+                                                                 self.product_id,
+                                                                 new=False))
+            if self.purchase_order_line_id:
+                self.purchase_order_line_id.write(self._extra_line_values(self.purchase_order_line_id.order_id,
+                                                                 self.product_id,
+                                                                 new=False))
+            self.unlink()
+            return
diff --git a/product_configurator_wizard/wizard/product_configurator.py b/product_configurator_wizard/wizard/product_configurator.py
index fc34d330..a1238077 100644
--- a/product_configurator_wizard/wizard/product_configurator.py
+++ b/product_configurator_wizard/wizard/product_configurator.py
@@ -236,10 +236,6 @@ def onchange(self, values, field_name, field_onchange):
         comodel_name='sale.order.line',
         readonly=True,
     )
-    purchase_order_line_id = fields.Many2one(
-        comodel_name='purchase.order.line',
-        readonly=True,
-    )
 
     @api.model
     def fields_get(self, allfields=None, attributes=None):
@@ -823,10 +819,6 @@ def action_config_done(self):
                 order_line_vals = self._extra_line_values(
                     self.order_line_id.order_id, self.product_id, new=False)
                 self.order_line_id.write(order_line_vals)
-            if self.purchase_order_line_id:
-                self.purchase_order_line_id.write(self._extra_line_values(self.purchase_order_line_id.order_id,
-                                                                 self.product_id,
-                                                                 new=False))
             self.unlink()
             return
         #
@@ -874,7 +866,7 @@ def action_config_done(self):
             order = self.env['sale.order'].browse(self.env.context.get('active_id'))
             line_vals = {'product_id': variant.id}
 
-        line_vals.update(self._extra_line_values(so, variant, new=True))
+        line_vals.update(self._extra_line_values(order, variant, new=True))
 
         order.write({
             'order_line': [(0, 0, line_vals)]

From 7315133e9a0efb95cb26c4bb7710e0038b6a1359 Mon Sep 17 00:00:00 2001
From: Eric Lemire <el@microcom.ca>
Date: Fri, 24 Mar 2017 16:30:32 -0400
Subject: [PATCH 6/8] Add missing files for purchase

---
 product_configurator_purchase/README.md       | 13 +++++
 product_configurator_purchase/__init__.py     |  3 ++
 product_configurator_purchase/__manifest__.py | 29 +++++++++++
 .../models/__init__.py                        |  3 ++
 .../models/purchase.py                        | 50 +++++++++++++++++++
 .../static/description/index.html             | 16 ++++++
 .../views/purchase_views.xml                  | 49 ++++++++++++++++++
 7 files changed, 163 insertions(+)
 create mode 100644 product_configurator_purchase/README.md
 create mode 100644 product_configurator_purchase/__init__.py
 create mode 100644 product_configurator_purchase/__manifest__.py
 create mode 100644 product_configurator_purchase/models/__init__.py
 create mode 100644 product_configurator_purchase/models/purchase.py
 create mode 100644 product_configurator_purchase/static/description/index.html
 create mode 100644 product_configurator_purchase/views/purchase_views.xml

diff --git a/product_configurator_purchase/README.md b/product_configurator_purchase/README.md
new file mode 100644
index 00000000..65ab0cab
--- /dev/null
+++ b/product_configurator_purchase/README.md
@@ -0,0 +1,13 @@
+#Odoo Product Configurator Purchase
+
+This module is an extension to product configurator to add widjet to purchase and add possibility to choose a product variant instead of always create a new one.
+
+Features
+========
+
+- Add configure button to Purchase order.
+
+Usage
+=====
+
+To complete
diff --git a/product_configurator_purchase/__init__.py b/product_configurator_purchase/__init__.py
new file mode 100644
index 00000000..cde864ba
--- /dev/null
+++ b/product_configurator_purchase/__init__.py
@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+from . import models
diff --git a/product_configurator_purchase/__manifest__.py b/product_configurator_purchase/__manifest__.py
new file mode 100644
index 00000000..61f31cdb
--- /dev/null
+++ b/product_configurator_purchase/__manifest__.py
@@ -0,0 +1,29 @@
+# -*- coding: utf-8 -*-
+
+{
+    'name': 'Product Configurator Purchase',
+    'version': '10.0',
+    'category': 'Generic Modules/Base',
+    'summary': 'product configuration interface for purchase',
+    'description': """
+        Purchase Product Configurator
+    """,
+    'author': 'Microcom',
+    'license': 'AGPL-3',
+    'website': 'http://www.microcom.ca/',
+    'depends': [
+        'purchase',
+        'product_configurator',
+        'product_configurator_wizard',
+    ],
+    "data": [
+        'views/purchase_views.xml',
+    ],
+    'demo': [
+    ],
+    'images': [
+    ],
+    'test': [],
+    'installable': True,
+    'auto_install': False,
+}
diff --git a/product_configurator_purchase/models/__init__.py b/product_configurator_purchase/models/__init__.py
new file mode 100644
index 00000000..b46d7f44
--- /dev/null
+++ b/product_configurator_purchase/models/__init__.py
@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+from . import purchase
diff --git a/product_configurator_purchase/models/purchase.py b/product_configurator_purchase/models/purchase.py
new file mode 100644
index 00000000..ac120770
--- /dev/null
+++ b/product_configurator_purchase/models/purchase.py
@@ -0,0 +1,50 @@
+# -*- coding: utf-8 -*-
+
+from odoo import models, fields, api
+
+
+class PurchaseOrderLine(models.Model):
+    _inherit = 'purchase.order.line'
+
+    custom_value_ids = fields.One2many(
+        comodel_name='product.attribute.value.custom',
+        inverse_name='product_id',
+        related="product_id.value_custom_ids",
+        string="Custom Values"
+    )
+
+    product_id = fields.Many2one(domain=[('config_ok', '=', False)])
+
+    config_ok = fields.Boolean(
+        related='product_id.config_ok',
+        string="Configurable"
+    )
+
+    @api.multi
+    def reconfigure_product(self):
+        """ Creates and launches a product configurator wizard with a linked
+        template and variant in order to re-configure a existing product. It is
+        esetially a shortcut to pre-fill configuration data of a variant"""
+
+        cfg_steps = self.product_id.product_tmpl_id.config_step_line_ids
+        active_step = str(cfg_steps[0].id) if cfg_steps else 'configure'
+
+        wizard_obj = self.env['product.configurator']
+        wizard = wizard_obj.create({
+            'product_id': self.product_id.id,
+            'state': active_step,
+            'purchase_order_line_id': self.id,
+        })
+
+        return {
+            'type': 'ir.actions.act_window',
+            'res_model': 'product.configurator',
+            'name': "Configure Product",
+            'view_mode': 'form',
+            'context': dict(
+                self.env.context,
+                wizard_id=wizard.id,
+            ),
+            'target': 'new',
+            'res_id': wizard.id,
+        }
diff --git a/product_configurator_purchase/static/description/index.html b/product_configurator_purchase/static/description/index.html
new file mode 100644
index 00000000..10021906
--- /dev/null
+++ b/product_configurator_purchase/static/description/index.html
@@ -0,0 +1,16 @@
+<section class="oe_container">
+    <div class="oe_row oe_spaced">
+        <div class="oe_span12">
+            <h2 class="oe_slogan">Product Configurator Purchase</h2>
+            <h3 class="oe_slogan">Generate products on-demand, easy and error-free</h3>
+        </div>
+    </div>
+</section>
+
+<section class="oe_container mt64">
+    <div class="oe_row oe_spaced">
+        <h2 class="oe_slogan">Configure products directly in your purchase orders</h2>
+        <h3 class="oe_slogan">This module is an extention to product configurator and Product configurator Wizard</h3>
+        <h3 class="oe_slogan">It allow to configure product from purchase orders in addition to Sale orders</h3>
+    </div>
+</section>
diff --git a/product_configurator_purchase/views/purchase_views.xml b/product_configurator_purchase/views/purchase_views.xml
new file mode 100644
index 00000000..328f7503
--- /dev/null
+++ b/product_configurator_purchase/views/purchase_views.xml
@@ -0,0 +1,49 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <!-- Add Configure Product button on sale order form view -->
+
+    <record id="action_wizard_product_purchase_configurator" model="ir.actions.act_window">
+        <field name="name">Configure Product</field>
+        <field name="res_model">product.configurator</field>
+        <field name="view_mode">form</field>
+        <field name="view_id" ref="product_configurator_wizard.product_configurator_form"/>
+        <field name="target">new</field>
+    </record>
+
+    <record id="purchase_order_form_config" model="ir.ui.view">
+        <field name="name">purchase.order.form.config</field>
+        <field name="model">purchase.order</field>
+        <field name="inherit_id" ref="purchase.purchase_order_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//field[@name='order_line']" position="before">
+                <button name="%(action_wizard_product_purchase_configurator)s"
+                        states="draft,sent"
+                        class="oe_highlight"
+                        type="action"
+                        style="margin-top: 15px;margin-bottom: 10px;"
+                        string="Configure Product"
+                        groups="product_configurator.group_product_configurator"/>
+            </xpath>
+
+            <xpath expr="//field[@name='order_line']/form//field[@name='product_id']" position="after">
+                <field name="config_ok" invisible="1"/>
+            </xpath>
+
+            <xpath expr="//field[@name='order_line']/tree//field[@name='product_id']" position="after">
+                <field name="config_ok" invisible="1"/>
+                <button name="reconfigure_product" groups="product_configurator.group_product_configurator"
+                        help="Reconfigure"
+                        icon="fa-cogs"
+                        type="object"
+                        attrs="{
+                            'invisible': [
+                                '|',
+                                ('config_ok','=',False),
+                                ('state','not in',['draft','sent'])
+                            ]}"/>
+            </xpath>
+        </field>
+    </record>
+
+</odoo>
\ No newline at end of file

From 6dade663a9617506ef45eede9cbaf5c0a8f53dc4 Mon Sep 17 00:00:00 2001
From: Eric Lemire <el@microcom.ca>
Date: Wed, 12 Apr 2017 11:22:14 -0400
Subject: [PATCH 7/8] Fix problem with date_palnned for purchase

---
 product_configurator_purchase/models/product_configurator.py | 4 ++--
 product_configurator_purchase/views/purchase_views.xml       | 3 +++
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/product_configurator_purchase/models/product_configurator.py b/product_configurator_purchase/models/product_configurator.py
index 7fec5940..091479ec 100644
--- a/product_configurator_purchase/models/product_configurator.py
+++ b/product_configurator_purchase/models/product_configurator.py
@@ -34,7 +34,7 @@ def action_config_done(self):
                                                                  new=False))
             if self.purchase_order_line_id:
                 self.purchase_order_line_id.write(self._extra_line_values(self.purchase_order_line_id.order_id,
-                                                                 self.product_id,
-                                                                 new=False))
+                                                                          self.product_id,
+                                                                          new=False))
             self.unlink()
             return
diff --git a/product_configurator_purchase/views/purchase_views.xml b/product_configurator_purchase/views/purchase_views.xml
index 328f7503..9acab767 100644
--- a/product_configurator_purchase/views/purchase_views.xml
+++ b/product_configurator_purchase/views/purchase_views.xml
@@ -43,6 +43,9 @@
                                 ('state','not in',['draft','sent'])
                             ]}"/>
             </xpath>
+            <field name="date_planned" position="attributes">
+                <attribute name="required">0</attribute>
+            </field>
         </field>
     </record>
 

From 5d8199510985ce7db703426901581186706da925 Mon Sep 17 00:00:00 2001
From: Eric Lemire <el@microcom.ca>
Date: Wed, 12 Apr 2017 14:10:16 -0400
Subject: [PATCH 8/8] fix date planned

---
 product_configurator_purchase/views/purchase_views.xml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/product_configurator_purchase/views/purchase_views.xml b/product_configurator_purchase/views/purchase_views.xml
index 9acab767..dc8fe803 100644
--- a/product_configurator_purchase/views/purchase_views.xml
+++ b/product_configurator_purchase/views/purchase_views.xml
@@ -43,9 +43,10 @@
                                 ('state','not in',['draft','sent'])
                             ]}"/>
             </xpath>
-            <field name="date_planned" position="attributes">
+
+            <xpath expr="//label[@for='date_planned']/../div/field[@name='date_planned']" position="attributes">
                 <attribute name="required">0</attribute>
-            </field>
+            </xpath>
         </field>
     </record>