improved support empty statements and oca camt parser

account_ebics/wizard/__init__.py

@@ -1,2 +1,3 @@
+from . import account_bank_statement_import
 from . import ebics_change_passphrase
 from . import ebics_xfer

account_ebics/wizard/account_bank_statement_import.py

@@ -0,0 +1,62 @@
+# Copyright 2009-2019 Noviat.
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+import logging
+
+from odoo import models, _
+
+_logger = logging.getLogger(__name__)
+
+
+class AccountBankStatementImport(models.TransientModel):
+    _inherit = 'account.bank.statement.import'
+
+    def _check_parsed_data(self, stmts_vals):
+        """ Basic and structural verifications """
+        if self.env.context.get('active_model') == 'ebics.file':
+            message = False
+            if len(stmts_vals) == 0:
+                message = _('This file doesn\'t contain any statement.')
+            if not message:
+                no_st_line = True
+                for vals in stmts_vals:
+                    if vals['transactions'] and len(vals['transactions']) > 0:
+                        no_st_line = False
+                        break
+                if no_st_line:
+                    message = _('This file doesn\'t contain any transaction.')
+                if message:
+                    log_msg = _(
+                        "Error detected while processing and EBICS File"
+                    ) + ':\n' + message
+                    _logger.warn(log_msg)
+                    return
+        super()._check_parsed_data(stmts_vals)
+
+    def _create_bank_statements(self, stmts_vals):
+        """
+        Return error message to ebics.file when handling empty camt.
+
+        Remarks/TODO:
+        We could add more info to the message (e.g. date, balance, ...)
+        and write this to the ebics.file, note field.
+        We could also create empty bank statement (in state done) to clearly
+        show days without transactions via the bank statement list view.
+        """
+        if self.env.context.get('active_model') == 'ebics.file':
+            transactions = False
+            for st_vals in stmts_vals:
+                if st_vals.get('transactions'):
+                    transactions = True
+                    break
+            if not transactions:
+                message = _('This file doesn\'t contain any transaction.')
+                statement_ids = []
+                notifications = {
+                    'type': 'warning',
+                    'message': message,
+                    'details': ''
+                }
+                return statement_ids, [notifications]
+
+        return super()._create_bank_statements(stmts_vals)

account_ebics/wizard/ebics_change_passphrase.py

@@ -1,4 +1,4 @@
-# Copyright 2009-2018 Noviat.
+# Copyright 2009-2019 Noviat.
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 import logging

account_ebics/wizard/ebics_xfer.py

@@ -1,4 +1,4 @@
-# Copyright 2009-2018 Noviat.
+# Copyright 2009-2019 Noviat.
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 """
@@ -199,7 +199,7 @@ class EbicsXfer(models.TransientModel):
                     self.note += _("Warning:")
                     self.note += '\n'
                     self.note += e.message
-                except:
+                except Exception:
                     self.note += '\n'
                     self.note += _("Unknown Error")
                     tb = ''.join(format_exception(*exc_info()))
@@ -321,7 +321,7 @@ class EbicsXfer(models.TransientModel):
                 self.note += _("EBICS Verification Error:")
                 self.note += '\n'
                 self.note += _("The EBICS response could not be verified.")
-            except:
+            except Exception:
                 self.note += '\n'
                 self.note += _("Unknown Error")
                 tb = ''.join(format_exception(*exc_info()))
@@ -359,7 +359,7 @@ class EbicsXfer(models.TransientModel):
         try:
             client = EbicsClient(
                 bank, user, version=self.ebics_config_id.ebics_version)
-        except:
+        except Exception:
             self.note += '\n'
             self.note += _("Unknown Error")
             tb = ''.join(format_exception(*exc_info()))