Skip to content

Commit

Permalink
[IMP] account_reconcile_widget: Allow to reconcile payment/debit orde…
Browse files Browse the repository at this point in the history
…rs from OCA/bank-payment

Payments without payment_id related aren't showed in statement
reconciliation.

With this change, we show them as blue lines, the same as other coming
from Odoo core payments.

TT34508
  • Loading branch information
pedrobaeza committed Feb 14, 2022
1 parent e5ab5ff commit 78e9e3a
Showing 1 changed file with 13 additions and 17 deletions.
30 changes: 13 additions & 17 deletions account_reconciliation_widget/models/reconciliation_widget.py
Original file line number Diff line number Diff line change
Expand Up @@ -769,15 +769,25 @@ def _domain_move_lines_for_reconciliation(
excluded_ids.extend(to_check_excluded)

domain_reconciliation = [
"&",
"&",
"&",
("statement_line_id", "=", False),
("account_id", "in", aml_accounts),
("payment_id", "<>", False),
("balance", "!=", 0.0),
]

if st_line.company_id.account_bank_reconciliation_start:
domain_reconciliation = expression.AND(
[
domain_reconciliation,
[
(
"date",
">=",
st_line.company_id.account_bank_reconciliation_start,
)
],
]
)
# default domain matching
domain_matching = [
"&",
Expand Down Expand Up @@ -830,20 +840,6 @@ def _domain_move_lines_for_reconciliation(
# filter on account.move.line having the same company as the statement
# line
domain = expression.AND([domain, [("company_id", "=", st_line.company_id.id)]])

if st_line.company_id.account_bank_reconciliation_start:
domain = expression.AND(
[
domain,
[
(
"date",
">=",
st_line.company_id.account_bank_reconciliation_start,
)
],
]
)
return domain

@api.model
Expand Down

0 comments on commit 78e9e3a

Please sign in to comment.