fixing small typo in fs widget which lead to nil reference

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

diff --git a/widgets/bat.lua b/widgets/bat.lua
index 48a14fb844f719cd83a57aa2ba35bf378387a528..626239b14c28d2b32b6f1a9fc29d739e9cff5543 100644 (file)
--- a/widgets/bat.lua
+++ b/widgets/bat.lua
@@ -32,6 +32,22 @@ local function worker(args)
 
     bat.widget = wibox.widget.textbox('')
 
 
     bat.widget = wibox.widget.textbox('')
 

+    bat_notification_low_preset = {
+        title = "Battery low",
+        text = "Plug the cable!",
+        timeout = 15,
+        fg = "#202020",
+        bg = "#CDCDCD"
+    }
+
+    bat_notification_critical_preset = {
+        title = "Battery exhausted",
+        text = "Shutdown imminent",
+        timeout = 15,
+        fg = "#000000",
+        bg = "#FFFFFF"
+    }
+

     function update()
         bat_now = {
             status = "Not present",
     function update()
         bat_now = {
             status = "Not present",


@@ -59,7 +75,7 @@ local function worker(args)
 
             bat_now.status = first_line(bstr .. "/status") or "N/A"
 
 
             bat_now.status = first_line(bstr .. "/status") or "N/A"
 

-            rate  = tonumber(rate)
+            rate  = tonumber(rate) or 1

             ratev = tonumber(ratev)
             rem   = tonumber(rem)
             tot   = tonumber(tot)
             ratev = tonumber(ratev)
             rem   = tonumber(rem)
             tot   = tonumber(tot)


@@ -81,13 +97,17 @@ local function worker(args)
 
             bat_now.time = string.format("%02d:%02d", hrs, min)
 
 
             bat_now.time = string.format("%02d:%02d", hrs, min)
 

-            local perc = (rem / tot) * 100
-            if perc <= 100 then
-                bat_now.perc = string.format("%d", perc)
-            elseif perc > 100 then
-                bat_now.perc = "100"
-            elseif perc < 0 then
-                bat_now.perc = "0"
+            bat_now.perc = first_line(bstr .. "/capacity")
+
+            if not bat_now.perc then
+                local perc = (rem / tot) * 100
+                if perc <= 100 then
+                    bat_now.perc = string.format("%d", perc)
+                elseif perc > 100 then
+                    bat_now.perc = "100"
+                elseif perc < 0 then
+                    bat_now.perc = "0"
+                end

             end
 
             if rate ~= nil and ratev ~= nil then
             end
 
             if rate ~= nil and ratev ~= nil then


@@ -96,42 +116,32 @@ local function worker(args)
                 bat_now.watt = "N/A"
             end
 
                 bat_now.watt = "N/A"
             end
 

-            -- notifications for low and critical states
-            if bat_now.status == "Discharging" and notify == "on"
-            then
-                if tonumber(bat_now.perc) <= 5
-                then
-                    bat.id = naughty.notify({
-                        text = "shutdown imminent",
-                        title = "battery nearly exhausted",
-                        position = "top_right",
-                        timeout = 15,
-                        fg="#000000",
-                        bg="#ffffff",
-                        ontop = true,
-                        replaces_id = bat.id
-                    }).id
-                elseif tonumber(bat_now.perc) <= 15
-                then
-                    bat.id = naughty.notify({
-                        text = "plug the cable",
-                        title = "battery low",
-                        position = "top_right",
-                        timeout = 15,
-                        fg="#202020",
-                        bg="#cdcdcd",
-                        ontop = true,
-                        replaces_id = bat.id
-                    }).id
-                end
-            end

         end
 
         widget = bat.widget
         settings()
         end
 
         widget = bat.widget
         settings()

+
+        -- notifications for low and critical states
+        if bat_now.status == "Discharging" and notify == "on" and bat_now.perc ~= nil
+        then
+            local nperc = tonumber(bat_now.perc) or 100
+            if nperc <= 5
+            then
+                bat.id = naughty.notify({
+                    preset = bat_notification_critical_preset,
+                    replaces_id = bat.id,
+                }).id
+            elseif nperc <= 15
+            then
+                bat.id = naughty.notify({
+                    preset = bat_notification_low_preset,
+                    replaces_id = bat.id,
+                }).id
+            end
+        end

     end
 
     end
 

-    newtimer("bat", timeout, update)
+    newtimer(battery, timeout, update)

 
     return bat.widget
 end
 
     return bat.widget
 end