diff --git a/account_ebics/README.rst b/account_ebics/README.rst index 335db82..170918d 100644 --- a/account_ebics/README.rst +++ b/account_ebics/README.rst @@ -59,6 +59,14 @@ We also recommend to consider the installation of the following modules: | +- account_bank_statement_import_camt_oca + + Required to handle camt.052 and camt.054 files. + + Cf. https://github.com/OCA/bank_statement_import + +| + - account_bank_statement_import_helper Required if you are processing bank statements with local bank account numbers (e.g. french CFONB files). @@ -87,6 +95,7 @@ The keycode of the licensed version. - fintech_register_users The licensed EBICS user ids. It must be a string or a list of user ids. + You should NOT specify this parameter if your license is subsciption based (with monthly recurring billing). @@ -110,6 +119,33 @@ Go to **Settings > Users** Add the users that are authorised to maintain the EBICS configuration to the 'EBICS Manager' Group. +| + +Go to **Accounting > Configuration > Miscellaneous > EBICS > EBICS File Formats** + +Check if the EBICS File formats that you want to process in Odoo are defined. + +Most commonly used formats for which support is available in Odoo should be there already. + +Please open an issue on https://github.com/Noviat/account_ebics to report missing EBICS File Formats. + +For File Formats of type 'Downloads' you can also specifiy a 'Download Process Method'. + +This is the method that will be executed when hitting the 'Process' button on the downloaded file. + +The following methods are currently available: + +- cfonb120 +- camt.053 +- camt.052 +- camt.054 + +All these methods require complimentary modules to be installed (cf. Installation section supra). + +You'll get an error message when the required module is not installed on your Odoo instance. + +| + Go to **Accounting > Configuration > Miscellaneous > EBICS > EBICS Configuration** Configure your EBICS configuration according to the contract with your bank. diff --git a/account_ebics/__manifest__.py b/account_ebics/__manifest__.py index 004b2cb..2a428e6 100644 --- a/account_ebics/__manifest__.py +++ b/account_ebics/__manifest__.py @@ -3,7 +3,7 @@ { 'name': 'EBICS banking protocol', - 'version': '13.0.1.2.1', + 'version': '13.0.1.3.0', 'license': 'LGPL-3', 'author': 'Noviat', 'website': 'www.noviat.com', diff --git a/account_ebics/data/ebics_file_format.xml b/account_ebics/data/ebics_file_format.xml index a910b5f..da1766f 100644 --- a/account_ebics/data/ebics_file_format.xml +++ b/account_ebics/data/ebics_file_format.xml @@ -2,104 +2,152 @@ - - - camt.053.001.02.stm - down - C53 - Bank Statement in Format camt.053 - c53.xml - + - - - camt.052.001.02.stm + + camt.052 down C52 - Bank Statement in Format camt.052 + camt.052 + bank to customer account report in format camt.052 c52.xml - - - pain.001.001.03.sct - up - CCT - Payment Order in Format pain.001.001.03 - xml + + camt.052 + down + Z52 + camt.052 + bank to customer account report in format camt.052 + c52.xml + + + + camt.053 + down + C53 + camt.053 + Bank to customer statement report in format camt.053 + c53.xml - - pain.008.001.02.sdd - up - CDD - Sepa Core Direct Debit Order in Format pain.008.001.02 - xml + + camt.053 + down + Z53 + camt.053 + Bank to customer statement report in format camt.053 + c53.xml + + + + camt.054 + down + C54 + camt.054 + Bank to customer debit credit notification in format camt.054 + c52.xml - - pain.008.001.02.sbb - up - CDB - Sepa Direct Debit (B2B) Order in Format pain.008.001.02 - xml + + camt.054 + down + Z54 + camt.054 + Bank to customer debit credit notification in format camt.054 + c52.xml - - - pain.001 - up - XE2 - Payment Order in Format pain.001.001.03 - xml - - - - - pain.008 - up - XE3 - Direct Debit Order in Format pain.008.001.02 - xml - - - - + camt.xxx.cfonb120.stm down FDL - Bank Statement in Format cfonb120 + cfonb120 + Bank to customer statement report in format cfonb120 cfonb120.dat - - + + pain.002 + down + CDZ + Payment status report for direct debit in format pain.002 + psr.xml + + + + pain.002 + down + Z01 + pain.002 + Payment status report for direct debit in format pain.002 + psr.xml + + + + + + pain.001.001.03 + up + CCT + Payment Order in format pain.001.001.03 + cct.xml + + + + pain.001.001.03 + up + XE2 + Payment Order in format pain.001.001.03 + cct.xml + + + + pain.008.001.02.sdd + up + CDD + Sepa Core Direct Debit Order in format pain.008.001.02 + sdd.xml + + + + pain.008.001.02.sdd + up + CDD + Sepa Core Direct Debit Order in format pain.008.001.02 + sdd.xml + + + + pain.008.001.02.sdd + up + XE3 + Sepa Core Direct Debit Order in format pain.008.001.02 + sdd.xml + + + + pain.008.001.02.sbb + up + CDB + Sepa Direct Debit (B2B) Order in format pain.008.001.02 + sbb.xml + + + + pain.008.001.02.sbb + up + XE4 + Sepa Direct Debit (B2B) Order in format pain.008.001.02 + sbb.xml + + + pain.001.001.02.sct up FUL - Payment Order in Format pain.001.001.02 - xml + Payment Order in format pain.001.001.02 + cct.xml diff --git a/account_ebics/migrations/13.0.1.3/post-migration.py b/account_ebics/migrations/13.0.1.3/post-migration.py new file mode 100644 index 0000000..a082120 --- /dev/null +++ b/account_ebics/migrations/13.0.1.3/post-migration.py @@ -0,0 +1,41 @@ +# Copyright 2009-2020 Noviat. +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). + +_FILE_FORMATS = [ + {'xml_id_name': 'ebics_ff_C52', + 'download_process_method': 'camt.052', + }, + {'xml_id_name': 'ebics_ff_C53', + 'download_process_method': 'camt.053', + }, + {'xml_id_name': 'ebics_ff_FDL_camt_xxx_cfonb120_stm', + 'download_process_method': 'cfonb120', + }, + +] + + +def migrate(cr, version): + for ff in _FILE_FORMATS: + _update_file_format(cr, ff) + + +def _update_file_format(cr, ff): + cr.execute( + """ + SELECT res_id FROM ir_model_data + WHERE module='account_ebics' AND name='{}' + """.format(ff['xml_id_name']) + ) + res = cr.fetchone() + if res: + cr.execute( + """ + UPDATE ebics_file_format + SET download_process_method='{download_process_method}' + WHERE id={ff_id}; + """.format( + download_process_method=ff['download_process_method'], + ff_id=res[0] + ) + ) diff --git a/account_ebics/migrations/13.0.1.3/pre-migration.py b/account_ebics/migrations/13.0.1.3/pre-migration.py new file mode 100644 index 0000000..7799b26 --- /dev/null +++ b/account_ebics/migrations/13.0.1.3/pre-migration.py @@ -0,0 +1,70 @@ +# Copyright 2009-2020 Noviat. +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). + +_FILE_FORMATS = [ + {'old_xml_id_name': 'ebics_ff_camt_052_001_02_stm', + 'new_xml_id_name': 'ebics_ff_C52', + 'new_name': 'camt.052', + }, + {'old_xml_id_name': 'ebics_ff_camt_053_001_02_stm', + 'new_xml_id_name': 'ebics_ff_C53', + 'new_name': 'camt.053', + }, + {'old_xml_id_name': 'ebics_ff_camt_xxx_cfonb120_stm', + 'new_xml_id_name': 'ebics_ff_FDL_camt_xxx_cfonb120_stm', + }, + {'old_xml_id_name': 'ebics_ff_pain_001_001_03_sct', + 'new_xml_id_name': 'ebics_ff_CCT', + }, + {'old_xml_id_name': 'ebics_ff_pain_001', + 'new_xml_id_name': 'ebics_ff_XE2', + 'new_name': 'pain.001.001.03', + }, + {'old_xml_id_name': 'ebics_ff_pain_008_001_02_sdd', + 'new_xml_id_name': 'ebics_ff_CDD', + }, + {'old_xml_id_name': 'ebics_ff_pain_008', + 'new_xml_id_name': 'ebics_ff_XE3', + }, + {'old_xml_id_name': 'ebics_ff_pain_008_001_02_sbb', + 'new_xml_id_name': 'ebics_ff_CDB', + }, + {'old_xml_id_name': 'ebics_ff_pain_001_001_02_sct', + 'new_xml_id_name': 'ebics_ff_FUL_pain_001_001_02_sct', + }, +] + + +def migrate(cr, version): + if not version: + return + + for ff in _FILE_FORMATS: + _update_file_format(cr, ff) + + +def _update_file_format(cr, ff): + cr.execute( + """ + SELECT id, res_id FROM ir_model_data + WHERE module='account_ebics' AND name='{}' + """.format(ff['old_xml_id_name']) + ) + res = cr.fetchone() + if res: + query = """ + UPDATE ir_model_data + SET name='{new_xml_id_name}' + WHERE id={xml_id}; + """.format( + new_xml_id_name=ff["new_xml_id_name"], xml_id=res[0] + ) + if ff.get('new_name'): + query += """ + UPDATE ebics_file_format + SET name='{new_name}' + WHERE id={ff_id}; + """.format( + new_name=ff["new_name"], ff_id=res[1] + ) + cr.execute(query) diff --git a/account_ebics/models/ebics_file.py b/account_ebics/models/ebics_file.py index 90d7de6..80dabf2 100644 --- a/account_ebics/models/ebics_file.py +++ b/account_ebics/models/ebics_file.py @@ -70,7 +70,7 @@ class EbicsFile(models.Model): raise UserError(_( "You can only remove EBICS files in state 'Draft'.")) # execute format specific actions - ff = ebics_file.format_id.name + ff = ebics_file.format_id.download_process_method if ff in ff_methods: if ff_methods[ff].get('unlink'): ff_methods[ff]['unlink'](ebics_file) @@ -92,7 +92,7 @@ class EbicsFile(models.Model): self = self.with_context(ctx) self.note_process = '' ff_methods = self._file_format_methods() - ff = self.format_id.name + ff = self.format_id.download_process_method if ff in ff_methods: if ff_methods[ff].get('process'): res = ff_methods[ff]['process'](self) @@ -120,15 +120,21 @@ class EbicsFile(models.Model): for extra file formats. """ res = { - 'camt.xxx.cfonb120.stm': + 'cfonb120': {'process': self._process_cfonb120, 'unlink': self._unlink_cfonb120}, - 'camt.052.001.02.stm': + 'camt.052': {'process': self._process_camt052, 'unlink': self._unlink_camt052}, - 'camt.053.001.02.stm': + 'camt.053': {'process': self._process_camt053, 'unlink': self._unlink_camt053}, + 'camt.054': + {'process': self._process_camt054, + 'unlink': self._unlink_camt054}, + 'pain.002': + {'process': self._process_pain002, + 'unlink': self._unlink_pain002}, } return res @@ -294,6 +300,20 @@ class EbicsFile(models.Model): """ pass + @staticmethod + def _process_camt054(self): + import_module = 'account_bank_statement_import_camt_oca' + self._check_import_module(import_module) + return self._process_camt053(self) + + @staticmethod + def _unlink_camt054(self): + """ + Placeholder for camt054 specific actions before removing the + EBICS data file and its related bank statements. + """ + pass + @staticmethod def _process_camt053(self): import_module = 'account_bank_statement_import_camt%' @@ -324,6 +344,23 @@ class EbicsFile(models.Model): """ pass + @staticmethod + def _process_pain002(self): + """ + Placeholder for processing pain.002 files. + TODO: + add import logic based upon OCA 'account_payment_return_import' + """ + pass + + @staticmethod + def _unlink_pain002(self): + """ + Placeholder for pain.002 specific actions before removing the + EBICS data file. + """ + raise NotImplementedError + def _process_undefined_format(self): raise UserError(_( "The current version of the 'account_ebics' module " diff --git a/account_ebics/models/ebics_file_format.py b/account_ebics/models/ebics_file_format.py index 55a9945..5864644 100644 --- a/account_ebics/models/ebics_file_format.py +++ b/account_ebics/models/ebics_file_format.py @@ -7,21 +7,35 @@ from odoo import api, fields, models class EbicsFileFormat(models.Model): _name = 'ebics.file.format' _description = 'EBICS File Formats' - _order = 'type,name' + _order = 'type,name,order_type' - name = fields.Selection( - selection=lambda self: self._selection_name(), - string='Request Type', required=True) + name = fields.Char( + string='Request Type', + required=True, + help="E.g. camt.xxx.cfonb120.stm, pain.001.001.03.sct.\n" + "Specify camt.052, camt.053, camt.054 for camt " + "Order Types such as C53, Z53, C54, Z54.\n" + "This name has to match the 'Request Type' in your " + "EBICS contract for Order Type 'FDL' or 'FUL'.\n") type = fields.Selection( selection=[('down', 'Download'), ('up', 'Upload')], required=True) - order_type = fields.Selection( - selection=lambda self: self._selection_order_type(), + order_type = fields.Char( string='Order Type', - help="For most banks is France you should use the " + required=True, + help="E.g. C53 (check your EBICS contract).\n" + "For most banks in France you should use the " "format neutral Order Types 'FUL' for upload " "and 'FDL' for download.") + download_process_method = fields.Selection( + selection='_selection_download_process_method', + help="Enable processing within Odoo of the downloaded file " + "via the 'Process' button." + "E.g. specify camt.053 to import a camt.053 file and create " + "a bank statement.") + # TODO: + # move signature_class parameter so that it can be set per EBICS config signature_class = fields.Selection( selection=[('E', 'Single signature'), ('T', 'Transport signature')], @@ -30,51 +44,19 @@ class EbicsFileFormat(models.Model): "ERP system when using class 'E' to prevent unauthorised " "users to make supplier payments." "\nLeave this field empty to use the default " - "defined for your bank connection.") + "defined for your EBICS UserID.") description = fields.Char() suffix = fields.Char( required=True, help="Specify the filename suffix for this File Format." - "\nE.g. camt.053.xml") + "\nE.g. c53.xml") @api.model - def _selection_order_type(self): - up = self._supported_upload_order_types() - down = self._supported_download_order_types() - selection = [(x, x) for x in up + down] - return selection + def _selection_download_process_method(self): + methods = self.env['ebics.file']._file_format_methods().keys() + return [(x, x) for x in methods] - def _supported_upload_order_types(self): - return ['FUL', 'CCT', 'CDD', 'CDB', 'XE2', 'XE3'] - - def _supported_download_order_types(self): - return ['FDL', 'C52', 'C53', 'C54'] - - @api.model - def _selection_name(self): - """ - List of supported EBICS Request Types. - Extend this method via a custom module when testing - a new Request Type and make a PR for the - account_ebics module when this new Request Type - is working correctly. - This PR should include at least updates to - - 'data/ebics_file_format.xml' - - 'models/ebics_file_format.py' - An overview of the EBICS Request Types can be found in - the doc folder of this module (EBICS_Annex2). - """ - request_types = [ - 'camt.052.001.02.stm', - 'camt.053.001.02.stm', - 'pain.001.001.03.sct', - 'pain.008.001.02.sdd', - 'pain.008.001.02.sbb', - 'camt.xxx.cfonb120.stm', - 'pain.001.001.02.sct', - 'camt.053', - 'pain.001', - 'pain.008', - ] - selection = [(x, x) for x in request_types] - return selection + @api.onchange('type') + def _onchange_type(self): + if self.type == 'up': + self.download_process_method = False diff --git a/account_ebics/models/fintech_ebics_register.py b/account_ebics/models/fintech_ebics_register.py index 5a97aaa..f908637 100644 --- a/account_ebics/models/fintech_ebics_register.py +++ b/account_ebics/models/fintech_ebics_register.py @@ -32,10 +32,10 @@ try: keycode=fintech_register_keycode, users=fintech_register_users) except RuntimeError as e: - if e.message == "'register' can be called only once": + if str(e) == "'register' can be called only once": pass else: - _logger.error(e.message) + _logger.error(str(e)) fintech.register() except Exception: msg = "fintech.register error" diff --git a/account_ebics/static/description/index.html b/account_ebics/static/description/index.html index ab15e10..42b8115 100644 --- a/account_ebics/static/description/index.html +++ b/account_ebics/static/description/index.html @@ -399,6 +399,15 @@ ul.auto-toc {

+
+

+
+