Bug 506444 - Price entry buggy in qt6-based master builds
Summary: Price entry buggy in qt6-based master builds
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (other bugs)
Version First Reported In: git (master)
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-07-01 15:38 UTC by Robby Engelmann
Modified: 2025-07-31 05:14 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
Screenshot Testcase (346.05 KB, image/jpeg)
2025-07-02 13:48 UTC, Robby Engelmann
Details
locale settings in KDE (71.31 KB, image/jpeg)
2025-07-02 13:53 UTC, Robby Engelmann
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Robby Engelmann 2025-07-01 15:38:42 UTC
In price entry for a dividend I always have to put three digits after the comma, otherwise the price is changed:
e.g.:
I put -> changed to
12  -> 12
12,0 -> 120,00
12,00 -> 1,20
12,000 -> 12,00
Comment 1 Ralf Habacker 2025-07-02 09:35:40 UTC
Please add detailed instructions on how you encountered the problem. You can start with this example file https://invent.kde.org/office/kmymoney/-/raw/master/kmymoney/plugins/views/reports/core/tests/data/test-investment.xml?ref_type=heads&inline=false
Comment 2 Robby Engelmann 2025-07-02 13:48:00 UTC
Created attachment 182866 [details]
Screenshot Testcase
Comment 3 Robby Engelmann 2025-07-02 13:52:48 UTC
I added an attachment showing the issue in your testcase + my locale settings
Comment 4 Robby Engelmann 2025-07-02 13:53:09 UTC
Created attachment 182867 [details]
locale settings in KDE
Comment 5 Thomas Baumgart 2025-07-06 14:15:15 UTC
There is a difference in your locale settings for numbers (1,000.01) and currencies (1.000,01). KMyMoney is using the currency setting. Does adjusting the currency settings makes a difference and solves the problem?
Comment 6 Felix 2025-07-06 14:37:18 UTC
I am hitting exactly the same issue. Thomas, in what way we should update the currency format?
Comment 7 Thomas Baumgart 2025-07-06 15:17:02 UTC
Git commit 6ac6d13034d19ba574c3c49193c53495e5119a3c by Thomas Baumgart.
Committed on 06/07/2025 at 15:16.
Pushed by tbaumgart into branch 'master'.

Fix a problem with keypad decimal on Qt6

M  +1    -1    kmymoney/widgets/amountedit.cpp

https://invent.kde.org/office/kmymoney/-/commit/6ac6d13034d19ba574c3c49193c53495e5119a3c
Comment 8 Robby Engelmann 2025-07-06 18:19:56 UTC
(In reply to Thomas Baumgart from comment #5)
> There is a difference in your locale settings for numbers (1,000.01) and
> currencies (1.000,01). KMyMoney is using the currency setting. Does
> adjusting the currency settings makes a difference and solves the problem?

I did not see any difference after setting currency to American (default) alike the numbers setting.
Comment 9 Thomas Baumgart 2025-07-07 07:29:26 UTC
Did you restart the desktop session (i.e. logout and login)? Apparently that is necessary from what I noticed.
Comment 10 Robby Engelmann 2025-07-07 10:18:52 UTC
(In reply to Thomas Baumgart from comment #9)
> Did you restart the desktop session (i.e. logout and login)? Apparently that
> is necessary from what I noticed.

not yet, will test later
Comment 11 Robby Engelmann 2025-07-07 11:24:55 UTC
(In reply to Robby Engelmann from comment #10)
> (In reply to Thomas Baumgart from comment #9)
> > Did you restart the desktop session (i.e. logout and login)? Apparently that
> > is necessary from what I noticed.
> 
> not yet, will test later

tested --> changing the currency locale setting to American (default) did not change the described behavior even after restar
 (but did not yet test with the patches included, because kmymoney currently fails building in openSUSE unstable extra repo)
Comment 12 Thomas Baumgart 2025-07-07 19:23:27 UTC
Git commit ff91fdc79a04ba552f5522bce2e0431c70d3b4d4 by Thomas Baumgart.
Committed on 07/07/2025 at 19:23.
Pushed by tbaumgart into branch '5.2'.

Fix a problem with keypad decimal on Qt6
(cherry picked from commit 6ac6d13034d19ba574c3c49193c53495e5119a3c)

M  +1    -1    kmymoney/widgets/amountedit.cpp

https://invent.kde.org/office/kmymoney/-/commit/ff91fdc79a04ba552f5522bce2e0431c70d3b4d4
Comment 13 Thomas Baumgart 2025-07-30 18:52:49 UTC
Update status
Comment 14 Robby Engelmann 2025-07-30 19:40:56 UTC
it appears to be fine now, thanks