[ADD] account_bank_statement_import_online_adyen

account_bank_statement_import_adyen/models/account_bank_statement_import.py

@@ -16,8 +16,18 @@ class AccountBankStatementImport(models.TransientModel):
     def _parse_file(self, data_file):
         """Parse an Adyen xlsx file and map merchant account strings to journals."""
         try:
-            return self.import_adyen_xlsx(data_file)
+            try:
+                return self._parse_adyen_file(data_file)
+            except Exception as exc:
+                if self.env.context.get("account_bank_statement_import_adyen", False):
+                    raise
+                _logger.info("Adyen parser error", exc_info=True)
+                raise ValueError("Not an adyen settlements file: %s" % exc)
         except ValueError:
+            _logger.debug(
+                _("Statement file was not a Adyen settlement details file."),
+                exc_info=True,
+            )
             return super()._parse_file(data_file)
 
     def _find_additional_data(self, currency_code, account_number):
@@ -40,25 +50,25 @@ class AccountBankStatementImport(models.TransientModel):
         return super()._find_additional_data(currency_code, account_number)
 
     @api.model
-    def balance(self, row):
+    def _balance(self, row):
         return -(float(row[15]) if row[15] else 0.0) + sum(
             float(row[i]) if row[i] else 0.0 for i in (16, 17, 18, 19, 20)
         )
 
     @api.model
-    def import_adyen_transaction(self, statement, statement_id, row):
+    def _import_adyen_transaction(self, statement, statement_id, row):
         transaction_id = str(len(statement["transactions"])).zfill(4)
         transaction = dict(
             unique_import_id=statement_id + transaction_id,
             date=fields.Date.from_string(row[6]),
-            amount=self.balance(row),
+            amount=self._balance(row),
             note="{} {} {} {}".format(row[2], row[3], row[4], row[21]),
             name="%s" % (row[3] or row[4] or row[9]),
         )
         statement["transactions"].append(transaction)
 
     @api.model
-    def parse_adyen_file(self, data_file):
+    def _parse_adyen_file(self, data_file):
         statements = []
         statement = None
         headers = False
@@ -70,6 +80,9 @@ class AccountBankStatementImport(models.TransientModel):
         import_model = self.env["base_import.import"]
         importer = import_model.create(
             {"file": data_file, "file_name": "Ayden settlemnt details"}
+        import_model = self.env["base_import.import"]
+        importer = import_model.create(
+            {"file": data_file, "file_name": "Ayden settlement details"}
         )
         rows = importer._read_file({})
 
@@ -105,10 +118,10 @@ class AccountBankStatementImport(models.TransientModel):
 
             row[8] = row[8].strip()
             if row[8] == "MerchantPayout":
-                payout -= self.balance(row)
+                payout -= self._balance(row)
             else:
-                balance += self.balance(row)
-            self.import_adyen_transaction(statement, statement_id, row)
+                balance += self._balance(row)
+            self._import_adyen_transaction(statement, statement_id, row)
             fees += sum(float(row[i]) if row[i] else 0.0 for i in (17, 18, 19, 20))
 
         if not headers:

account_bank_statement_import_adyen/tests/test_import_adyen.py

@@ -23,9 +23,6 @@ class TestImportAdyen(SavepointCase):
                 "currency_id": cls.env.ref("base.USD").id,
             }
         )
-        # Enable reconcilation on the default journal account to trigger
-        # the functionality from account_bank_statement_clearing_account
-        cls.journal.default_debit_account_id.reconcile = True
 
     def test_01_import_adyen(self):
         """ Test that the Adyen statement can be imported and that the
@@ -68,7 +65,9 @@ class TestImportAdyen(SavepointCase):
             import_wizard = self.env["account.bank.statement.import"].create(
                 {"attachment_ids": [(0, 0, {"name": "test file", "datas": data_file})]}
             )
-            import_wizard.import_file()
+            import_wizard.with_context(
+                {"account_bank_statement_import_adyen": True}
+            ).import_file()
             # statement name is account number + '-' + date of last line:
             statements = self.env["account.bank.statement"].search(
                 [("name", "=", statement_name)]

account_bank_statement_import_online_adyen/README.rst

@@ -0,0 +1,118 @@
+============================================
+Online Bank Statements: Adyen payment report
+============================================
+
+.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
+    :target: https://odoo-community.org/page/development-status
+    :alt: Beta
+.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :alt: License: AGPL-3
+.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fbank--statement--import-lightgray.png?logo=github
+    :target: https://github.com/OCA/bank-statement-import/tree/13.0/account_bank_statement_import_online_adyen
+    :alt: OCA/bank-statement-import
+.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
+    :target: https://translation.odoo-community.org/projects/bank-statement-import-13-0/bank-statement-import-13-0-account_bank_statement_import_online_adyen
+    :alt: Translate me on Weblate
+.. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png
+    :target: https://runbot.odoo-community.org/runbot/174/13.0
+    :alt: Try me on Runbot
+
+|badge1| |badge2| |badge3| |badge4| |badge5| 
+
+This module automates the download and import of Adyen payment reports.
+
+**Table of contents**
+
+.. contents::
+   :local:
+
+Configuration
+=============
+
+To configure online bank statements provider:
+
+#. Go to *Invoicing > Configuration > Bank Accounts*
+#. Open bank account to configure and edit it
+#. Set *Bank Feeds* to *Online*
+#. Select *Adyen* as online bank statements provider in
+   *Online Bank Statements (OCA)* section
+#. Save the bank account
+#. Click on provider and configure provider-specific settings.
+
+or, alternatively:
+
+#. Go to *Invoicing > Overview*
+#. Open settings of the corresponding journal account
+#. Switch to *Bank Account* tab
+#. Set *Bank Feeds* to *Online*
+#. Select *Adyen* as online bank statements provider in
+   *Online Bank Statements (OCA)* section
+#. Save the bank account
+#. Click on provider and configure provider-specific settings.
+
+To obtain *Login* and *Key*:
+
+#. Open `Adyen website <https://www.adyen.com/signup/>`_.
+
+Check also ``account_bank_statement_import_online`` configuration instructions
+for more information.
+
+Usage
+=====
+
+To pull historical bank statements:
+
+#. Go to *Invoicing > Configuration > Bank Accounts*
+#. Select specific bank accounts
+#. Launch *Actions > Online Bank Statements Pull Wizard*
+#. Configure date interval and click *Pull*
+
+If historical data is not needed, then just simply wait for the scheduled
+activity "Pull Online Bank Statements" to be executed for getting new
+transactions.
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues <https://github.com/OCA/bank-statement-import/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us smashing it by providing a detailed and welcomed
+`feedback <https://github.com/OCA/bank-statement-import/issues/new?body=module:%20account_bank_statement_import_online_adyen%0Aversion:%2013.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+
+Do not contact contributors directly about support or help with technical issues.
+
+Credits
+=======
+
+Authors
+~~~~~~~
+
+* Ronald Portier (Therp BV)
+
+Contributors
+~~~~~~~~~~~~
+
+* Ronald Portier - Therp BV
+
+Maintainers
+~~~~~~~~~~~
+
+This module is maintained by the OCA.
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
+
+This module is part of the `OCA/bank-statement-import <https://github.com/OCA/bank-statement-import/tree/13.0/account_bank_statement_import_online_adyen>`_ project on GitHub.
+
+You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

account_bank_statement_import_online_adyen/__init__.py

@@ -0,0 +1,3 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+from . import models
+from .tests import online_bank_statement_provider_dummy

account_bank_statement_import_online_adyen/__manifest__.py

@@ -0,0 +1,16 @@
+# Copyright 2021 - Therp BV <https://therp.nl>.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+{
+    "name": "Online Bank Statements: Adyen payment report",
+    "version": "13.0.1.0.0",
+    "category": "Account",
+    "website": "https://github.com/OCA/bank-statement-import",
+    "author": "Ronald Portier (Therp BV), Odoo Community Association (OCA)",
+    "license": "AGPL-3",
+    "installable": True,
+    "depends": [
+        "account_bank_statement_import_adyen",
+        "account_bank_statement_import_online",
+    ],
+    "data": ["views/online_bank_statement_provider.xml"],
+}

account_bank_statement_import_online_adyen/models/__init__.py

@@ -0,0 +1,4 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from . import account_journal
+from . import online_bank_statement_provider

account_bank_statement_import_online_adyen/models/account_journal.py

@@ -0,0 +1,30 @@
+# Copyright 2021 Therp BV <https://therp.nl>.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+from odoo import api, models
+
+
+class AccountJournal(models.Model):
+    _inherit = "account.journal"
+
+    def write(self, vals):
+        """Do not reset a provider to file_import, if that will delete provider."""
+        # TODO: In the future place this in super account_bank_statement_import_online.
+        for this in self:
+            is_online = this.bank_statements_source == "online"
+            if is_online and vals.get("bank_statements_source", "online") != "online":
+                vals.pop("bank_statements_source")
+            super(AccountJournal, this).write(vals)
+        return True
+
+    @api.model
+    def _selection_online_bank_statement_provider(self):
+        res = super()._selection_online_bank_statement_provider()
+        res.append(("dummy_adyen", "Dummy Adyen"))
+        return res
+
+    @api.model
+    def values_online_bank_statement_provider(self):
+        res = super().values_online_bank_statement_provider()
+        if self.user_has_groups("base.group_no_one"):
+            res += [("dummy_adyen", "Dummy Adyen")]
+        return res

account_bank_statement_import_online_adyen/models/online_bank_statement_provider.py

@@ -0,0 +1,100 @@
+# Copyright 2021 Therp BV <https://therp.nl>.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+import logging
+from html import escape
+
+import requests
+
+from odoo import _, api, fields, models
+from odoo.exceptions import UserError
+
+_logger = logging.getLogger(__name__)
+
+
+class OnlineBankStatementProvider(models.Model):
+    _inherit = "online.bank.statement.provider"
+
+    download_file_name = fields.Char()
+    next_batch_number = fields.Integer()
+
+    @api.model
+    def _selection_service(self):
+        res = super()._selection_service()
+        res.append(("dummy_adyen", "Dummy Adyen"))
+        return res
+
+    @api.model
+    def _get_available_services(self):
+        return super()._get_available_services() + [("adyen", "Adyen payment report")]
+
+    def _pull(self, date_since, date_until):  # noqa: C901
+        """Split between adyen providers and others."""
+        adyen_providers = self.filtered(lambda r: r.service in ("adyen", "dummy_adyen"))
+        other_providers = self.filtered(
+            lambda r: r.service not in ("adyen", "dummy_adyen")
+        )
+        if other_providers:
+            super(OnlineBankStatementProvider, other_providers)._pull(
+                date_since, date_until
+            )
+        for provider in adyen_providers:
+            # TODO: incrementing batch number
+            is_scheduled = self.env.context.get("scheduled")
+            try:
+                data_file = self._adyen_get_settlement_details_file()
+                import_wizard = self.env["account.bank.statement.import"].create(
+                    {
+                        "attachment_ids": [
+                            (0, 0, {"name": "test file", "datas": data_file})
+                        ]
+                    }
+                )
+                import_wizard.with_context(
+                    {"account_bank_statement_import_adyen": True}
+                ).import_file()
+            except BaseException as e:
+                if is_scheduled:
+                    _logger.warning(
+                        'Online Bank Statement Provider "%s" failed to'
+                        " obtain statement data" % (provider.name,),
+                        exc_info=True,
+                    )
+                    provider.message_post(
+                        body=_(
+                            "Failed to obtain statement data for period "
+                            ": %s. See server logs for more details."
+                        )
+                        % (escape(str(e)) or _("N/A"),),
+                        subject=_("Issue with Online Bank Statement Provider"),
+                    )
+                    break
+                raise
+            if is_scheduled:
+                provider._schedule_next_run()
+
+    def _adyen_get_settlement_details_file(self):
+        """Retrieve daily generated settlement details file.
+
+        The file could be retrieved with wget using:
+        $ wget \
+            --http-user='[YourReportUser]@Company.[YourCompanyAccount]' \
+            --http-password='[YourReportUserPassword]' \
+            --quiet --no-check-certificate \
+            https://ca-test.adyen.com/reports/download/MerchantAccount/ +
+                [YourMerchantAccount]/[ReportFileName]"
+        """
+        batch_number = self.next_batch_number
+        download_file_name = self.download_file_name % batch_number
+        URL = "/".join(
+            [self.api_base, self.journal_id.adyen_merchant_account, download_file_name]
+        )
+        response = requests.get(URL, auth=(self.username, self.password))
+        if response.status_code == 200:
+            return response.content
+        else:
+            raise UserError(_("%s \n\n %s") % (response.status_code, response.text))
+
+    def _schedule_next_run(self):
+        """Set next run date and autoincrement batch number."""
+        super()._schedule_next_run()
+        self.next_batch_number += 1

account_bank_statement_import_online_adyen/readme/CONFIGURE.rst

@@ -0,0 +1,27 @@
+To configure online bank statements provider:
+
+#. Go to *Invoicing > Configuration > Bank Accounts*
+#. Open bank account to configure and edit it
+#. Set *Bank Feeds* to *Online*
+#. Select *Adyen* as online bank statements provider in
+   *Online Bank Statements (OCA)* section
+#. Save the bank account
+#. Click on provider and configure provider-specific settings.
+
+or, alternatively:
+
+#. Go to *Invoicing > Overview*
+#. Open settings of the corresponding journal account
+#. Switch to *Bank Account* tab
+#. Set *Bank Feeds* to *Online*
+#. Select *Adyen* as online bank statements provider in
+   *Online Bank Statements (OCA)* section
+#. Save the bank account
+#. Click on provider and configure provider-specific settings.
+
+To obtain *Login* and *Key*:
+
+#. Open `Adyen website <https://www.adyen.com/signup/>`_.
+
+Check also ``account_bank_statement_import_online`` configuration instructions
+for more information.

account_bank_statement_import_online_adyen/readme/CONTRIBUTORS.rst

@@ -0,0 +1 @@
+* Ronald Portier - Therp BV

account_bank_statement_import_online_adyen/readme/DESCRIPTION.rst

@@ -0,0 +1 @@
+This module automates the download and import of Adyen payment reports.

account_bank_statement_import_online_adyen/readme/USAGE.rst

@@ -0,0 +1,10 @@
+To pull historical bank statements:
+
+#. Go to *Invoicing > Configuration > Bank Accounts*
+#. Select specific bank accounts
+#. Launch *Actions > Online Bank Statements Pull Wizard*
+#. Configure date interval and click *Pull*
+
+If historical data is not needed, then just simply wait for the scheduled
+activity "Pull Online Bank Statements" to be executed for getting new
+transactions.

account_bank_statement_import_online_adyen/static/description/index.html

@@ -0,0 +1,464 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<head>
+<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+<meta name="generator" content="Docutils 0.15.1: http://docutils.sourceforge.net/" />
+<title>Online Bank Statements: Adyen payment report</title>
+<style type="text/css">
+
+/*
+:Author: David Goodger (goodger@python.org)
+:Id: $Id: html4css1.css 7952 2016-07-26 18:15:59Z milde $
+:Copyright: This stylesheet has been placed in the public domain.
+
+Default cascading style sheet for the HTML output of Docutils.
+
+See http://docutils.sf.net/docs/howto/html-stylesheets.html for how to
+customize this style sheet.
+*/
+
+/* used to remove borders from tables and images */
+.borderless, table.borderless td, table.borderless th {
+  border: 0 }
+
+table.borderless td, table.borderless th {
+  /* Override padding for "table.docutils td" with "! important".
+     The right padding separates the table cells. */
+  padding: 0 0.5em 0 0 ! important }
+
+.first {
+  /* Override more specific margin styles with "! important". */
+  margin-top: 0 ! important }
+
+.last, .with-subtitle {
+  margin-bottom: 0 ! important }
+
+.hidden {
+  display: none }
+
+.subscript {
+  vertical-align: sub;
+  font-size: smaller }
+
+.superscript {
+  vertical-align: super;
+  font-size: smaller }
+
+a.toc-backref {
+  text-decoration: none ;
+  color: black }
+
+blockquote.epigraph {
+  margin: 2em 5em ; }
+
+dl.docutils dd {
+  margin-bottom: 0.5em }
+
+object[type="image/svg+xml"], object[type="application/x-shockwave-flash"] {
+  overflow: hidden;
+}
+
+/* Uncomment (and remove this text!) to get bold-faced definition list terms
+dl.docutils dt {
+  font-weight: bold }
+*/
+
+div.abstract {
+  margin: 2em 5em }
+
+div.abstract p.topic-title {
+  font-weight: bold ;
+  text-align: center }
+
+div.admonition, div.attention, div.caution, div.danger, div.error,
+div.hint, div.important, div.note, div.tip, div.warning {
+  margin: 2em ;
+  border: medium outset ;
+  padding: 1em }
+
+div.admonition p.admonition-title, div.hint p.admonition-title,
+div.important p.admonition-title, div.note p.admonition-title,
+div.tip p.admonition-title {
+  font-weight: bold ;
+  font-family: sans-serif }
+
+div.attention p.admonition-title, div.caution p.admonition-title,
+div.danger p.admonition-title, div.error p.admonition-title,
+div.warning p.admonition-title, .code .error {
+  color: red ;
+  font-weight: bold ;
+  font-family: sans-serif }
+
+/* Uncomment (and remove this text!) to get reduced vertical space in
+   compound paragraphs.
+div.compound .compound-first, div.compound .compound-middle {
+  margin-bottom: 0.5em }
+
+div.compound .compound-last, div.compound .compound-middle {
+  margin-top: 0.5em }
+*/
+
+div.dedication {
+  margin: 2em 5em ;
+  text-align: center ;
+  font-style: italic }
+
+div.dedication p.topic-title {
+  font-weight: bold ;
+  font-style: normal }
+
+div.figure {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+div.footer, div.header {
+  clear: both;
+  font-size: smaller }
+
+div.line-block {
+  display: block ;
+  margin-top: 1em ;
+  margin-bottom: 1em }
+
+div.line-block div.line-block {
+  margin-top: 0 ;
+  margin-bottom: 0 ;
+  margin-left: 1.5em }
+
+div.sidebar {
+  margin: 0 0 0.5em 1em ;
+  border: medium outset ;
+  padding: 1em ;
+  background-color: #ffffee ;
+  width: 40% ;
+  float: right ;
+  clear: right }
+
+div.sidebar p.rubric {
+  font-family: sans-serif ;
+  font-size: medium }
+
+div.system-messages {
+  margin: 5em }
+
+div.system-messages h1 {
+  color: red }
+
+div.system-message {
+  border: medium outset ;
+  padding: 1em }
+
+div.system-message p.system-message-title {
+  color: red ;
+  font-weight: bold }
+
+div.topic {
+  margin: 2em }
+
+h1.section-subtitle, h2.section-subtitle, h3.section-subtitle,
+h4.section-subtitle, h5.section-subtitle, h6.section-subtitle {
+  margin-top: 0.4em }
+
+h1.title {
+  text-align: center }
+
+h2.subtitle {
+  text-align: center }
+
+hr.docutils {
+  width: 75% }
+
+img.align-left, .figure.align-left, object.align-left, table.align-left {
+  clear: left ;
+  float: left ;
+  margin-right: 1em }
+
+img.align-right, .figure.align-right, object.align-right, table.align-right {
+  clear: right ;
+  float: right ;
+  margin-left: 1em }
+
+img.align-center, .figure.align-center, object.align-center {
+  display: block;
+  margin-left: auto;
+  margin-right: auto;
+}
+
+table.align-center {
+  margin-left: auto;
+  margin-right: auto;
+}
+
+.align-left {
+  text-align: left }
+
+.align-center {
+  clear: both ;
+  text-align: center }
+
+.align-right {
+  text-align: right }
+
+/* reset inner alignment in figures */
+div.align-right {
+  text-align: inherit }
+
+/* div.align-center * { */
+/*   text-align: left } */
+
+.align-top    {
+  vertical-align: top }
+
+.align-middle {
+  vertical-align: middle }
+
+.align-bottom {
+  vertical-align: bottom }
+
+ol.simple, ul.simple {
+  margin-bottom: 1em }
+
+ol.arabic {
+  list-style: decimal }
+
+ol.loweralpha {
+  list-style: lower-alpha }
+
+ol.upperalpha {
+  list-style: upper-alpha }
+
+ol.lowerroman {
+  list-style: lower-roman }
+
+ol.upperroman {
+  list-style: upper-roman }
+
+p.attribution {
+  text-align: right ;
+  margin-left: 50% }
+
+p.caption {
+  font-style: italic }
+
+p.credits {
+  font-style: italic ;
+  font-size: smaller }
+
+p.label {
+  white-space: nowrap }
+
+p.rubric {
+  font-weight: bold ;
+  font-size: larger ;
+  color: maroon ;
+  text-align: center }
+
+p.sidebar-title {
+  font-family: sans-serif ;
+  font-weight: bold ;
+  font-size: larger }
+
+p.sidebar-subtitle {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+p.topic-title {
+  font-weight: bold }
+
+pre.address {
+  margin-bottom: 0 ;
+  margin-top: 0 ;
+  font: inherit }
+
+pre.literal-block, pre.doctest-block, pre.math, pre.code {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+pre.code .ln { color: grey; } /* line numbers */
+pre.code, code { background-color: #eeeeee }
+pre.code .comment, code .comment { color: #5C6576 }
+pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
+pre.code .literal.string, code .literal.string { color: #0C5404 }
+pre.code .name.builtin, code .name.builtin { color: #352B84 }
+pre.code .deleted, code .deleted { background-color: #DEB0A1}
+pre.code .inserted, code .inserted { background-color: #A3D289}
+
+span.classifier {
+  font-family: sans-serif ;
+  font-style: oblique }
+
+span.classifier-delimiter {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+span.interpreted {
+  font-family: sans-serif }
+
+span.option {
+  white-space: nowrap }
+
+span.pre {
+  white-space: pre }
+
+span.problematic {
+  color: red }
+
+span.section-subtitle {
+  /* font-size relative to parent (h1..h6 element) */
+  font-size: 80% }
+
+table.citation {
+  border-left: solid 1px gray;
+  margin-left: 1px }
+
+table.docinfo {
+  margin: 2em 4em }
+
+table.docutils {
+  margin-top: 0.5em ;
+  margin-bottom: 0.5em }
+
+table.footnote {
+  border-left: solid 1px black;
+  margin-left: 1px }
+
+table.docutils td, table.docutils th,
+table.docinfo td, table.docinfo th {
+  padding-left: 0.5em ;
+  padding-right: 0.5em ;
+  vertical-align: top }
+
+table.docutils th.field-name, table.docinfo th.docinfo-name {
+  font-weight: bold ;
+  text-align: left ;
+  white-space: nowrap ;
+  padding-left: 0 }
+
+/* "booktabs" style (no vertical lines) */
+table.docutils.booktabs {
+  border: 0px;
+  border-top: 2px solid;
+  border-bottom: 2px solid;
+  border-collapse: collapse;
+}
+table.docutils.booktabs * {
+  border: 0px;
+}
+table.docutils.booktabs th {
+  border-bottom: thin solid;
+  text-align: left;
+}
+
+h1 tt.docutils, h2 tt.docutils, h3 tt.docutils,
+h4 tt.docutils, h5 tt.docutils, h6 tt.docutils {
+  font-size: 100% }
+
+ul.auto-toc {
+  list-style-type: none }
+
+</style>
+</head>
+<body>
+<div class="document" id="online-bank-statements-adyen-payment-report">
+<h1 class="title">Online Bank Statements: Adyen payment report</h1>
+
+<!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! This file is generated by oca-gen-addon-readme !!
+!! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
+<p><a class="reference external" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external" href="https://github.com/OCA/bank-statement-import/tree/13.0/account_bank_statement_import_online_adyen"><img alt="OCA/bank-statement-import" src="https://img.shields.io/badge/github-OCA%2Fbank--statement--import-lightgray.png?logo=github" /></a> <a class="reference external" href="https://translation.odoo-community.org/projects/bank-statement-import-13-0/bank-statement-import-13-0-account_bank_statement_import_online_adyen"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external" href="https://runbot.odoo-community.org/runbot/174/13.0"><img alt="Try me on Runbot" src="https://img.shields.io/badge/runbot-Try%20me-875A7B.png" /></a></p>
+<p>This module automates the download and import of Adyen payment reports.</p>
+<p><strong>Table of contents</strong></p>
+<div class="contents local topic" id="contents">
+<ul class="simple">
+<li><a class="reference internal" href="#configuration" id="id1">Configuration</a></li>
+<li><a class="reference internal" href="#usage" id="id2">Usage</a></li>
+<li><a class="reference internal" href="#bug-tracker" id="id3">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="id4">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="id5">Authors</a></li>
+<li><a class="reference internal" href="#contributors" id="id6">Contributors</a></li>
+<li><a class="reference internal" href="#maintainers" id="id7">Maintainers</a></li>
+</ul>
+</li>
+</ul>
+</div>
+<div class="section" id="configuration">
+<h1><a class="toc-backref" href="#id1">Configuration</a></h1>
+<p>To configure online bank statements provider:</p>
+<ol class="arabic simple">
+<li>Go to <em>Invoicing &gt; Configuration &gt; Bank Accounts</em></li>
+<li>Open bank account to configure and edit it</li>
+<li>Set <em>Bank Feeds</em> to <em>Online</em></li>
+<li>Select <em>Adyen</em> as online bank statements provider in
+<em>Online Bank Statements (OCA)</em> section</li>
+<li>Save the bank account</li>
+<li>Click on provider and configure provider-specific settings.</li>
+</ol>
+<p>or, alternatively:</p>
+<ol class="arabic simple">
+<li>Go to <em>Invoicing &gt; Overview</em></li>
+<li>Open settings of the corresponding journal account</li>
+<li>Switch to <em>Bank Account</em> tab</li>
+<li>Set <em>Bank Feeds</em> to <em>Online</em></li>
+<li>Select <em>Adyen</em> as online bank statements provider in
+<em>Online Bank Statements (OCA)</em> section</li>
+<li>Save the bank account</li>
+<li>Click on provider and configure provider-specific settings.</li>
+</ol>
+<p>To obtain <em>Login</em> and <em>Key</em>:</p>
+<ol class="arabic simple">
+<li>Open <a class="reference external" href="https://www.adyen.com/signup/">Adyen website</a>.</li>
+</ol>
+<p>Check also <tt class="docutils literal">account_bank_statement_import_online</tt> configuration instructions
+for more information.</p>
+</div>
+<div class="section" id="usage">
+<h1><a class="toc-backref" href="#id2">Usage</a></h1>
+<p>To pull historical bank statements:</p>
+<ol class="arabic simple">
+<li>Go to <em>Invoicing &gt; Configuration &gt; Bank Accounts</em></li>
+<li>Select specific bank accounts</li>
+<li>Launch <em>Actions &gt; Online Bank Statements Pull Wizard</em></li>
+<li>Configure date interval and click <em>Pull</em></li>
+</ol>
+<p>If historical data is not needed, then just simply wait for the scheduled
+activity “Pull Online Bank Statements” to be executed for getting new
+transactions.</p>
+</div>
+<div class="section" id="bug-tracker">
+<h1><a class="toc-backref" href="#id3">Bug Tracker</a></h1>
+<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/bank-statement-import/issues">GitHub Issues</a>.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us smashing it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/OCA/bank-statement-import/issues/new?body=module:%20account_bank_statement_import_online_adyen%0Aversion:%2013.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<p>Do not contact contributors directly about support or help with technical issues.</p>
+</div>
+<div class="section" id="credits">
+<h1><a class="toc-backref" href="#id4">Credits</a></h1>
+<div class="section" id="authors">
+<h2><a class="toc-backref" href="#id5">Authors</a></h2>
+<ul class="simple">
+<li>Ronald Portier (Therp BV)</li>
+</ul>
+</div>
+<div class="section" id="contributors">
+<h2><a class="toc-backref" href="#id6">Contributors</a></h2>
+<ul class="simple">
+<li>Ronald Portier - Therp BV</li>
+</ul>
+</div>
+<div class="section" id="maintainers">
+<h2><a class="toc-backref" href="#id7">Maintainers</a></h2>
+<p>This module is maintained by the OCA.</p>
+<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/bank-statement-import/tree/13.0/account_bank_statement_import_online_adyen">OCA/bank-statement-import</a> project on GitHub.</p>
+<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
+</div>
+</div>
+</div>
+</body>
+</html>

account_bank_statement_import_online_adyen/tests/__init__.py

@@ -0,0 +1,2 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+from . import test_import_online

account_bank_statement_import_online_adyen/tests/online_bank_statement_provider_dummy.py

@@ -0,0 +1,23 @@
+# Copyright 2021 Therp BV <https://therp.nl>.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+import base64
+
+from odoo import models
+from odoo.modules.module import get_module_resource
+
+
+class OnlineBankStatementProviderDummy(models.Model):
+    _inherit = "online.bank.statement.provider"
+
+    def _adyen_get_settlement_details_file(self):
+        """Get file from disk, instead of from url."""
+        if self.service != "dummy_adyen":
+            # Not a dummy, get the regular adyen method.
+            return super()._adyen_get_settlement_details_file()
+        testfile = get_module_resource(
+            "account_bank_statement_import_adyen", "test_files", self.download_file_name
+        )
+        with open(testfile, "rb") as datafile:
+            data_file = datafile.read()
+            data_file = base64.b64encode(data_file)
+        return data_file

account_bank_statement_import_online_adyen/tests/test_import_online.py

@@ -0,0 +1,66 @@
+# Copyright 2021 Therp BV <https://therp.nl>.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+from dateutil.relativedelta import relativedelta
+
+from odoo import fields
+
+from odoo.addons.account_bank_statement_import_adyen.tests.test_import_adyen import (
+    TestImportAdyen,
+)
+
+
+class TestImportOnline(TestImportAdyen):
+    """Do the same tests as with the offline adyen import."""
+
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.now = fields.Datetime.now()
+        cls.journal.write(
+            {
+                "bank_statements_source": "online",
+                "online_bank_statement_provider": "dummy_adyen",
+            }
+        )
+
+    def test_03_import_adyen_invalid(self):
+        """Override super test: online module test will return without statements."""
+        with self.assertRaisesRegex(AssertionError, "account.bank.statement()"):
+            self._test_statement_import(
+                "adyen_test_invalid.xls", "invalid",
+            )
+
+    def _test_statement_import(self, file_name, statement_name):
+        """Test correct creation of single statement.
+
+        Getting an adyen statement online should result in:
+            1. A valid imported statement;
+            2. An incremented batch number;
+            3. The current date being set as the date_since in the provider.
+        """
+        provider = self.journal.online_bank_statement_provider_id
+        provider.write(
+            {
+                "api_base": (
+                    "https://ca-test.adyen.com/reports/download/MerchantAccount"
+                ),
+                "download_file_name": file_name,
+                "interval_type": "days",
+                "interval_number": 1,
+                "service": "dummy_adyen",
+                "next_batch_number": 1,
+            }
+        )
+        # Pull from yesterday, until today
+        yesterday = self.now - relativedelta(days=1)
+        provider.with_context(scheduled=True)._pull(yesterday, self.now)
+        # statement name is account number + '-' + date of last line:
+        statements = self.env["account.bank.statement"].search(
+            [("name", "=", statement_name)]
+        )
+        self.assertTrue(statements)
+        self.assertEqual(len(statements), 1)
+        self.assertEqual(provider.next_batch_number, 2)
+        self.assertEqual(provider.last_successful_run, self.now)
+        self.assertTrue(provider.next_run > self.now)
+        return statements

account_bank_statement_import_online_adyen/views/online_bank_statement_provider.xml

@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<odoo>
+    <record model="ir.ui.view" id="online_bank_statement_provider_form">
+        <field name="name">online.bank.statement.provider.form</field>
+        <field name="model">online.bank.statement.provider</field>
+        <field
+            name="inherit_id"
+            ref="account_bank_statement_import_online.online_bank_statement_provider_form"
+        />
+        <field name="arch" type="xml">
+            <xpath expr="//page[@name='configuration']" position="inside">
+                <group name="adyen" attrs="{'invisible':[('service','!=','adyen')]}">
+                    <field
+                        name="api_base"
+                        placeholder="https://ca-test.adyen.com/reports/download/MerchantAccount"
+                    />
+                    <field name="username" />
+                    <field name="password" />
+                    <field
+                        name="download_file_name"
+                        placeholder="settlement_detail_report_batch_%s.xlsx"
+                    />
+                    <field name="next_batch_number" />
+                </group>
+            </xpath>
+        </field>
+    </record>
+</odoo>