johannesgerer / buchhaltung

Smart tools for the ledger universe
http://johannesgerer.com/buchhaltung
MIT License
97 stars 8 forks source link

buchhaltung match needs multiple iterations for all open transactions #30

Open varac opened 6 years ago

varac commented 6 years ago

buchhaltung update found 12 new transactions:

found 12 new of 143 total transactions

First round of buchhaltung match:

» buchhaltung match
...
Current Transaction: 1, Remaining: 11
..
Current Transaction: 3, Remaining: 9
...
Current Transaction: 5, Remaining: 7
...
Current Transaction: 7, Remaining: 5
...
Current Transaction: 9, Remaining: 3
...
Current Transaction: 11, Remaining: 1
...
Current Transaction: 12, Remaining: 0
...
Current Transaction: 12, Remaining: 0
<< DONE! Use 'save' to exit >>
[<, >, save, RET]:  save

3 Transactions were changed

But we need more rounds to match all transactions.

Second round:

» buchhaltung match
...
Current Transaction: 1, Remaining: 4
...
Current Transaction: 3, Remaining: 2
...
Current Transaction: 5, Remaining: 0
...
Current Transaction: 5, Remaining: 0
<< DONE! Use 'save' to exit >>
...
3 Transactions were changed

Third round ...

johannesgerer commented 6 years ago

Would you be able to post a minimal reproducible example containing of an import.ledger and the account you enter in matching?

Until I see that, I can only guess:

Assuming there is now bug in the code, the only way to get this behaviour is by entering the account TODO, which then it will be picked up if you rerun buchhaltung match.

ohbadiah commented 6 years ago

This isn't, by chance, the same issue addressed in #7, is it? This was the fix.

ohbadiah commented 6 years ago

I think this issue can be closed, since it was indeed found to be the same as reported in #7, and released with version 0.0.7.