Bug 408752 - Comma separator in stored constant returns nan
Summary: Comma separator in stored constant returns nan
Status: CONFIRMED
Alias: None
Product: kcalc
Classification: Applications
Component: general (show other bugs)
Version: 18.12
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Gabriel Barrantes
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-15 19:51 UTC by Dan
Modified: 2024-06-19 13:14 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dan 2019-06-15 19:51:47 UTC
SUMMARY
Comma separator in stored constant returns nan

STEPS TO REPRODUCE
1. In Settings select Science Mode
2. In Settings turn on Constants Buttons
3. With a number greater than one thousand in the display field (with comma separators) select the Shift button. The C1-C6 buttons will all change to Store
4. Select one of the Store buttons
5. Clear the display with AC button
6. Hover over the button the number was stored in (C1-C6). The stored number  should be displayed as a tool tip
7. Select the button the number was stored in

OBSERVED RESULT
The display field shows nan

EXPECTED RESULT
The stored number

SOFTWARE/OS VERSIONS
Linux/KDE Plasma:
KDE Plasma Version: 5.15.5
KDE Frameworks Version: 5.58.0
Qt Version: 5.12.1

ADDITIONAL INFORMATION
In Settings if you select Configure Kcalc...
you can edit the entry to remove the commas and it will work.
Comment 1 Christoph Feck 2019-07-03 13:00:32 UTC
Confirmed also in current master version.
Comment 2 Joseph-Eugene Winzer 2019-07-04 15:16:37 UTC
Submitted a fix to https://phabricator.kde.org/D22263
Comment 3 Christoph Feck 2019-07-05 01:55:58 UTC
Oh, I already posted https://phabricator.kde.org/D22239 yesterday...

Please set the reviewers, and let them decide which approach is better.
Comment 4 Joseph-Eugene Winzer 2019-07-05 13:42:02 UTC
(In reply to Christoph Feck from comment #3)
> Oh, I already posted https://phabricator.kde.org/D22239 yesterday...
> 
> Please set the reviewers, and let them decide which approach is better.

I'm sorry. I tried to look if there's a patch for it already but I'm not really familiar with the Phabricator interface so I must have missed it :/

I set the same reviewers as you did now. Thanks for the info.