Disregard non-powersupply batteries for powerdevil

Review Request #110982 - Created June 12, 2013 and submitted

Information
Kai Uwe Broulik
kde-workspace
master
Reviewers
solid
Don't add non-powersupply batteries to the battery list.
I have seen a few peripheral device (in my case Apple Magic Mouse) that report themselves as primary battery errorneously but correctly set the power supply property. PowerDevil doesn't know of this and adds them to the list of batteries, resulting in false battery warnings, eg. your notebook battery is at 40% and your mouse almost empty it will trigger a "Your battery is low" notification.

In the future would of course be cool to handle those separately so we can notify "Your mouse is running low". But for now I guess this suffices, and so the notifications are consistent with what the new battery monitor tells.
Nope, compiles. Looks safe though.
Dario Freddi
Kai Uwe Broulik
Review request changed

Status: Closed (submitted)

Change Summary:

http://quickgit.kde.org/?p=kde-workspace.git&a=commit&h=282ad253df4c7a7f3f57a8e72b3a83c6d39562d6
Loading...