Bug 206808 - wrong celsius to fahrenheit conversion
Summary: wrong celsius to fahrenheit conversion
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-09 00:59 UTC by Alessandro Pilleri
Modified: 2009-09-09 08:19 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Alessandro Pilleri 2009-09-09 00:59:02 UTC 
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

The plasmoid system-monitor temperature do wrong conversion from celsius to fahrenheit, in the function Temperature::dataUpdated in file kdebase/workspace/plasma/applets/system-monitor/temperature.cpp

else if (data["units"].toString() == "C" &&
        KGlobal::locale()->measureSystem() != KLocale::Metric) {
        value = (value * (5.0 / 9.0)) + 32;
    }
 
The last line above should be replaced by: value = (value * (9.0 / 5.0)) + 32;

This is just a wish: is it possible to modify the plasmoid to choose to see only the temperature  (just the number without the meter), so one can add the plasmoid to a panel to see directly the temperature instead of the meter that is difficult to understand when it's small...
Comment 1 Petri Damstén 2009-09-09 08:19:17 UTC 
SVN commit 1021408 by pdamsten:

Fix conversion.
BUG:206808

 M  +1 -1      CMakeLists.txt  
 M  +0 -1      applet.cpp  
 M  +6 -2      temperature.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1021408