Bug 501116 - result of calculation is displayed in a strange format
Summary: result of calculation is displayed in a strange format
Status: RESOLVED FIXED
Alias: None
Product: Kalk
Classification: Applications
Component: General (other bugs)
Version First Reported In: 24.12.2
Platform: Flatpak Linux
: NOR normal
Target Milestone: ---
Assignee: hanyoung
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-03-05 20:10 UTC by Marlo heinicke
Modified: 2025-08-06 15:16 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 25.08.0
Sentry Crash Report:


Attachments
Screenshot of the problem (50.52 KB, image/png)
2025-03-05 20:10 UTC, Marlo heinicke
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Marlo heinicke 2025-03-05 20:10:16 UTC
Created attachment 179154 [details]
Screenshot of the problem

I tried to do a simple calculation, like for example 55*0,7 and instead of showing the result (which is 38,5) the app is showing [0 7].
Comment 1 Bug Janitor Service 2025-07-31 23:11:55 UTC
A possibly relevant merge request was started @ https://invent.kde.org/utilities/kalk/-/merge_requests/119
Comment 2 Devin Lin 2025-08-01 00:32:44 UTC
Git commit b452c5a3460b78fe6deff3ff9d95f7ca21838c28 by Devin Lin, on behalf of Albin ZE.
Committed on 31/07/2025 at 23:12.
Pushed by devinlin into branch 'master'.

Fix: Correctly handle locale-specific decimal separators
Related: bug 507525, bug 501190

Fixes a bug where locales that use a comma for decimals, like de_DE, would break calculations. The comma wasn't being converted to a period for the math engine, leading to it being misinterpreted and causing weird results. This now handles the conversion before and after the calculation.

This was originally created as
https://invent.kde.org/utilities/kalk/-/merge_requests/118, but was
updated by @devinlin to expedite getting the fix in due to its severity.

M  +6    -1    src/qalculateengine.cpp

https://invent.kde.org/utilities/kalk/-/commit/b452c5a3460b78fe6deff3ff9d95f7ca21838c28
Comment 3 Devin Lin 2025-08-06 15:16:02 UTC
Git commit 445b382f56ff83261edf70b2f2b14f5ee84cb4f0 by Devin Lin, on behalf of Albin ZE.
Committed on 06/08/2025 at 15:13.
Pushed by devinlin into branch 'release/25.08'.

Fix: Correctly handle locale-specific decimal separators
Related: bug 507525, bug 501190

Fixes a bug where locales that use a comma for decimals, like de_DE, would break calculations. The comma wasn't being converted to a period for the math engine, leading to it being misinterpreted and causing weird results. This now handles the conversion before and after the calculation.

This was originally created as
https://invent.kde.org/utilities/kalk/-/merge_requests/118, but was
updated by @devinlin to expedite getting the fix in due to its severity.

M  +6    -1    src/qalculateengine.cpp

https://invent.kde.org/utilities/kalk/-/commit/445b382f56ff83261edf70b2f2b14f5ee84cb4f0