[IMP] account,*: Improve bank statements/payments workflow

- Create journal entries as soon as bank/cash statement lines are created, temporary booked on a suspense account set on the journal.
- Simplify the management of "blue" lines in the reconciliation widget. A "blue" line is now a journal item using a temporary liquidity account (outstanding payment/receipt accounts, set on the journal).
- Adapt and simplify the bank reconciliation report.
- Remove the bank reconciliation threshold date. The reconciliation report will show the not already reconciled journal entries using a liquidity account and the not already reconciled journal entries using a temporary liquidity account. Without accounting, an account.payment will involve directly the liquidity account and then, will be considered as a statement line directly.
- Remove the post_at bank reconciliation feature. The "paid" state will be set on the invoices only if reconciled with a journal entry involving the journal's liquidity account.
    With invoicing, the payment will do that so the "in_payment" state should never be shown up.
    With accounting, only the statement lines have the power to move an invoice to the "paid" state.
- Fix various corner cases about the management of multi-currency in bank statement lines.
- Fix the conversion dates in multi-currency: Since the bank/cash is always used on the statement lines, it will use always the real "bank" date instead of the fictive payment one.
- Ensure the 'reconcile' method will raise an error if the involved moves are not posted.

related enterprise PR odoo/enterprise#7019

closes odoo/odoo#41301

--task: 2092096
Related: odoo/upgrade#1018
Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>

1 files changed, 8 insertions, 8 deletions

diff --git a/addons/membership/tests/test_membership.py b/addons/membership/tests/test_membership.py
index d315983138c..ae6c05aa094 100644
--- a/addons/membership/tests/test_membership.py
+++ b/addons/membership/tests/test_membership.py
@@ -53,14 +53,14 @@ class TestMembership(TestMembershipCommon):
 
         # the invoice is paid -> customer goes to paid status
         bank_journal = self.env['account.journal'].create({'name': 'Bank', 'type': 'bank', 'code': 'BNK67'})
-        self.env['account.payment'].create({
-            'payment_method_id': self.env.ref("account.account_payment_method_manual_in").id,
-            'payment_type': 'inbound',
-            'invoice_ids': [(6, False, invoice.ids)],
-            'amount': 86.25,
-            'journal_id': bank_journal.id,
-            'partner_type': 'customer',
-        }).post()
+
+        payment = self.env['account.payment.register']\
+            .with_context(active_model='account.move', active_ids=invoice.ids)\
+            .create({
+                'amount': 86.25,
+                'journal_id': bank_journal.id,
+            })\
+            ._create_payments()
 
         self.assertEqual(
             self.partner_1.membership_state, 'paid',