battery: always update status #199 #203

[etc/awesome.git] / widgets / bat.lua

diff --git a/widgets/bat.lua b/widgets/bat.lua
index e5b5afc153b4e49fe87c46d3baa6754f8eed7974..0da970887357a08fe21edebd0eb5845014d00608 100644 (file)
--- a/widgets/bat.lua
+++ b/widgets/bat.lua
@@ -28,7 +28,7 @@ local function worker(args)
     local timeout   = args.timeout or 30
     local batteries = args.batteries or (args.battery and {args.battery}) or {"BAT0"}
     local ac        = args.ac or "AC0"
-    local notify    = args.notify or true
+    local notify    = args.notify or "on"
     local settings  = args.settings or function() end
 
     bat.widget = wibox.widget.textbox('')
@@ -92,10 +92,7 @@ local function worker(args)
                 local energy_percentage = tonumber(first_line(bstr .. "/capacity")) or
                                           math.floor((energy_now / energy_full) * 100)
 
-                if bat_now.n_status[i] ~= "Charging" and bat_now.n_status[i] ~= "Discharging"
-                then
-                    bat_now.n_status[i] = first_line(bstr .. "/status") or "N/A"
-                end
+                bat_now.n_status[i] = first_line(bstr .. "/status") or "N/A"
 
                 sum_rate_current      = sum_rate_current + (rate_current or 0)
                 sum_rate_voltage      = sum_rate_voltage + rate_voltage
@@ -109,8 +106,8 @@ local function worker(args)
         bat_now.status = bat_now.n_status[1]
         bat_now.ac_status = first_line(string.format("/sys/class/power_supply/%s/online", ac)) or "N/A"
 
-        -- update {perc,time,watt} iff rate > 0 and battery not full
-        if (sum_rate_current > 0 or sum_rate_power > 0) and not (bat_now.status == "Full")
+        -- update {perc,time,watt} iff battery not full and rate > 0
+        if bat_now.status ~= "Full" and (sum_rate_current > 0 or sum_rate_power > 0)
         then
             local rate_time = 0
 
@@ -124,26 +121,25 @@ local function worker(args)
             local minutes = math.floor((rate_time - hours) * 60)
             local watt    = sum_rate_power / 1e6
 
-            bat_now.perc  = string.format("%d", math.min(100, sum_energy_percentage / #batteries))
+            bat_now.perc  = tonumber(string.format("%d", math.min(100, sum_energy_percentage / #batteries)))
             bat_now.time  = string.format("%02d:%02d", hours, minutes)
-            bat_now.watt  = string.format("%.2fW", watt)
+            bat_now.watt  = tonumber(string.format("%.2fW", watt))
         end
 
         widget = bat.widget
         settings()
 
         -- notifications for low and critical states
-        if notify and bat_now.perc and bat_now.status == "Discharging" then
-            local nperc = tonumber(bat_now.perc) or 100
-            if nperc <= 5 then
+        if notify == "on" and bat_now.perc and bat_now.status == "Discharging" then
+            if perc <= 5 then
                 bat.id = naughty.notify({
                     preset = bat_notification_critical_preset,
-                    replaces_id = bat.id,
+                    replaces_id = bat.id
                 }).id
-            elseif nperc <= 15 then
+            elseif perc <= 15 then
                 bat.id = naughty.notify({
                     preset = bat_notification_low_preset,
-                    replaces_id = bat.id,
+                    replaces_id = bat.id
                 }).id
             end
         end