From d660fd5204dcb42c45b466a1fbc0ca7f8bf7cbe6 Mon Sep 17 00:00:00 2001
From: "Andrea Grazioso (agr-odoo)" <agr@odoo.com>
Date: Mon, 18 Mar 2024 12:19:44 +0100
Subject: [PATCH] [FIX] l10n_sa_edi: missing reason when reversing entries

With a SA company setup
Open a jounrnal entry
Hit 'Reverse Entry' > Reverse
Error: "For Credit/Debit notes issued in Saudi Arabia, you need to
specify a Reason"

This occurs because with SA localization we need to provide a reason for
move reversal but by default the reason field is invisible for journal entries

opw-3789732

closes odoo/odoo#157996

Signed-off-by: Claire Bretton (clbr) <clbr@odoo.com>
---
 addons/l10n_sa_edi/__manifest__.py               |  1 +
 .../wizard/account_move_reversal_views.xml       | 16 ++++++++++++++++
 2 files changed, 17 insertions(+)
 create mode 100644 addons/l10n_sa_edi/wizard/account_move_reversal_views.xml

diff --git a/addons/l10n_sa_edi/__manifest__.py b/addons/l10n_sa_edi/__manifest__.py
index e21d9c25656..02c5ac1e0cc 100644
--- a/addons/l10n_sa_edi/__manifest__.py
+++ b/addons/l10n_sa_edi/__manifest__.py
@@ -25,6 +25,7 @@ E-invoice implementation for the Kingdom of Saudi Arabia
         'data/ubl_21_zatca.xml',
         'data/res_country_data.xml',
         'wizard/l10n_sa_edi_otp_wizard.xml',
+        'wizard/account_move_reversal_views.xml',
         'views/account_tax_views.xml',
         'views/account_journal_views.xml',
         'views/res_partner_views.xml',
diff --git a/addons/l10n_sa_edi/wizard/account_move_reversal_views.xml b/addons/l10n_sa_edi/wizard/account_move_reversal_views.xml
new file mode 100644
index 00000000000..1f4c4c81867
--- /dev/null
+++ b/addons/l10n_sa_edi/wizard/account_move_reversal_views.xml
@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <data>
+        <record id="view_account_move_reversal_inherit_l10n_sa_edi" model="ir.ui.view">
+            <field name="name">account.move.reversal.form.inherit.l10n_sa_edi</field>
+            <field name="inherit_id" ref="account.view_account_move_reversal"/>
+            <field name="model">account.move.reversal</field>
+            <field name="arch" type="xml">
+                <field name="reason" position="replace">
+                    <field name="country_code" invisible="1"/>
+                    <field name="reason" string="Reason" invisible="move_type == 'entry' and country_code != 'SA'" required="country_code == 'SA'"/>
+                </field>
+            </field>
+        </record>
+    </data>
+</odoo>