From: Simon Désaulniers <sim.desaulniers@gmail.com>
Date: Wed, 11 Jan 2017 05:15:23 +0000 (-0500)
Subject: bat: find battery status when multiple batteries
X-Git-Url: https://git.madduck.net/etc/awesome.git/commitdiff_plain/ae086776bf94a932719899ec828e002f67ebd098?ds=sidebyside;hp=--cc

bat: find battery status when multiple batteries

Battery status was previously assumed to the first battery's status. This is
wrong as it doesn't support the case where the first battery is full and the
second is discharging, therefor showing battery percentage to 100% when the
actual battery percentage is lower.
---

ae086776bf94a932719899ec828e002f67ebd098
diff --git a/widgets/bat.lua b/widgets/bat.lua
index 8e483f7..38104b5 100644
--- a/widgets/bat.lua
+++ b/widgets/bat.lua
@@ -108,7 +108,16 @@ local function worker(args)
             end
         end
 
+        -- When one of the battery is charging, others' status are either
+        -- "Full", "Unknown" or "Charging". When the laptop is not plugged in,
+        -- one or more of the batteries may be full, but only one battery
+        -- discharging suffices to set global status to "Discharging".
         bat_now.status = bat_now.n_status[1]
+        for _,status in ipairs(bat_now.n_status) do
+            if status == "Discharging" or status == "Charging" then
+                bat_now.status = status
+            end
+        end
         bat_now.ac_status = tonumber(first_line(string.format("%s%s/online", pspath, ac))) or "N/A"
 
         if bat_now.status ~= "N/A" then