Bug 494865

Summary: ledger view: Ctrl+A selects all transactions but does not update balance
Product: [Applications] kmymoney Reporter: Moritz Bunkus <kde>
Component: ux-uiAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: minor    
Priority: NOR    
Version: 5.1.3   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 5.2
Sentry Crash Report:

Description Moritz Bunkus 2024-10-16 13:31:16 UTC 
SUMMARY

In the ledger view you can select multiple transactions the usual ways, e.g. by holding Ctrl while mouse-clicking on them, or holding Shift & mouse-clicking for selecting a whole region. When multiple transactions are selected that way, the balance shown below the transaction list on the right will be updated with the balance over all selected transactions.

Pressing Ctrl+A is another well-known way to select multiple entries at once in general, and it works in the ledger view, too: it selects all transactions, just as I would expect it to. However, whenever I use Ctrl+A the balance is not updated at all, no matter of I had no transactions selected before, one or any other number.

STEPS TO REPRODUCE
1. Open ledger view
2. Select multiple transactions with e.g. the Ctrl & mouse-click or the shift & mouse-click method.
3. Observe the balance updating.
4. Deselect all transactions.
5. Select all transactions by pressing Ctrl+A
6. Observe that the balance is not updated

OBSERVED RESULT

The balance is not updated when I use Ctrl+A to select all transactions.

EXPECTED RESULT

The balance should be updated no matter which method I use for selecting multiple transactions, including Ctrl+A.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 6.1.5
KDE Frameworks Version: 6.6.0
Qt Version: 6.7.2
Comment 1 Thomas Baumgart 2024-10-19 06:53:03 UTC 
In case multiple transactions are selected, the sum of those transactions (actually the splits referencing the current selected account) is shown in the location where you would normally see the balance. Using Ctrl+A one would expect to see the same value as for the balance unless you have filtered out some transactions (e.g. don't show reconciled ones).
Comment 2 Moritz Bunkus 2024-10-19 09:56:38 UTC 
I see your point, but respectfully, you're wrong. Even if no transactions are hidden, the thing that's shown should be the sum sign & the balance of the selected transactions instead of the word "Balance:" followed by the sum. This isn't the case. Here's what actually happens:

• no transactions selected at all; you'll see: "Balance: 12345" (example values, obviously)
• select some transactions via Ctrl + mouse-clicking; you'll see: "Σ: 78"
• select all transactions via Ctrl+A; you'll still see: "Σ: 78" (!!!! obviously wrong!)

On top of that you can also try the following sequence & see how things change:

• same starting point as before: no transactions selected at all; you'll see: "Balance: 12345"
• select all transactions via Ctrl+A; you'll still see: "Balance: 12345"
• deselect a single of the selected transactions via Ctrl + mouse-clicking; you might now see: "Σ: 989"

All of this is without any filter active, nor with "hide reconciled transactions".

Now for a third test: I have prepared an leder & inserted three transactions into it, values are +10, +5 & +2. The +10 is reconciled. Then I turn on "hide reconciled transactions". Now:

• initially with no transactions selected I see "Balance: 17"
• selecting all via Ctrl+A doesn't change that value at all, even though only the two transactions with +5 & +2 are selected; I would expect to see "Σ: 7", instead I still see "Balance: 17"
• now I deselect one transaction with Ctrl + mouse-clicking; it instantly changes to "Σ: 2" (if I deselect the +5).
• I re-select the deselected one via Ctrl + mouse-clicking so that all visible (not reconciled) transactions are selected; I now see the expected "Σ: 7"

It's quite obvious that changing the selection of transactions via Ctrl+A doesn't actually trigger the signal to update the value shown there, making it temporarily invalid.

Please just give it a try & you'll see that this is indeed a bug. The instructions are easy to follow, the first two examples don't even require changes to be made. Thanks.
Comment 3 Thomas Baumgart 2024-10-19 10:09:08 UTC 
Sorry for the confusion, but it seems we're comparing apples and oranges here (aka stable vs. development): you are most likely referring the stable version 5.1.3 (as mentioned in the "Version" entry of this bug report) which shows this problem (I have not verified it) while I am talking about the development version (to be released as version 5.2) which I mentioned in the "Version fixed in" field of the bug report.

If you want to check yourself, you can find preview versions on the download page at https://kmymoney.org/appimage.html as installable AppImage. That is why I consider this as FIXED (or UNMAINTAINED if you want, as we won't add anymore fixes to the 5.1 branch).
Comment 4 Moritz Bunkus 2024-10-19 10:13:46 UTC 
Ooooh you're right, I had totally overlooked the "version fixed in change" for some reason. Seems I was confused by the resolution of "reported" + "later". Sorry for that. Yes, I am talking about release 5.1.3; I've tried building the current git master, but that won't work due to library requirements, and I didn't want to invest too much time into it. Therefore I had only tested with 5.1.3.

Now let me give me a quick try with the development AppImage… yep, it's indeed fixed there. Thanks!