[IMP] account_usability_misc: do not hide bank statement lines creation buttons

account_advanced_protection_features/models/account_bank_statement.py

@@ -19,21 +19,19 @@ class AccountBankStatement(models.Model):
 class AccountBankStatementLine(models.Model):
     _inherit = "account.bank.statement.line"
 
-    @api.model_create_multi
-    def create(self, vals_list):
-        # Condition on action_name is here to check permissions only for the manual creation.
-        # Import of files and pulling online bank statements should not be concerned
-        if self.env.context.get("action_name") == "action_bank_statement_tree":
-            for vals in vals_list:
-                journal_id = vals.get("journal_id")
-                if journal_id:
-                    journal = self.env["account.journal"].browse(journal_id)
-                    if not journal.allow_bank_statement_line_creation:
-                        raise UserError(
-                            _("Manual creation of bank statement lines is not allowed for the journal %s.")
-                            % journal.display_name
-                        )
-        return super().create(vals_list)
+    # This function is called when a user opens the form to create a new bank statement line
+    @api.model
+    def default_get(self, fields_list):
+        res = super().default_get(fields_list)
+        journal_id = self._context.get('default_journal_id') or res.get('journal_id')
+        if journal_id:
+            journal = self.env["account.journal"].browse(journal_id)
+            if not journal.allow_bank_statement_line_creation:
+                raise UserError(
+                    _("Manual creation of bank statement lines is not allowed for the journal %s.")
+                    % journal.display_name
+                )
+        return res
 
     def unlink(self):
         for line in self:

account_advanced_protection_features/tests/test_account_bank_statement.py

@@ -14,45 +14,22 @@ class TestBankStatementLineCreation(TransactionCase):
         )
         cls.partner = cls.env["res.partner"].create({"name": "Test Partner"})
 
-    def test_manual_create_blocks_when_not_allowed(self):
-        """Test that manual creation from UI raises UserError when not allowed."""
+    def test_default_get_blocks_creation_when_not_allowed(self):
+        """Test that default_get raises UserError when creation is not allowed."""
         self.bank_journal.allow_bank_statement_line_creation = False
 
         with self.assertRaises(UserError):
             self.env["account.bank.statement.line"].with_context(
-                action_name="action_bank_statement_tree"
-            ).create({
-                "journal_id": self.bank_journal.id,
-                "amount": 100.0,
-                "payment_ref": "Test",
-                "date": "2024-01-01",
-            })
+                default_journal_id=self.bank_journal.id
+            ).default_get(["journal_id", "amount", "payment_ref"])
 
-    def test_manual_create_works_when_allowed(self):
-        """Test that manual creation from UI works when allowed."""
+    def test_default_get_allows_creation_when_allowed(self):
+        """Test that default_get works when creation is allowed."""
         self.bank_journal.allow_bank_statement_line_creation = True
 
-        line = self.env["account.bank.statement.line"].with_context(
-            action_name="action_bank_statement_tree"
-        ).create({
-            "journal_id": self.bank_journal.id,
-            "amount": 100.0,
-            "payment_ref": "Test",
-            "date": "2024-01-01",
-        })
+        # Should not raise
+        result = self.env["account.bank.statement.line"].with_context(
+            default_journal_id=self.bank_journal.id
+        ).default_get(["journal_id", "amount", "payment_ref"])
 
-        self.assertTrue(line.exists())
-
-    def test_programmatic_create_works_even_when_not_allowed(self):
-        """Test that creation without UI context (e.g. OFX import, pulling online bank statements)
-        works even when manual creation is not allowed."""
-        self.bank_journal.allow_bank_statement_line_creation = False
-
-        line = self.env["account.bank.statement.line"].create({
-            "journal_id": self.bank_journal.id,
-            "amount": 100.0,
-            "payment_ref": "Test Import",
-            "date": "2024-01-01",
-        })
-
-        self.assertTrue(line.exists())
+        self.assertIsInstance(result, dict)