Bug 240175 - Powerdevil suspend notification shows seconds when value is minutes
Summary: Powerdevil suspend notification shows seconds when value is minutes
Status: RESOLVED DUPLICATE of bug 233103
Alias: None
Product: solid
Classification: Frameworks and Libraries
Component: powermanagement-daemon (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR minor
Target Milestone: ---
Assignee: Dario Freddi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-31 07:52 UTC by Mike Bridge
Modified: 2010-10-02 12:51 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Bridge 2010-05-31 07:52:59 UTC
Version:           unspecified (using KDE 4.4.3) 
OS:                Linux

"Your battery level is critical, the computer will "
"be halted in %1 seconds.",

Powerdevil is set to wait 3 min prior to suspension. Notification should either reflect 3 minutes or calculate and display 90 seconds. Currently, It displays 3 seconds to suspend and almost triggered a heart attack in this user. 

Reproducible: Always

Steps to Reproduce:
Run battery down to powerdevil limits and notification is triggered using wrong value.


Expected Results:  
If the value is minutes, the system should either display x minutes, or calc the correct number of seconds.

I'm not smart enough to do the fixing myself, but I suspect the fixing would be somewhere around - /trunk/KDE/kdebase/workspace/powerdevil/daemon/PowerDevilDaemon.cpp. About line 500, perhaps?
Comment 1 Aaron J. Seigo 2010-06-01 00:48:41 UTC
wrong component; but it's all seconds here in trunk. re-assigning in any case.
Comment 2 Anselmo L. S. Melo (anselmolsm) 2010-07-04 16:45:47 UTC
This issue was already reported in bug 233103, later fixed in commit 1114072 (http://websvn.kde.org/?view=rev&revision=1114072).

You cannot see the fix in 4.4.3 because it seems the commit was not backported (at least I couldn't find it in branch 4.4), but it is already available in 4.5rc1.

Thanks for reporting anyway =)

*** This bug has been marked as a duplicate of bug 233103 ***