Bug 498613

Summary:	Application-wide Currency Formatting Issue in Multiple Distros and Package Versions
Product:	[Applications] skrooge	Reporter:	kdebugreport.neatly850
Component:	general	Assignee:	Stephane MANKOWSKI <stephane>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kdebugreport.neatly850
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	25.1.0
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/office/skrooge/-/commit/1d59a1d7318251c65506931b542e551d8a9ff97e	Version Fixed In:
Sentry Crash Report:
Attachments:	Screenshot from 25.1.0 (identical to what I've seen with 2.32.0) KDE settings

Description kdebugreport.neatly850 2025-01-13 15:52:51 UTC

Created attachment 177322 [details]
Screenshot from 25.1.0 (identical to what I've seen with 2.32.0)

SUMMARY
Currency formatting for USD is incorrect, despite en_US.UTF-8 for a locale setting.   This issue involves all fields (e.g., amount, to be checked, checked, etc.) on all pages (e.g., dashboard, accounts, transactions, reports, tables/graphs, etc.).  Per the additional link below, it is impacting several different distributions.  As a result, it is very difficult to read figures at a glance and is a constant and significant distraction.  That said, thank you for creating and supporting this application.

STEPS TO REPRODUCE
1.  Create an account and/or transaction using USD or simply using USD as the file's primary currency (see attached file from 25.1.0 announcement).  I use 2.32, but see my later comment under Software/OS. 
2.  Look at any field (e.g., amount, to be checked, checked, etc.) on all pages (e.g., dashboard, accounts, transactions, reports, tables/graphs, etc.)

OBSERVED RESULT
Currency formatting appears 1000.00$

EXPECTED RESULT
$1,000.00 
(currency symbol at the front, a comma for the thousands separator, and a period for the fractional separator)

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora 41 (GNOME - my regular setup and KDE - another user), Ubuntu 24.04 GNOME (tested in a VM by another user using Flatpak) and Debian Stable 12 GNOME (tested by me in a VM with and without a Flatpak).  Whatever system was used for the 25.1.0 official announcement and its screenshot.
KDE Frameworks Version: 5 (as tested) and 6 (announcement screenshot)
Qt Version: 5 (as tested) and 6 (announcement screenshot)

Judging from the screenshot in the "This Week in KDE Apps", this issue is also impacting the latest release 25.1.0 under Qt6/Framework6.  Look row about the edit box with:  total balance, balance, to be checked, and checked fields
https://blogs.kde.org/2025/01/12/this-week-in-kde-apps-usability-improvements-new-features-and-updated-apps/#skrooge

ADDITIONAL INFORMATION
This issue was replicated by another user.  See this discussion topic in the KDE forum.  
https://discuss.kde.org/t/skrooge-application-wide-currency-formatting-issue/28202/

Comment 1 kdebugreport.neatly850 2025-01-13 15:54:09 UTC

Thank you again for creating this application.  As a new user, I can already see it being tremendously helpful.

Comment 2 Stephane MANKOWSKI 2025-01-13 19:42:16 UTC

Created attachment 177327 [details]
KDE settings

The amounts formatting in Skrooge is base on KDE settings.
Could you check them and modify then if needed ?

For information, I'm french and for me the formatting is correct.

Regards.

Comment 3 kdebugreport.neatly850 2025-01-14 20:52:53 UTC

Hi Stephane - Thank you for your quick response, and I apologize for my delay.  Normally, I run Fedora 41 GNOME (and Debian 12 GNOME) on Qubes as VMs.  To remove some potential confounding issues, I setup another VM (Debian 12 KDE) and observed the same issue as filed in this bug report.  My KDE currency settings were correct:  $24.00 (Standard format for American English).  

Then, to be certain, I decided to boot off a newly created Fedora 41 KDE USB stick, so no VMs (or Qubes OS) were involved.  After doing a dnf update, I then installed skrooge (from the Fedora repo; ver. 2.32.0).  The same issue was observed and the KDE currency settings were correct.  Let me know if you need other information.  Thanks again.

Comment 4 Stephane MANKOWSKI 2025-02-15 15:23:42 UTC

Git commit 1d59a1d7318251c65506931b542e551d8a9ff97e by Stéphane MANKOWSKI.
Committed on 15/02/2025 at 15:23.
Pushed by smankowski into branch 'master'.

Application-wide Currency Formatting Issue in Multiple Distros and Package Versions

M  +1    -0    CHANGELOG
M  +3    -1    skgbasemodeler/skgservices.cpp
M  +3    -0    tests/skgbasemodelertest/skgtestbase.cpp

https://invent.kde.org/office/skrooge/-/commit/1d59a1d7318251c65506931b542e551d8a9ff97e