From d6893ae7277ba626c3383fbc3dbf4355b75386fc Mon Sep 17 00:00:00 2001
From: Victor Feyens <vfe@odoo.com>
Date: Tue, 21 Jan 2020 14:06:29 +0000
Subject: [PATCH] [FIX] product,sale: When multi-currency is enabled,
 pricelists should also be enabled

Since https://github.com/odoo/odoo/commit/7d8df4bec192e5a2ff3341dcf43d59e06a57f9a9, the groups
managing pricelists features have been updated.

But the onchange feature enabling pricelist on multi-currency activation was wrongly
adapted, it was only activating advanced pricelists, iff basic pricelists were enabled.

The wanted feature is to activate pricelist (basic pricelist features: group_product_pricelist)
when multi-currency is enabled.

Also add an implied_group on multi_currency group to enforce this behavior.

closes odoo/odoo#49633

X-original-commit: f862f9650395b02f6a43c57cdf232a309fa6948e
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
---
 addons/product/models/res_config_settings.py | 4 ++--
 addons/sale/security/sale_security.xml       | 8 ++++++++
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/addons/product/models/res_config_settings.py b/addons/product/models/res_config_settings.py
index 96bbc88c5f9..e74e05fe781 100644
--- a/addons/product/models/res_config_settings.py
+++ b/addons/product/models/res_config_settings.py
@@ -52,8 +52,8 @@ class ResConfigSettings(models.TransientModel):
 
     @api.onchange('group_multi_currency')
     def _onchange_group_multi_currency(self):
-        if self.group_product_pricelist and not self.group_sale_pricelist and self.group_multi_currency:
-            self.group_sale_pricelist = True
+        if self.group_multi_currency:
+            self.group_product_pricelist = True
 
     @api.onchange('group_product_pricelist')
     def _onchange_group_sale_pricelist(self):
diff --git a/addons/sale/security/sale_security.xml b/addons/sale/security/sale_security.xml
index a4f9ce82f11..103f6d35df0 100644
--- a/addons/sale/security/sale_security.xml
+++ b/addons/sale/security/sale_security.xml
@@ -1,6 +1,14 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
 
+    <record id="base.group_multi_currency" model="res.groups">
+        <!--
+            Sales multi-currency is based on pricelists, if multi-currency is enabled,
+            enable pricelists.
+        -->
+        <field name="implied_ids" eval="[(4, ref('product.group_product_pricelist'))]"/>
+    </record>
+
      <record id="group_auto_done_setting" model="res.groups">
         <field name="name">Lock Confirmed Sales</field>
         <field name="category_id" ref="base.module_category_hidden"/>