From 9224c424878b09f477a1358b7442b13fa4461fa4 Mon Sep 17 00:00:00 2001
From: aliya <alta@odoo.com>
Date: Mon, 20 Nov 2023 14:56:55 +0000
Subject: [PATCH] [FIX] account_peppol: disable peppol for certain edi formats

If the format set on a partner does not work for our peppol implementation, `enable_peppol` should be `False` and the checkbox should be hidden.
This commit checks invalid partners when computing `enable_peppol` on account_move_send wizard.
Since we hide the checkbox in this case, we don't need to show the peppol warning when opening send&print to such partners.

closes odoo/odoo#143565

Signed-off-by: Laurent Smet (las) <las@odoo.com>
Signed-off-by: Aliya Tastemirova (alta) <alta@odoo.com>
---
 addons/account_peppol/views/res_partner_views.xml | 4 ++--
 addons/account_peppol/wizard/account_move_send.py | 7 +++++--
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/addons/account_peppol/views/res_partner_views.xml b/addons/account_peppol/views/res_partner_views.xml
index e688a40c194..d531e73eed4 100644
--- a/addons/account_peppol/views/res_partner_views.xml
+++ b/addons/account_peppol/views/res_partner_views.xml
@@ -19,9 +19,9 @@
                     <field name="account_peppol_is_endpoint_valid" invisible="1"/>
                     <field name="account_peppol_validity_last_check" invisible="1"/>
                     <label for="account_peppol_verification_label"
-                           invisible="ubl_cii_format in (False, 'facturx') or not peppol_endpoint"/>
+                           invisible="ubl_cii_format in (False, 'facturx', 'oioubl_201') or not peppol_endpoint"/>
                     <div class="row"
-                        invisible="ubl_cii_format in (False, 'facturx') or not peppol_endpoint">
+                        invisible="ubl_cii_format in (False, 'facturx', 'oioubl_201') or not peppol_endpoint">
                         <div class="col-4">
                             <field name="account_peppol_verification_label"/>
                         </div>
diff --git a/addons/account_peppol/wizard/account_move_send.py b/addons/account_peppol/wizard/account_move_send.py
index 0c602369797..6a99a676c35 100644
--- a/addons/account_peppol/wizard/account_move_send.py
+++ b/addons/account_peppol/wizard/account_move_send.py
@@ -55,7 +55,7 @@ class AccountMoveSend(models.TransientModel):
     def _compute_peppol_warning(self):
         for wizard in self:
             invalid_partners = wizard.move_ids.partner_id.filtered(
-                lambda partner: not partner.account_peppol_is_endpoint_valid or partner.ubl_cii_format in {False, 'facturx'})
+                lambda partner: not partner.account_peppol_is_endpoint_valid)
             if not invalid_partners:
                 wizard.peppol_warning = False
             else:
@@ -68,13 +68,16 @@ class AccountMoveSend(models.TransientModel):
     def _compute_enable_peppol(self):
         for wizard in self:
             # show peppol option if either the ubl option is available or any move already has a ubl file generated
-            # and moves are not processing/done
+            # and moves are not processing/done and if partners have an edi format set to one that works for peppol
+            invalid_partners = wizard.move_ids.partner_id.filtered(
+                lambda partner: partner.ubl_cii_format in {False, 'facturx', 'oioubl_201'})
             wizard.enable_peppol = (
                 wizard.company_id.account_peppol_proxy_state == 'active' \
                 and (
                     wizard.enable_ubl_cii_xml
                     or any(m.ubl_cii_xml_id and m.peppol_move_state not in ('processing', 'done') for m in wizard.move_ids)
                 )
+                and not invalid_partners
             )
 
     @api.depends('company_id.account_edi_proxy_client_ids.edi_mode')