Bug 249357

Summary: KRunner handles inconsistently decimal points in maths' operations
Product: [Plasma] krunner Reporter: Wojciech Ryrych <rrh>
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: major CC: agostinelli, m.wege, mat69
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: this works for me

Description Wojciech Ryrych 2010-08-28 22:23:51 UTC 
Version:           unspecified (using KDE 4.5.0) 
OS:                Linux

KRunner requires from me to use dot for decimal point in mathematical operations but displays result with comma point (in fractions).
I think the problem is with localization. In Poland decimal point is comma so result is displayed correctly.

Reproducible: Always

Steps to Reproduce:
1. Alt + F2
2. type =0.1+0.1

Actual Results:  
0,1 (comma point)

Expected Results:  
you should be able to type =0,1+0,1 and decimal point as dot should not be allowed

I think also other countries use comma decimal point. Calculator plugin for KRunner should respect global localization rules.
Comment 1 Christoph Feck 2010-08-29 13:17:46 UTC 
The problem isn't in the calculator runner, because that correctly handles conversion from/to locale character.

But it is the ";" handling that allows multiple expressions to be entered, and this incorrectly allows the "," character.
Comment 2 Wojciech Ryrych 2010-09-21 18:00:11 UTC 
So it won't be fixed?
Comment 3 Matthias Fuchs 2010-12-05 20:39:26 UTC 
*** Bug 242849 has been marked as a duplicate of this bug. ***
Comment 4 Matthias Fuchs 2010-12-05 20:50:21 UTC 
Created attachment 54180 [details]
this works for me

As it appears to me the problem is the qalculate engine _not_ correctly handling a different set of decimal symbols.

If I also replace the decimal symbol with "." when using the qalculate engine it works perfectly for me.
Comment 5 m.wege 2010-12-06 12:22:12 UTC 
Kcalc handels this correctly. May be this could help.
Comment 6 Matteo Agostinelli 2010-12-06 14:36:41 UTC 
Fixed in r1204235