From 52ba7626ecbb7cf7da59a22d383b9ff92b36ffa4 Mon Sep 17 00:00:00 2001
From: Laurent Smet <las@odoo.com>
Date: Tue, 24 Aug 2021 14:03:55 +0000
Subject: [PATCH] [FIX] account_edi: Prevent opening the EDI documents in any
 case

Also, remove the duplicated 'edi_document_ids' field inside the view.
'edi_document_ids' is now in debug mode for payments.

closes odoo/odoo#76123

X-original-commit: ad346eec9fd06e430b9b5859578b00e0b28cc443
Signed-off-by: Florian Gilbert <FlorianGilbert@users.noreply.github.com>
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
---
 addons/account_edi/views/account_move_views.xml    | 8 +++++---
 addons/account_edi/views/account_payment_views.xml | 7 +++++--
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/addons/account_edi/views/account_move_views.xml b/addons/account_edi/views/account_move_views.xml
index c2bff6451a2..92ea04867e1 100644
--- a/addons/account_edi/views/account_move_views.xml
+++ b/addons/account_edi/views/account_move_views.xml
@@ -95,12 +95,14 @@
                     </div>
                 </xpath>
                 <xpath expr="//div[@name='journal_div']" position="after">
-                    <field name="edi_document_ids" invisible="1" />
                     <field name="edi_state" attrs="{'invisible': ['|', ('edi_state', '=', False), ('state', '=', 'draft')]}"/>
                 </xpath>
                 <xpath expr="//page[@id='other_tab']" position="after">
-                    <page id="edi_documents" string="EDI Documents" groups="base.group_no_one" attrs="{'invisible': [('edi_document_ids', '=', [])]}">
-                        <field name="edi_document_ids">
+                    <page id="edi_documents"
+                          string="EDI Documents"
+                          groups="base.group_no_one"
+                          attrs="{'invisible': [('edi_document_ids', '=', [])]}">
+                        <field name="edi_document_ids" options="{'no_open': '1'}">
                             <tree create="false" delete="false" edit="false" decoration-danger="error">
                                 <field name="name"/>
                                 <field name="edi_format_name"/>
diff --git a/addons/account_edi/views/account_payment_views.xml b/addons/account_edi/views/account_payment_views.xml
index 91df2183e99..729efc1baa0 100644
--- a/addons/account_edi/views/account_payment_views.xml
+++ b/addons/account_edi/views/account_payment_views.xml
@@ -48,12 +48,15 @@
                     </div>
                 </xpath>
                 <xpath expr="//field[@name='journal_id']" position="after">
-                    <field name="edi_document_ids" invisible="1" />
                     <field name="edi_state" attrs="{'invisible': ['|', ('edi_state', '=', False), ('state', '=', 'draft')]}"/>
                 </xpath>
                 <xpath expr="//group[@name='group2']" position="after">
                     <group groups="base.group_no_one">
-                        <field name="edi_document_ids" string="EDI Documents" attrs="{'invisible': [('edi_document_ids', '=', [])]}">
+                        <field name="edi_document_ids"
+                               string="EDI Documents"
+                               groups="base.group_no_one"
+                               options="{'no_open': '1'}"
+                               attrs="{'invisible': [('edi_document_ids', '=', [])]}">
                             <tree create="false" delete="false" edit="false" decoration-danger="error">
                                 <field name="name"/>
                                 <field name="edi_format_name"/>