Skip to content

journal: Respect posting tag overrides - #2587

Open
Wheezily wants to merge 1 commit into
plaintextaccounting:mainfrom
Wheezily:bounty/posting-tag-overrides-transaction-tag
Open

journal: Respect posting tag overrides#2587
Wheezily wants to merge 1 commit into
plaintextaccounting:mainfrom
Wheezily:bounty/posting-tag-overrides-transaction-tag

Conversation

@Wheezily

Copy link
Copy Markdown

Fixes #1950.

Posting tag matching now treats explicitly set posting tags as overriding inherited transaction tags with the same name. Same-level repeated tag values are still kept; only inherited same-name tags are filtered out.

Validation:

  • cabal test hledger-lib:unittest -j1 --test-options='--pattern=override' in haskell:9.8
  • cabal test hledger-lib:unittest -j1 --test-options='--pattern=Query' in haskell:9.8
  • cabal test hledger-lib:unittest -j1 in haskell:9.8
  • git diff --check

@simonmichael

Copy link
Copy Markdown
Collaborator

Thanks for the PR, I have responded on #1950.

@simonmichael simonmichael added the journal The journal file format, and its features. label May 18, 2026
@simonmichael

simonmichael commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Hi @Wheezily, are you still interested in this PR ? These things would help it move forward:

@simonmichael

Copy link
Copy Markdown
Collaborator

See also #1950 (comment).

@simonmichael simonmichael changed the title Respect posting tag overrides journal: Respect posting tag overrides Jul 31, 2026
@simonmichael simonmichael added needs-tests To unblock: needs more automated tests or test updates needs-ai-compliance To unblock: needs AI disclosure or other AI policy requirement labels Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

journal The journal file format, and its features. needs-ai-compliance To unblock: needs AI disclosure or other AI policy requirement needs-tests To unblock: needs more automated tests or test updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inherited Tag Values are not Overwritten [💰$20 for 100% human contributions]

2 participants