Repository navigation
auto postings break redundant equity/cost detection and transaction balancing #2110
Description
This minimal example is balanced if --auto is not passed, but reported as unbalanced with --auto, despite the auto posting rule not applying to any of the postings in the transaction afaict:
2023-11-01 * conversion posting
assets:shares 7 TEST @@ 50.00€
equity:conversion -7 TEST
equity:conversion 50.00€
equity:opening -50.00€
= expenses:foobar
expenses:foobar *-1
expenses:foo *0.5
liabilities:bar *0.5
% hledger print -f auto_equity_conversion_bug_2110.journal
2023-11-01 * conversion posting
assets:shares 7 TEST @@ 50.00€
equity:conversion -7 TEST
equity:conversion 50.00€
equity:opening -50.00€
% hledger print --auto -f auto_equity_conversion_bug_2110.journal
hledger: Error: /...../auto_equity_conversion_bug_2110.journal:1-5:
1 | 2023-11-01 * conversion posting
| assets:shares 7 TEST @@ 50.00€
| equity:conversion -7 TEST
| equity:conversion 50.00€
| equity:opening -50.00€
This multi-commodity transaction is unbalanced.
The real postings' sum should be 0 but is: -7 TEST, 50.00€
Consider adjusting this entry's amounts, adding missing postings,
or recording conversion price(s) with @, @@ or equity postings.
Tested with 1.30, 1.31 and master, all running on Linux (NixOS unstable).
If I remove the conversion postings it works, even with --infer-equity (and of course it works without the auto posting). The manual mentions that transactions need to be balanced before and after auto postings are applied, but I think that's the case here as the auto posting rule doesn't even match anything?
Activity
fix: auto postings: detect redundant costs properly (fix #2110)
simonmichael commentedon Nov 7, 2023
Good catch! Thank you, fixed in master.
[-]auto postings break equity conversion balance[/-][+]auto postings break redundant equity/cost detection and transaction balancing[/+]