From 6c55c7b5860945872fc5eeba8b1afc7424499313 Mon Sep 17 00:00:00 2001
From: Moises Lopez <moylop260@vauxoo.com>
Date: Fri, 13 May 2022 17:14:30 +0000
Subject: [PATCH] [FIX] account_move_name_sequence: Fix required journal fields
 in view

The required flag was wrong for sequence_id and refund_sequence_id

So, it was not possible to store any change in journal for journal different to sale and purchase
---
 account_move_name_sequence/__manifest__.py           | 2 +-
 account_move_name_sequence/views/account_journal.xml | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/account_move_name_sequence/__manifest__.py b/account_move_name_sequence/__manifest__.py
index 3a836fbd6..a4960cd83 100644
--- a/account_move_name_sequence/__manifest__.py
+++ b/account_move_name_sequence/__manifest__.py
@@ -6,7 +6,7 @@
 
 {
     "name": "Account Move Number Sequence",
-    "version": "14.0.1.1.0",
+    "version": "14.0.1.2.0",
     "category": "Accounting",
     "license": "AGPL-3",
     "summary": "Generate journal entry number from sequence",
diff --git a/account_move_name_sequence/views/account_journal.xml b/account_move_name_sequence/views/account_journal.xml
index 6cc643a73..d274975a6 100644
--- a/account_move_name_sequence/views/account_journal.xml
+++ b/account_move_name_sequence/views/account_journal.xml
@@ -13,15 +13,15 @@
             <field name="refund_sequence" position="before">
                 <field
                     name="sequence_id"
-                    required="0"
+                    required="1"
                     context="{'default_name': name, 'default_company_id': company_id, 'default_implementation': 'no_gap', 'default_padding': 4, 'default_use_date_range': True, 'default_prefix': (code or 'UNKNOWN') + '/%%(range_year)s/'}"
                 />
             </field>
             <field name="refund_sequence" position="after">
                 <field
                     name="refund_sequence_id"
-                    attrs="{'invisible': ['|', ('type', 'not in', ('sale', 'purchase')), ('refund_sequence', '=', False)]}"
-                    required="1"
+                    attrs="{'invisible': ['|', ('type', 'not in', ('sale', 'purchase')), ('refund_sequence', '=', False)],
+                            'required': [('type', 'in', ('sale', 'purchase')), ('refund_sequence', '=', True)]}"
                     context="{'default_name': name, 'default_company_id': company_id, 'default_implementation': 'no_gap', 'default_padding': 4, 'default_use_date_range': True, 'default_prefix': 'R' + (code or 'UNKNOWN') + '/%%(range_year)s/'}"
                 />
             </field>