Skip to content

[18.0][FIX] partner_statement: fix account_type comparison in report titles#1470

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
dmoranp:18.0-fix-partner-statement-title
Mar 11, 2026
Merged

[18.0][FIX] partner_statement: fix account_type comparison in report titles#1470
OCA-git-bot merged 1 commit intoOCA:18.0from
dmoranp:18.0-fix-partner-statement-title

Conversation

@dmoranp
Copy link
Copy Markdown
Contributor

@dmoranp dmoranp commented Mar 11, 2026

Description

The _get_title methods in outstanding_statement.py, activity_statement.py, and detailed_activity_statement.py compare account_type against "receivable", but the wizard (statement_common.py) sends "asset_receivable" via a Selection field.

This mismatch causes all partner statements to always display the supplier title variant, regardless of the selected account type.

Changes

  • Updated 5 comparisons across 3 files from "receivable" to "asset_receivable"
    • outstanding_statement.py (line 19)
    • activity_statement.py (lines 23, 38)
    • detailed_activity_statement.py (lines 21, 31)

Fixes #1465

The wizard sends "asset_receivable" but _get_title methods compared
against the old "receivable" value, causing all statements to always
show the supplier title variant.

Fixes OCA#1465
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @MiquelRForgeFlow,
some modules you are maintaining are being modified, check this out!

@MiquelRForgeFlow
Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-1470-by-MiquelRForgeFlow-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e20f124 into OCA:18.0 Mar 11, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at f9f173c. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[18.0] [partner_statement] Activity/Outstanding statement always shows "Supplier Statement" title

3 participants