[IMP] account_payment_partner: pre-commit auto fixes

This commit is contained in:
Miquel Alzanillas
2023-11-22 22:18:54 +01:00
parent 6034b7348e
commit 4d46d09a4b
13 changed files with 103 additions and 110 deletions

View File

@@ -17,39 +17,34 @@ Account Payment Partner
: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--payment-lightgray.png?logo=github
:target: https://github.com/OCA/bank-payment/tree/16.0/account_payment_partner
:target: https://github.com/OCA/bank-payment/tree/17.0/account_payment_partner
:alt: OCA/bank-payment
.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
:target: https://translation.odoo-community.org/projects/bank-payment-16-0/bank-payment-16-0-account_payment_partner
:target: https://translation.odoo-community.org/projects/bank-payment-17-0/bank-payment-17-0-account_payment_partner
:alt: Translate me on Weblate
.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
:target: https://runboat.odoo-community.org/builds?repo=OCA/bank-payment&target_branch=16.0
:target: https://runboat.odoo-community.org/builds?repo=OCA/bank-payment&target_branch=17.0
:alt: Try me on Runboat
|badge1| |badge2| |badge3| |badge4| |badge5|
This module adds several fields:
* the *Supplier Payment Mode* and *Customer Payment Mode* on Partners,
- the *Supplier Payment Mode* and *Customer Payment Mode* on Partners,
- the *Payment Mode* on Invoices.
- the *Show bank account* on Payment Mode.
- the *# of digits for customer bank account* on Payment Mode.
- the *Bank account from journals* on Payment Mode.
- the *Payment mode* on Invoices Analysis.
* the *Payment Mode* on Invoices.
On a Payment Order, in the wizard *Select Invoices to Pay*, the invoices
will be filtered per Payment Mode.
* the *Show bank account* on Payment Mode.
* the *# of digits for customer bank account* on Payment Mode.
* the *Bank account from journals* on Payment Mode.
* the *Payment mode* on Invoices Analysis.
On a Payment Order, in the wizard *Select Invoices to Pay*, the invoices will
be filtered per Payment Mode.
Allows to print in the invoice to which account number the payment
(via SEPA direct debit) is going to be charged so the customer knows that
Allows to print in the invoice to which account number the payment (via
SEPA direct debit) is going to be charged so the customer knows that
information, but there are some customers that don't want that everyone
looking at the invoice sees the full account number (and even GDPR can say a
word about that), so that's the reason behind the several options.
looking at the invoice sees the full account number (and even GDPR can
say a word about that), so that's the reason behind the several options.
**Table of contents**
@@ -61,11 +56,11 @@ Usage
You are able to add a payment mode directly on a partner.
This payment mode is automatically associated to the invoice related to the
partner. This default value could be changed in a draft invoice.
This payment mode is automatically associated to the invoice related to
the partner. This default value could be changed in a draft invoice.
When you create a payment order, only invoices related to chosen payment mode
are displayed.
When you create a payment order, only invoices related to chosen payment
mode are displayed.
Invoices without any payment mode are displayed too.
@@ -73,10 +68,10 @@ Changelog
=========
10.0.1.2.0 (2018-05-24)
~~~~~~~~~~~~~~~~~~~~~~~
-----------------------
* [IMP] Add options to show partner bank account in invoice report
(`#458 <https://github.com/OCA/bank-payment/issues/458>`_)
- [IMP] Add options to show partner bank account in invoice report
(`#458 <https://github.com/OCA/bank-payment/issues/458>`__)
Bug Tracker
===========
@@ -84,7 +79,7 @@ Bug Tracker
Bugs are tracked on `GitHub Issues <https://github.com/OCA/bank-payment/issues>`_.
In case of trouble, please check there if your issue has already been reported.
If you spotted it first, help us to smash it by providing a detailed and welcomed
`feedback <https://github.com/OCA/bank-payment/issues/new?body=module:%20account_payment_partner%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
`feedback <https://github.com/OCA/bank-payment/issues/new?body=module:%20account_payment_partner%0Aversion:%2017.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.
@@ -92,33 +87,35 @@ Credits
=======
Authors
~~~~~~~
-------
* Akretion
* Tecnativa
Contributors
~~~~~~~~~~~~
------------
* Alexis de Lattre <alexis.delattre@akretion.com>
* Raphaël Valyi
* Stefan Rijnhart (Therp)
* Alexandre Fayolle
* Stéphane Bidoul <stephane.bidoul@acsone.eu>
* Danimar Ribeiro
* Angel Moya <angel.moya@domatix.com>
* `Tecnativa <https://www.tecnativa.com>`_:
- Alexis de Lattre <alexis.delattre@akretion.com>
- Raphaël Valyi
- Stefan Rijnhart (Therp)
- Alexandre Fayolle
- Stéphane Bidoul <stephane.bidoul@acsone.eu>
- Danimar Ribeiro
- Angel Moya <angel.moya@domatix.com>
- `Tecnativa <https://www.tecnativa.com>`__:
* Pedro M. Baeza
* Carlos Dauden
* Víctor Martínez
* `DynApps <https://www.dynapps.be>`_:
- Pedro M. Baeza
- Carlos Dauden
- Víctor Martínez
* Raf Ven <raf.ven@dynapps.be>
* Marçal Isern <marsal.isern@qubiq.es>
- `DynApps <https://www.dynapps.be>`__:
- Raf Ven <raf.ven@dynapps.be>
- Marçal Isern <marsal.isern@qubiq.es>
Maintainers
~~~~~~~~~~~
-----------
This module is maintained by the OCA.
@@ -130,6 +127,6 @@ 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-payment <https://github.com/OCA/bank-payment/tree/16.0/account_payment_partner>`_ project on GitHub.
This module is part of the `OCA/bank-payment <https://github.com/OCA/bank-payment/tree/17.0/account_payment_partner>`_ project on GitHub.
You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

View File

@@ -6,7 +6,7 @@
{
"name": "Account Payment Partner",
"version": "16.0.1.2.2",
"version": "17.0.1.0.0",
"category": "Banking addons",
"license": "AGPL-3",
"summary": "Adds payment mode on partners and invoices",

View File

@@ -130,7 +130,7 @@ class AccountMove(models.Model):
def _reverse_moves(self, default_values_list=None, cancel=False):
if not default_values_list:
default_values_list = [{} for _ in self]
for move, default_values in zip(self, default_values_list):
for move, default_values in zip(self, default_values_list, strict=True):
default_values[
"payment_mode_id"
] = move.payment_mode_id.refund_payment_mode_id.id

View File

@@ -0,0 +1,3 @@
[build-system]
requires = ["whool"]
build-backend = "whool.buildapi"

View File

@@ -0,0 +1,15 @@
- Alexis de Lattre \<<alexis.delattre@akretion.com>\>
- Raphaël Valyi
- Stefan Rijnhart (Therp)
- Alexandre Fayolle
- Stéphane Bidoul \<<stephane.bidoul@acsone.eu>\>
- Danimar Ribeiro
- Angel Moya \<<angel.moya@domatix.com>\>
- [Tecnativa](https://www.tecnativa.com):
- Pedro M. Baeza
- Carlos Dauden
- Víctor Martínez
- [DynApps](https://www.dynapps.be):
- Raf Ven \<<raf.ven@dynapps.be>\>
- Marçal Isern \<<marsal.isern@qubiq.es>\>
- Miquel Alzanillas \<<malzanillas@apsl.net>\>

View File

@@ -1,16 +0,0 @@
* Alexis de Lattre <alexis.delattre@akretion.com>
* Raphaël Valyi
* Stefan Rijnhart (Therp)
* Alexandre Fayolle
* Stéphane Bidoul <stephane.bidoul@acsone.eu>
* Danimar Ribeiro
* Angel Moya <angel.moya@domatix.com>
* `Tecnativa <https://www.tecnativa.com>`_:
* Pedro M. Baeza
* Carlos Dauden
* Víctor Martínez
* `DynApps <https://www.dynapps.be>`_:
* Raf Ven <raf.ven@dynapps.be>
* Marçal Isern <marsal.isern@qubiq.es>

View File

@@ -0,0 +1,17 @@
This module adds several fields:
- the *Supplier Payment Mode* and *Customer Payment Mode* on Partners,
- the *Payment Mode* on Invoices.
- the *Show bank account* on Payment Mode.
- the *\# of digits for customer bank account* on Payment Mode.
- the *Bank account from journals* on Payment Mode.
- the *Payment mode* on Invoices Analysis.
On a Payment Order, in the wizard *Select Invoices to Pay*, the invoices
will be filtered per Payment Mode.
Allows to print in the invoice to which account number the payment (via
SEPA direct debit) is going to be charged so the customer knows that
information, but there are some customers that don't want that everyone
looking at the invoice sees the full account number (and even GDPR can
say a word about that), so that's the reason behind the several options.

View File

@@ -1,22 +0,0 @@
This module adds several fields:
* the *Supplier Payment Mode* and *Customer Payment Mode* on Partners,
* the *Payment Mode* on Invoices.
* the *Show bank account* on Payment Mode.
* the *# of digits for customer bank account* on Payment Mode.
* the *Bank account from journals* on Payment Mode.
* the *Payment mode* on Invoices Analysis.
On a Payment Order, in the wizard *Select Invoices to Pay*, the invoices will
be filtered per Payment Mode.
Allows to print in the invoice to which account number the payment
(via SEPA direct debit) is going to be charged so the customer knows that
information, but there are some customers that don't want that everyone
looking at the invoice sees the full account number (and even GDPR can say a
word about that), so that's the reason behind the several options.

View File

@@ -0,0 +1,4 @@
## 10.0.1.2.0 (2018-05-24)
- \[IMP\] Add options to show partner bank account in invoice report
([\#458](https://github.com/OCA/bank-payment/issues/458))

View File

@@ -1,5 +0,0 @@
10.0.1.2.0 (2018-05-24)
~~~~~~~~~~~~~~~~~~~~~~~
* [IMP] Add options to show partner bank account in invoice report
(`#458 <https://github.com/OCA/bank-payment/issues/458>`_)

View File

@@ -1,9 +1,9 @@
You are able to add a payment mode directly on a partner.
This payment mode is automatically associated to the invoice related to the
partner. This default value could be changed in a draft invoice.
This payment mode is automatically associated to the invoice related to
the partner. This default value could be changed in a draft invoice.
When you create a payment order, only invoices related to chosen payment mode
are displayed.
When you create a payment order, only invoices related to chosen payment
mode are displayed.
Invoices without any payment mode are displayed too.

View File

@@ -369,7 +369,7 @@ ul.auto-toc {
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!! source digest: sha256:1e2b38367db6bacc47711402f2eb9ef0f076f2c8a0317874407c547af811eb59
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Mature" src="https://img.shields.io/badge/maturity-Mature-brightgreen.png" /></a> <a class="reference external image-reference" 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 image-reference" href="https://github.com/OCA/bank-payment/tree/16.0/account_payment_partner"><img alt="OCA/bank-payment" src="https://img.shields.io/badge/github-OCA%2Fbank--payment-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/bank-payment-16-0/bank-payment-16-0-account_payment_partner"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/bank-payment&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Mature" src="https://img.shields.io/badge/maturity-Mature-brightgreen.png" /></a> <a class="reference external image-reference" 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 image-reference" href="https://github.com/OCA/bank-payment/tree/17.0/account_payment_partner"><img alt="OCA/bank-payment" src="https://img.shields.io/badge/github-OCA%2Fbank--payment-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/bank-payment-17-0/bank-payment-17-0-account_payment_partner"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/bank-payment&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
<p>This module adds several fields:</p>
<ul class="simple">
<li>the <em>Supplier Payment Mode</em> and <em>Customer Payment Mode</em> on Partners,</li>
@@ -379,13 +379,13 @@ ul.auto-toc {
<li>the <em>Bank account from journals</em> on Payment Mode.</li>
<li>the <em>Payment mode</em> on Invoices Analysis.</li>
</ul>
<p>On a Payment Order, in the wizard <em>Select Invoices to Pay</em>, the invoices will
be filtered per Payment Mode.</p>
<p>Allows to print in the invoice to which account number the payment
(via SEPA direct debit) is going to be charged so the customer knows that
<p>On a Payment Order, in the wizard <em>Select Invoices to Pay</em>, the invoices
will be filtered per Payment Mode.</p>
<p>Allows to print in the invoice to which account number the payment (via
SEPA direct debit) is going to be charged so the customer knows that
information, but there are some customers that dont want that everyone
looking at the invoice sees the full account number (and even GDPR can say a
word about that), so thats the reason behind the several options.</p>
looking at the invoice sees the full account number (and even GDPR can
say a word about that), so thats the reason behind the several options.</p>
<p><strong>Table of contents</strong></p>
<div class="contents local topic" id="contents">
<ul class="simple">
@@ -406,10 +406,10 @@ word about that), so thats the reason behind the several options.</p>
<div class="section" id="usage">
<h1><a class="toc-backref" href="#toc-entry-1">Usage</a></h1>
<p>You are able to add a payment mode directly on a partner.</p>
<p>This payment mode is automatically associated to the invoice related to the
partner. This default value could be changed in a draft invoice.</p>
<p>When you create a payment order, only invoices related to chosen payment mode
are displayed.</p>
<p>This payment mode is automatically associated to the invoice related to
the partner. This default value could be changed in a draft invoice.</p>
<p>When you create a payment order, only invoices related to chosen payment
mode are displayed.</p>
<p>Invoices without any payment mode are displayed too.</p>
</div>
<div class="section" id="changelog">
@@ -427,7 +427,7 @@ are displayed.</p>
<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/bank-payment/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 to smash it by providing a detailed and welcomed
<a class="reference external" href="https://github.com/OCA/bank-payment/issues/new?body=module:%20account_payment_partner%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
<a class="reference external" href="https://github.com/OCA/bank-payment/issues/new?body=module:%20account_payment_partner%0Aversion:%2017.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">
@@ -469,7 +469,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
<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-payment/tree/16.0/account_payment_partner">OCA/bank-payment</a> project on GitHub.</p>
<p>This module is part of the <a class="reference external" href="https://github.com/OCA/bank-payment/tree/17.0/account_payment_partner">OCA/bank-payment</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>

View File

@@ -251,7 +251,7 @@ class TestAccountPaymentPartner(TransactionCase):
)
invoice.action_post()
aml = invoice.line_ids.filtered(
lambda l: l.account_id.account_type == "liability_payable"
lambda x: x.account_id.account_type == "liability_payable"
)
self.assertEqual(invoice.payment_mode_id, aml[0].payment_mode_id)
# Test payment mode change on aml
@@ -269,7 +269,7 @@ class TestAccountPaymentPartner(TransactionCase):
)
invoice.action_post()
aml = invoice.line_ids.filtered(
lambda l: l.account_id.account_type == "asset_receivable"
lambda x: x.account_id.account_type == "asset_receivable"
)
self.assertEqual(invoice.payment_mode_id, aml[0].payment_mode_id)