Comment 10 for bug 104794

Revision history for this message
Scott Kitterman (kitterman) wrote :

Since this bug has been sitting "Fix Committed" for some time before Feisty was released, I decided to try and confirm if it made it into Feisty or not. Based on the comments above, the code I expected to find was:

kde-guidance-0.8.0/powermanager/powermanage.py
 @@ -184,7 +184,10 @@
              if properties["battery.rechargeable.is_charging"]:
                  state = "charging"
              elif properties["battery.rechargeable.is_discharging"]:
                  if self.onBattery():
                      state = "discharging"
                  else:
                      state = "charged"
              elif not properties["battery.rechargeable.is_discharging"] \
                   and not properties["battery.rechargeable.is_charging"]:
                  if current == 0:

What I found was:

            if properties["battery.rechargeable.is_charging"]:
                state = "charging"
            elif properties["battery.rechargeable.is_discharging"]:
                state = "discharging"
            elif not properties["battery.rechargeable.is_discharging"] \
                 and not properties["battery.rechargeable.is_charging"]:
                if current == 0:
                    state = "empty"
                else:
                    state = "charged"
            else:
                print "Unknown battery state ..."

So it looks to me like this section of the code was modified in a way not documented by this bug. It appears to my that the modified code would also resolve this issue and I can not reproduce the problem. I believe that this bug should be marked fix released, but would appreciate it if the original reporter would verify that the current Feisty version does not have this problem for them.