Fixes for the silly case where a client's borders are bigger than its allotted space

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

diff --git a/widgets/net.lua b/widgets/net.lua
index e086c4b9dd386341a8fc5be31d058f28183d3d90..a578ae4ca9cc405454c4cc57a6b23cd5a9be2a0e 100644 (file)
--- a/widgets/net.lua
+++ b/widgets/net.lua
@@ -13,10 +13,9 @@ local notify_fg    = require("beautiful").fg_focus
 local naughty      = require("naughty")
 local wibox        = require("wibox")
 
 local naughty      = require("naughty")
 local wibox        = require("wibox")
 

-local io           = io
-local tostring     = tostring

 local string       = { format = string.format,
 local string       = { format = string.format,

-                       gsub   = string.gsub }
+                       gsub   = string.gsub,
+                       match  = string.match }

 
 local setmetatable = setmetatable
 
 
 local setmetatable = setmetatable
 


@@ -28,64 +27,70 @@ local net = {
 }
 
 function net.get_device()
 }
 
 function net.get_device()

-    f = io.popen("ip link show | cut -d' ' -f2,9")
-    ws = f:read("*all")
-    f:close()
-    ws = ws:match("%w+: UP")
+    local ws = helpers.read_pipe("ip link show | cut -d' ' -f2,9")
+    ws = ws:match("%w+: UP") or ws:match("ppp%w+: UNKNOWN")

     if ws ~= nil then
     if ws ~= nil then

-        return ws:gsub(": UP", "")
+        return ws:match("(%w+):")

     else
     else

-        return ""
+        return "network off"

     end
 end
 
     end
 end
 

-function worker(args)
+local function worker(args)

     local args = args or {}
     local timeout = args.timeout or 2
     local args = args or {}
     local timeout = args.timeout or 2

-    local iface = args.iface or net.get_device()

     local units = args.units or 1024 --kb
     local units = args.units or 1024 --kb

+    local notify = args.notify or "on"
+    local screen = args.screen or 1

     local settings = args.settings or function() end
 
     local settings = args.settings or function() end
 

-    widget = wibox.widget.textbox('')
+    iface = args.iface or net.get_device()
+
+    net.widget = wibox.widget.textbox('')

 
     helpers.set_map(iface, true)
 
 
     helpers.set_map(iface, true)
 

-    function update() 
-        if iface == "" then iface = net.get_device() end
+    function update()
+        net_now = {}

 
 

-        carrier = helpers.first_line('/sys/class/net/' .. iface ..
+        if iface == "" or string.match(iface, "network off")
+        then
+            iface = net.get_device()
+        end
+
+        net_now.carrier = helpers.first_line('/sys/class/net/' .. iface ..

                                            '/carrier') or "0"
                                            '/carrier') or "0"

-        state = helpers.first_line('/sys/class/net/' .. iface ..
+        net_now.state = helpers.first_line('/sys/class/net/' .. iface ..

                                            '/operstate') or "down"
         local now_t = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/tx_bytes') or 0
         local now_r = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/rx_bytes') or 0
 
                                            '/operstate') or "down"
         local now_t = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/tx_bytes') or 0
         local now_r = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/rx_bytes') or 0
 

-        sent = tostring((now_t - net.last_t) / timeout / units)
-        sent = string.gsub(string.format('%.1f', sent), ",", ".")
+        net_now.sent = (now_t - net.last_t) / timeout / units
+        net_now.sent = string.gsub(string.format('%.1f', net_now.sent), ",", ".")

 
 

-        received = tostring((now_r - net.last_r) / timeout / units)
-        received = string.gsub(string.format('%.1f', received), ",", ".")
+        net_now.received = (now_r - net.last_r) / timeout / units
+        net_now.received = string.gsub(string.format('%.1f', net_now.received), ",", ".")

 
 

+        widget = net.widget

         settings()
 
         net.last_t = now_t
         net.last_r = now_r
 
         settings()
 
         net.last_t = now_t
         net.last_r = now_r
 

-        if carrier ~= "1"
+        if net_now.carrier ~= "1" and notify == "on"

         then
             if helpers.get_map(iface)
             then
         then
             if helpers.get_map(iface)
             then

-                n_title = iface
-                if n_title == "" then n_title = "network" end

                 naughty.notify({
                 naughty.notify({

-                    title    = n_title,
+                    title    = iface,

                     text     = "no carrier",
                     timeout  = 7,
                     position = "top_left",
                     icon     = helpers.icons_dir .. "no_net.png",
                     text     = "no carrier",
                     timeout  = 7,
                     position = "top_left",
                     icon     = helpers.icons_dir .. "no_net.png",

-                    fg       = notify_fg or "#FFFFFF"
+                    fg       = notify_fg or "#FFFFFF",
+                    screen   = screen

                 })
                 helpers.set_map(iface, false)
             end
                 })
                 helpers.set_map(iface, false)
             end


@@ -95,8 +100,7 @@ function worker(args)
     end
 
     helpers.newtimer(iface, timeout, update)
     end
 
     helpers.newtimer(iface, timeout, update)

-
-    return widget
+    return net.widget

 end
 
 return setmetatable(net, { __call = function(_, ...) return worker(...) end })
 end
 
 return setmetatable(net, { __call = function(_, ...) return worker(...) end })