buchhaltung icon indicating copy to clipboard operation
buchhaltung copied to clipboard

buchhaltung match needs multiple iterations for all open transactions

Open varac opened this issue 7 years ago • 3 comments

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 ...

varac avatar Feb 02 '18 12:02 varac

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.

johannesgerer avatar Feb 04 '18 18:02 johannesgerer

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

ohbadiah avatar Feb 06 '18 20:02 ohbadiah

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.

ohbadiah avatar Mar 09 '18 17:03 ohbadiah