From 7de04f337662628830461482358744be0672017e Mon Sep 17 00:00:00 2001
From: Carolina Fernandez <carolina.fernandez@tecnativa.com>
Date: Tue, 30 Jul 2024 05:00:59 -0700
Subject: [PATCH] [IMP] account_statement_import_online: Add button in journal
 to see provider configuration

TT49855
---
 .../i18n/account_statement_import_online.pot  |  1 +
 account_statement_import_online/i18n/es.po    |  1 +
 .../models/account_journal.py                 | 10 ++++++
 .../views/account_journal.xml                 | 36 +++++++++----------
 4 files changed, 29 insertions(+), 19 deletions(-)

diff --git a/account_statement_import_online/i18n/account_statement_import_online.pot b/account_statement_import_online/i18n/account_statement_import_online.pot
index 4d51bd7e..1a84bd45 100644
--- a/account_statement_import_online/i18n/account_statement_import_online.pot
+++ b/account_statement_import_online/i18n/account_statement_import_online.pot
@@ -95,6 +95,7 @@ msgstr ""
 
 #. module: account_statement_import_online
 #: model_terms:ir.ui.view,arch_db:account_statement_import_online.online_bank_statement_provider_form
+#: model_terms:ir.ui.view,arch_db:account_statement_import_online.view_account_journal_form
 msgid "Configuration"
 msgstr ""
 
diff --git a/account_statement_import_online/i18n/es.po b/account_statement_import_online/i18n/es.po
index afa51d9d..47917797 100644
--- a/account_statement_import_online/i18n/es.po
+++ b/account_statement_import_online/i18n/es.po
@@ -98,6 +98,7 @@ msgstr "Compañía relacionada a este diario"
 
 #. module: account_statement_import_online
 #: model_terms:ir.ui.view,arch_db:account_statement_import_online.online_bank_statement_provider_form
+#: model_terms:ir.ui.view,arch_db:account_statement_import_online.view_account_journal_form
 msgid "Configuration"
 msgstr "Configuración"
 
diff --git a/account_statement_import_online/models/account_journal.py b/account_statement_import_online/models/account_journal.py
index b3c2647d..ea7ca514 100644
--- a/account_statement_import_online/models/account_journal.py
+++ b/account_statement_import_online/models/account_journal.py
@@ -95,3 +95,13 @@ class AccountJournal(models.Model):
         self.ensure_one()
         provider = self.online_bank_statement_provider_id
         return provider.action_online_bank_statements_pull_wizard()
+
+    def action_open_online_bank_statement_provider(self):
+        return {
+            "type": "ir.actions.act_window",
+            "name": "Statement Provider",
+            "view_mode": "form",
+            "res_model": "online.bank.statement.provider",
+            "res_id": self.online_bank_statement_provider_id.id,
+            "target": "current",
+        }
diff --git a/account_statement_import_online/views/account_journal.xml b/account_statement_import_online/views/account_journal.xml
index a1fdbd56..7de04199 100644
--- a/account_statement_import_online/views/account_journal.xml
+++ b/account_statement_import_online/views/account_journal.xml
@@ -21,6 +21,7 @@
                     groups="account.group_account_user"
                     attrs="{'invisible': [('bank_statements_source', '!=', 'online')]}"
                 >
+
                     <label
                         for="online_bank_statement_provider"
                         string="Provider"
@@ -28,25 +29,22 @@
                         class="oe_edit_only"
                         groups="account.group_account_user"
                     />
-                    <field
-                        name="online_bank_statement_provider"
-                        nolabel="1"
-                        attrs="{'required': [('bank_statements_source', '=', 'online')]}"
-                        class="oe_edit_only"
-                        groups="account.group_account_user"
-                    />
-                    <label
-                        for="online_bank_statement_provider_id"
-                        string="Provider"
-                        attrs="{'invisible': [('online_bank_statement_provider_id', '=', False)]}"
-                        class="oe_read_only"
-                    />
-                    <field
-                        name="online_bank_statement_provider_id"
-                        nolabel="1"
-                        attrs="{'invisible': [('online_bank_statement_provider_id', '=', False)]}"
-                        class="oe_read_only"
-                    />
+                    <div>
+                        <field
+                            name="online_bank_statement_provider"
+                            nolabel="1"
+                            attrs="{'required': [('bank_statements_source', '=', 'online')]}"
+                            class="oe_inline"
+                            groups="account.group_account_user"
+                        />
+                        <field name="online_bank_statement_provider_id" invisible="1" />
+                        <button
+                            name="action_open_online_bank_statement_provider"
+                            type="object"
+                            string="Configuration"
+                            attrs="{'invisible': [('online_bank_statement_provider_id', '=', False)]}"
+                        />
+                    </div>
                 </group>
             </xpath>
             <xpath expr="//div[@name='button_box']" position="inside">