[Bug 253874] New: powerdevil daemon hibernate (do action) when battery is critical and AC addapter is pluged in

[Lukáš Karas]

https://bugs.kde.org/show_bug.cgi?id=253874

           Summary: powerdevil daemon hibernate (do action) when battery
                    is critical and AC addapter is pluged in
           Product: kde
           Version: 4.5
          Platform: Ubuntu Packages
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: minor
          Priority: NOR
         Component: general
        AssignedTo: unassigned-bugs at kde.org
        ReportedBy: lukas.karas at centrum.cz


Version:           4.5 (using KDE 4.5.1) 
OS:                Linux

I have set critical level of battery to 5%, when this level is reached,
powerdevil hibernate my system - it is ok. But, when I plug charger in and
switch notebook on, powedevil hibernate it again after awakening (battery is
about 3% at this moment). I have to wait moment before switch on notebook. I
can awake my system as far as battery level is higher than 5%.

Reproducible: Always

Steps to Reproduce:
Set action when battery is critical to hibernate, discharge battery (about 5%)
and after that set critical level high (about 40%). After hibernate plug
charger and awake system. When system is awakening and battery is still lower
than 40%, powerdevil hibernate it again.


Expected Results:  
powerdevil never (no automaticaly) hibernate (shut down or suspend) system when
charger is pluged in

patch with fix should looks like:

Index: PowerDevilDaemon.cpp
===================================================================
--- PowerDevilDaemon.cpp        (revision 1184888)
+++ PowerDevilDaemon.cpp        (working copy)
@@ -501,7 +501,9 @@
         return;
     }

-    if (charge <= PowerDevilSettings::batteryCriticalLevel()) {
+    if (charge <= PowerDevilSettings::batteryCriticalLevel() &&
+               isChargerUnpluged()
+       ) {
         switch (PowerDevilSettings::batLowAction()) {
         case Shutdown:
             if (PowerDevilSettings::waitBeforeSuspending()) {

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.