You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Xitian9 opened this issue
Dec 1, 2023
· 3 comments
Labels
A-BUGSomething wrong, confusing or sub-standard in the software, docs, or user experience.docsDocumentation-related.impact3Affects just a few users.journalThe journal file format, and its features.severity2Minor to moderate usability/doc bug, reasonably easy to avoid or tolerate.
Occasionally one may want to use --alias to combine different accounts into one. However, if one does this, balance assertions may start to fail because the assertions meant for the original accounts no longer hold for the merged account names.
This issue can be worked around by using --ignore-assertions, so this is not a high priority, but it seems suboptimal. Perhaps account aliases should only be applied after parsing and verifying assertions.
A slightly larger issue arises in that balance assignments will give different values if aliases are applied. This is in some sense worse because the output of the journal will silently be different, rather than raising an ignorable error. However, some people may rely on this functionality to generate different output in different cases. That seems dangerous and probably shouldn't be allowed.
The text was updated successfully, but these errors were encountered:
Xitian9
added
the
A-BUG
Something wrong, confusing or sub-standard in the software, docs, or user experience.
label
Dec 1, 2023
simonmichael
added
impact3
Affects just a few users.
docs
Documentation-related.
journal
The journal file format, and its features.
severity2
Minor to moderate usability/doc bug, reasonably easy to avoid or tolerate.
labels
Dec 13, 2023
They are a powerful sharp tool and I'm not sure it's possible to keep everything working with arbitrary aliases. So I'm not sure this is a bug. I'd be glad to try out alternatives though.
I don't think this a bug (but an example might be useful to get your point across).
Maybe the documentation could clarify hledger's order of operations so people can plan accordingly, when using these features together.
A-BUGSomething wrong, confusing or sub-standard in the software, docs, or user experience.docsDocumentation-related.impact3Affects just a few users.journalThe journal file format, and its features.severity2Minor to moderate usability/doc bug, reasonably easy to avoid or tolerate.
Occasionally one may want to use
--alias
to combine different accounts into one. However, if one does this, balance assertions may start to fail because the assertions meant for the original accounts no longer hold for the merged account names.This issue can be worked around by using
--ignore-assertions
, so this is not a high priority, but it seems suboptimal. Perhaps account aliases should only be applied after parsing and verifying assertions.A slightly larger issue arises in that balance assignments will give different values if aliases are applied. This is in some sense worse because the output of the journal will silently be different, rather than raising an ignorable error. However, some people may rely on this functionality to generate different output in different cases. That seems dangerous and probably shouldn't be allowed.
The text was updated successfully, but these errors were encountered: