From 627c5980c9ab586a6356d4c860d83a44ab6abe6a Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Fri, 1 Jul 2022 08:41:56 +0200
Subject: [PATCH] account_usability: Add constraint on account.journal

reconcile=False by default on suspense account.
---
 account_usability/models/account_journal.py | 5 +++++
 account_usability/views/account_journal.xml | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/account_usability/models/account_journal.py b/account_usability/models/account_journal.py
index 1910faa..bf8f080 100644
--- a/account_usability/models/account_journal.py
+++ b/account_usability/models/account_journal.py
@@ -21,6 +21,11 @@ class AccountJournal(models.Model):
         'account.account.type',
         default=lambda self: self.env.ref('account.data_account_type_current_assets').id)
 
+    # SQL constraint in the 'account' module: unique(code, name, company_id) !!!
+    _sql_constraints = [(
+        'code_unique', 'unique(code, company_id)',
+        'Another journal already has this code in this company!')]
+
     @api.depends(
         'name', 'currency_id', 'company_id', 'company_id.currency_id', 'code')
     def name_get(self):
diff --git a/account_usability/views/account_journal.xml b/account_usability/views/account_journal.xml
index 0f2c5e5..06379c1 100644
--- a/account_usability/views/account_journal.xml
+++ b/account_usability/views/account_journal.xml
@@ -17,7 +17,7 @@
             <field name="account_type_current_assets_id" invisible="1"/>
         </field>
         <field name="suspense_account_id" position="attributes">
-            <attribute name="context">{'default_user_type_id': account_type_current_assets_id, 'default_reconcile': True}</attribute>
+            <attribute name="context">{'default_user_type_id': account_type_current_assets_id, 'default_reconcile': False}</attribute>
         </field>
         <field name="payment_debit_account_id" position="attributes">
             <attribute name="context">{'default_user_type_id': account_type_current_assets_id, 'default_reconcile': True}</attribute>