widgets fix

diff --git a/helpers.lua b/helpers.lua
index a8c4cd7297bc01c830c06c4df98e0d5e05b1780a..81910d21b206bc2b151cdb9214b84e8bb80e4b3b 100644 (file)
--- a/helpers.lua
+++ b/helpers.lua
@@ -8,8 +8,10 @@
 --]]
 
 local debug  = require("debug")
 --]]
 
 local debug  = require("debug")

-local rawget = rawget
+
+local capi   = { timer = timer }

 local io     = { open = io.open }
 local io     = { open = io.open }

+local rawget = rawget

 
 -- Lain helper functions for internal use
 -- lain.helpers
 
 -- Lain helper functions for internal use
 -- lain.helpers


@@ -49,7 +51,7 @@ end
 helpers.timer_table = {}
 
 function helpers.newtimer(name, timeout, fun, nostart)
 helpers.timer_table = {}
 
 function helpers.newtimer(name, timeout, fun, nostart)

-    helpers.timer_table[name] = timer({ timeout = timeout })
+    helpers.timer_table[name] = capi.timer({ timeout = timeout })

     helpers.timer_table[name]:connect_signal("timeout", fun)
     helpers.timer_table[name]:start()
     if not nostart then
     helpers.timer_table[name]:connect_signal("timeout", fun)
     helpers.timer_table[name]:start()
     if not nostart then

diff --git a/widgets/alsa.lua b/widgets/alsa.lua
index 887f398cc11670ceccd3a518f1dcb0c6ca07db92..3e025bbd252712f8f152e65c5a6fc783ffe8348c 100644 (file)
--- a/widgets/alsa.lua
+++ b/widgets/alsa.lua
@@ -29,30 +29,31 @@ local function worker(args)
     alsa.widget = wibox.widget.textbox('')
 
     function alsa.update()
     alsa.widget = wibox.widget.textbox('')
 
     function alsa.update()

-        local f = io.popen('amixer get ' .. channel)
+        local f = assert(io.popen('amixer get ' .. channel))

         local mixer = f:read("*all")
         f:close()
 
         local mixer = f:read("*all")
         f:close()
 

-        volume = {}
+        volume_now = {}

 
 

-        volume.level, volume.status = string.match(mixer, "([%d]+)%%.*%[([%l]*)")
+        volume_now.level, volume_now.status = string.match(mixer, "([%d]+)%%.*%[([%l]*)")

 
 

-        if volume.level == nil
+        if volume_now.level == nil

         then
         then

-            volume.level  = 0
-            volume.status = "off"
+            volume_now.level  = "0"
+            volume_now.status = "off"

         end
 
         end
 

-        if volume.status == ""
+        if volume_now.status == ""

         then
         then

-            if volume.level == 0
+            if volume_now.level == "0"

             then
             then

-                volume.status = "off"
+                volume_now.status = "off"

             else
             else

-                volume.status = "on"
+                volume_now.status = "on"

             end
         end
 
             end
         end
 

+        widget = alsa.widget

         settings()
     end
 
         settings()
     end
 

diff --git a/widgets/bat.lua b/widgets/bat.lua
index ba1fa58f0aea99c57444c8c75b810f3ec2fe50aa..09b135fc29bc03463a411471c592b91430f6fa77 100644 (file)
--- a/widgets/bat.lua
+++ b/widgets/bat.lua
@@ -30,9 +30,9 @@ local function worker(args)
 
     bat.widget = wibox.widget.textbox('')
 
 
     bat.widget = wibox.widget.textbox('')
 

-    function bat.update()
+    function update()

         bat_now = {
         bat_now = {

-            status = "not present",
+            status = "Not present",

             perc   = "N/A",
             time   = "N/A",
             watt   = "N/A"
             perc   = "N/A",
             time   = "N/A",
             watt   = "N/A"


@@ -110,7 +110,7 @@ local function worker(args)
         settings()
     end
 
         settings()
     end
 

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

 
     return bat.widget
 end
 
     return bat.widget
 end

diff --git a/widgets/calendar.lua b/widgets/calendar.lua
index e28e735bf2441ebabf1a47957415712181867697..57fadd946d03acff62d8828cbe40c5459ca9f34b 100644 (file)
--- a/widgets/calendar.lua
+++ b/widgets/calendar.lua
@@ -21,12 +21,12 @@ local setmetatable = setmetatable
 -- Calendar notification
 -- lain.widgets.calendar
 local calendar = {}
 -- Calendar notification
 -- lain.widgets.calendar
 local calendar = {}

-local notification = nil
+local cal_notification = nil

 
 function calendar:hide()
 
 function calendar:hide()

-    if notification ~= nil then
-        naughty.destroy(notification)
-        notification = nil
+    if cal_notification ~= nil then
+        naughty.destroy(cal_notification)
+        cal_notification = nil

     end
 end
 
     end
 end
 


@@ -95,12 +95,12 @@ function calendar:show(t_out, inc_offset)
              .. "</span></tt>"
     f:close()
 
              .. "</span></tt>"
     f:close()
 

-    notification = naughty.notify({ text = c_text,
-                                    icon = calendar.notify_icon,
-                                    position = calendar.position,
-                                    fg = calendar.fg,
-                                    bg = calendar.bg,
-                                    timeout = tims })
+    cal_notification = naughty.notify({ text = c_text,
+                                        icon = calendar.notify_icon,
+                                        position = calendar.position,
+                                        fg = calendar.fg,
+                                        bg = calendar.bg,
+                                        timeout = tims })

 end
 
 function calendar:attach(widget, args)
 end
 
 function calendar:attach(widget, args)

diff --git a/widgets/cpu.lua b/widgets/cpu.lua
index f307b26b3bcc021218555ca0f65ef7e800183209..55a3799457271598d60e05847480a23bf149bc49 100644 (file)
--- a/widgets/cpu.lua
+++ b/widgets/cpu.lua
@@ -33,7 +33,7 @@ local function worker(args)
 
     cpu.widget = wibox.widget.textbox('')
 
 
     cpu.widget = wibox.widget.textbox('')
 

-    function cpu.update()
+    function update()

         -- Read the amount of time the CPUs have spent performing
         -- different kinds of work. Read the first line of /proc/stat
         -- which is the sum of all CPUs.
         -- Read the amount of time the CPUs have spent performing
         -- different kinds of work. Read the first line of /proc/stat
         -- which is the sum of all CPUs.


@@ -68,7 +68,7 @@ local function worker(args)
         cpu.last_total = total
     end
 
         cpu.last_total = total
     end
 

-    newtimer("cpu", timeout, cpu.update)
+    newtimer("cpu", timeout, update)

 
     return cpu.widget
 end
 
     return cpu.widget
 end

diff --git a/widgets/fs.lua b/widgets/fs.lua
index b5800103673e7287880bacac9a47c85b305f162c..17037c4009c21535a6e83134f27a6d367c5b357b 100644 (file)
--- a/widgets/fs.lua
+++ b/widgets/fs.lua
@@ -27,7 +27,7 @@ local setmetatable = setmetatable
 local fs = {}
 
 local notification  = nil
 local fs = {}
 
 local notification  = nil

-notification_preset = { fg = beautiful.fg_normal }
+fs_notification_preset = { fg = beautiful.fg_normal }

 
 function fs:hide()
     if notification ~= nil then
 
 function fs:hide()
     if notification ~= nil then


@@ -44,7 +44,7 @@ function fs:show(t_out)
     f:close()
 
     notification = naughty.notify({
     f:close()
 
     notification = naughty.notify({

-        preset = notification_preset,
+        preset = fs_notification_preset,

         text = ws,
        timeout = t_out
     })
         text = ws,
        timeout = t_out
     })


@@ -63,7 +63,7 @@ local function worker(args)
 
     helpers.set_map("fs", false)
 
 
     helpers.set_map("fs", false)
 

-    function fs.update()
+    function update()

         fs_info = {} 
 
         local f = io.popen("LC_ALL=C df -kP")
         fs_info = {} 
 
         local f = io.popen("LC_ALL=C df -kP")


@@ -108,20 +108,12 @@ local function worker(args)
         end
     end
 
         end
     end
 

-    helpers.newtimer(partition, timeout, fs.update)
+    helpers.newtimer(partition, timeout, update)

 
     widget:connect_signal('mouse::enter', function () fs:show(0) end)
     widget:connect_signal('mouse::leave', function () fs:hide() end)
 
 
     widget:connect_signal('mouse::enter', function () fs:show(0) end)
     widget:connect_signal('mouse::leave', function () fs:hide() end)
 

-    output = {
-        widget = fs.widget,
-        show = function(t_out)
-                  fs.update()
-                  fs:show(t_out)
-               end
-    }
-
-    return setmetatable(output, { __index = output.widget })
+    return setmetatable(fs, { __index = fs.widget })

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

diff --git a/widgets/imap.lua b/widgets/imap.lua
index 4df3610db18364fdfb4b5fa63e3a6c3b97249928..6574276221e54de1f138dc1e6389ddf9d781f576 100644 (file)
--- a/widgets/imap.lua
+++ b/widgets/imap.lua
@@ -47,8 +47,8 @@ local function worker(args)
 
     imap.widget = wibox.widget.textbox('')
 
 
     imap.widget = wibox.widget.textbox('')
 

-    function imap.update()
-        notification_preset = {
+    function update()
+        mail_notification_preset = {

             icon     = helpers.icons_dir .. "mail.png",
             position = "top_left"
         }
             icon     = helpers.icons_dir .. "mail.png",
             position = "top_left"
         }


@@ -73,13 +73,13 @@ local function worker(args)
             else
                 nt = mail .. " has <b>" .. mailcount .. "</b> new messages"
             end
             else
                 nt = mail .. " has <b>" .. mailcount .. "</b> new messages"
             end

-            naughty.notify({ preset = notification_preset, text = nt })
+            naughty.notify({ preset = mail_notification_preset, text = nt })

         end
 
         helpers.set_map(mail, mailcount)
     end
 
         end
 
         helpers.set_map(mail, mailcount)
     end
 

-    helpers.newtimer(mail, timeout, imap.update, true)
+    helpers.newtimer(mail, timeout, update, true)

     return imap.widget
 end
 
     return imap.widget
 end
 

diff --git a/widgets/maildir.lua b/widgets/maildir.lua
index 4ac34bc25fece45f199de6a001b025cbff331447..d4608812e7c62e92e30cfced6eae628b747fc393 100644 (file)
--- a/widgets/maildir.lua
+++ b/widgets/maildir.lua
@@ -33,7 +33,7 @@ local function worker(args)
 
     maildir.widget = wibox.widget.textbox('')
 
 
     maildir.widget = wibox.widget.textbox('')
 

-    function maildir.update()
+    function update()

         -- Find pathes to mailboxes.
         local p = io.popen("find " .. mailpath ..
                            " -mindepth 1 -maxdepth 1 -type d" ..
         -- Find pathes to mailboxes.
         local p = io.popen("find " .. mailpath ..
                            " -mindepth 1 -maxdepth 1 -type d" ..


@@ -88,8 +88,7 @@ local function worker(args)
         settings()
     end
 
         settings()
     end
 

-    newtimer(mailpath, timeout, maildir.update, true)
-
+    newtimer(mailpath, timeout, update, true)

     return maildir.widget
 end
 
     return maildir.widget
 end
 

diff --git a/widgets/mem.lua b/widgets/mem.lua
index f0141494247700b4c3ee6ec8295a129fc2b3f743..78ed3e0a4076705e3254de5f9340de491c6806da 100644 (file)
--- a/widgets/mem.lua
+++ b/widgets/mem.lua
@@ -1,12 +1,10 @@
 
 --[[
 
 --[[

-                                                           
-     Licensed under GNU General Public License v2          
-      * (c) 2013,      Luke Bonham                         
-      * (c) 2010-2012, Peter Hofmann                       
-      * (c) 2010,      Adrian C.      <anrxc@sysphere.org> 
-      * (c) 2009,      Lucas de Vries <lucas@glacicle.com> 
-                                                           
+                                                   
+     Licensed under GNU General Public License v2  
+      * (c) 2013,      Luke Bonham                 
+      * (c) 2010-2012, Peter Hofmann               
+                                                   

 --]]
 
 local newtimer        = require("lain.helpers").newtimer
 --]]
 
 local newtimer        = require("lain.helpers").newtimer


@@ -32,30 +30,30 @@ local function worker(args)
 
     mem.widget = wibox.widget.textbox('')
 
 
     mem.widget = wibox.widget.textbox('')
 

-    function mem.update()
-        mem = {}
+    function update()
+        mem_now = {}

         for line in io.lines("/proc/meminfo")
         do
             for k, v in string.gmatch(line, "([%a]+):[%s]+([%d]+).+")
             do
         for line in io.lines("/proc/meminfo")
         do
             for k, v in string.gmatch(line, "([%a]+):[%s]+([%d]+).+")
             do

-                if     k == "MemTotal"  then mem.total = math.floor(v / 1024)
-                elseif k == "MemFree"   then mem.free  = math.floor(v / 1024)
-                elseif k == "Buffers"   then mem.buf   = math.floor(v / 1024)
-                elseif k == "Cached"    then mem.cache = math.floor(v / 1024)
-                elseif k == "SwapTotal" then mem.swap  = math.floor(v / 1024)
-                elseif k == "SwapFree"  then mem.swapf = math.floor(v / 1024)
+                if     k == "MemTotal"  then mem_now.total = math.floor(v / 1024)
+                elseif k == "MemFree"   then mem_now.free  = math.floor(v / 1024)
+                elseif k == "Buffers"   then mem_now.buf   = math.floor(v / 1024)
+                elseif k == "Cached"    then mem_now.cache = math.floor(v / 1024)
+                elseif k == "SwapTotal" then mem_now.swap  = math.floor(v / 1024)
+                elseif k == "SwapFree"  then mem_now.swapf = math.floor(v / 1024)

                 end
             end
         end
 
                 end
             end
         end
 

-        used = mem.total - (mem.free + mem.buf + mem.cache)
-        swapused = mem.swap - mem.swapf
+        mem_now.used = mem_now.total - (mem_now.free + mem_now.buf + mem_now.cache)
+        mem_now.swapused = mem_now.swap - mem_now.swapf

 
         widget = mem.widget
         settings()
     end
 
 
         widget = mem.widget
         settings()
     end
 

-    newtimer("mem", timeout, mem.update)
+    newtimer("mem", timeout, update)

 
     return mem.widget
 end
 
     return mem.widget
 end

diff --git a/widgets/mpd.lua b/widgets/mpd.lua
index 016f2a3a4e5de2a150e7276c810e7507601548ea..cd97b37cb5fee86377b85d2bca9e218ecbddbdc2 100644 (file)
--- a/widgets/mpd.lua
+++ b/widgets/mpd.lua
@@ -40,7 +40,7 @@ local function worker(args)
 
     mpd.widget = wibox.widget.textbox('')
 
 
     mpd.widget = wibox.widget.textbox('')
 

-    notification_preset = {
+    mpd_notification_preset = {

         title   = "Now playing",
         timeout = 6
     }
         title   = "Now playing",
         timeout = 6
     }


@@ -73,8 +73,8 @@ local function worker(args)
 
         f:close()
 
 
         f:close()
 

-        notification_preset.text = string.format("%s (%s) - %s\n%s", mpd_now.artist,
-                                   mpd_now.album, mpd_now.date, mpd_now.title)
+        mpd_notification_preset.text = string.format("%s (%s) - %s\n%s", mpd_now.artist,
+                                       mpd_now.album, mpd_now.date, mpd_now.title)

         widget = mpd.widget
         settings()
 
         widget = mpd.widget
         settings()
 


@@ -87,7 +87,7 @@ local function worker(args)
                 os.execute(string.format("%s %q %q", mpdcover, music_dir, mpd_now.file))
 
                 mpd.id = naughty.notify({
                 os.execute(string.format("%s %q %q", mpdcover, music_dir, mpd_now.file))
 
                 mpd.id = naughty.notify({

-                    preset = notification_preset,
+                    preset = mpd_notification_preset,

                     icon = "/tmp/mpdcover.png",
                     replaces_id = mpd.id
                 }).id
                     icon = "/tmp/mpdcover.png",
                     replaces_id = mpd.id
                 }).id

diff --git a/widgets/net.lua b/widgets/net.lua
index d692df1268976f75a44379cab22bd227e22c6717..b3deca6b471a68bd4e3640b9a4f54fc49874ca37 100644 (file)
--- a/widgets/net.lua
+++ b/widgets/net.lua
@@ -50,23 +50,25 @@ local function worker(args)
 
     helpers.set_map(iface, true)
 
 
     helpers.set_map(iface, true)
 

-    function net.update() 
+    function update() 
+        net_now = {}
+

         if iface == "" then iface = net.get_device() end
 
         if iface == "" then iface = net.get_device() end
 

-        carrier = helpers.first_line('/sys/class/net/' .. iface ..
+        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 = tostring((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 = tostring((now_r - net.last_r) / timeout / units)
+        net_now.received = string.gsub(string.format('%.1f', net_now.received), ",", ".")

 
         widget = net.widget
         settings()
 
         widget = net.widget
         settings()


@@ -74,7 +76,7 @@ local function worker(args)
         net.last_t = now_t
         net.last_r = now_r
 
         net.last_t = now_t
         net.last_r = now_r
 

-        if carrier ~= "1"
+        if net_now.carrier ~= "1"

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


@@ -95,8 +97,7 @@ local function worker(args)
         end
     end
 
         end
     end
 

-    helpers.newtimer(iface, timeout, net.update)
-
+    helpers.newtimer(iface, timeout, update)

     return net.widget
 end
 
     return net.widget
 end
 

diff --git a/widgets/sysload.lua b/widgets/sysload.lua
index 90bdedfbcde5149e82d3c18c3dbd85bf2160ef0d..868398c9f6b593855b67a6f47fd7785f54dd93a8 100644 (file)
--- a/widgets/sysload.lua
+++ b/widgets/sysload.lua
@@ -28,19 +28,18 @@ local function worker(args)
 
     sysload.widget = wibox.widget.textbox('')
 
 
     sysload.widget = wibox.widget.textbox('')
 

-    function sysload.update()
+    function update()

         local f = io.open("/proc/loadavg")
         local ret = f:read("*all")
         f:close()
         
         local f = io.open("/proc/loadavg")
         local ret = f:read("*all")
         f:close()
         

-        a, b, c = string.match(ret, "([^%s]+) ([^%s]+) ([^%s]+)")
+        load_1, load_5, load_15 = string.match(ret, "([^%s]+) ([^%s]+) ([^%s]+)")

 
         widget = sysload.widget
         settings()
     end
 
 
         widget = sysload.widget
         settings()
     end
 

-    newtimer("sysload", timeout, sysload.update)
-
+    newtimer("sysload", timeout, update)

     return sysload.widget
 end
 
     return sysload.widget
 end
 

diff --git a/widgets/temp.lua b/widgets/temp.lua
index e5681389faffcad70a828b17665be2c3bb2ca9e8..b57c477929b1e0315b03b0ba4181671aa4278289 100644 (file)
--- a/widgets/temp.lua
+++ b/widgets/temp.lua
@@ -26,7 +26,7 @@ local function worker(args)
 
     temp.widget = wibox.widget.textbox('')
 
 
     temp.widget = wibox.widget.textbox('')
 

-    function temp.update()
+    function update()

         local f = io.open("/sys/class/thermal/thermal_zone0/temp")
         coretemp_now = tonumber(f:read("*all")) / 1000
         f:close()
         local f = io.open("/sys/class/thermal/thermal_zone0/temp")
         coretemp_now = tonumber(f:read("*all")) / 1000
         f:close()


@@ -34,8 +34,7 @@ local function worker(args)
         settings()
     end
 
         settings()
     end
 

-    newtimer("coretemp", timeout, temp.update)
-
+    newtimer("coretemp", timeout, update)

     return temp.widget
 end
 
     return temp.widget
 end
 

diff --git a/widgets/yawn/init.lua b/widgets/yawn/init.lua
index 4d14966b17e827df82d1963ece8dd5989f3aaa00..4c90df4c440795f7e8e4e59f66f4374776e69078 100644 (file)
--- a/widgets/yawn/init.lua
+++ b/widgets/yawn/init.lua
@@ -43,7 +43,7 @@ local city_id             = nil
 local sky                 = nil
 local settings            = function() end
 
 local sky                 = nil
 local settings            = function() end
 

-notification_preset = {}
+yawn_notification_preset = {}

 
 local function fetch_weather()
     local url = api_url .. units_set .. city_id
 
 local function fetch_weather()
     local url = api_url .. units_set .. city_id


@@ -56,6 +56,7 @@ local function fetch_weather()
     -- widgets won't display
     if text == "" or text:match("City not found")
     then
     -- widgets won't display
     if text == "" or text:match("City not found")
     then

+        yawn.icon:set_image(icon_path .. "na.png")

         if text == "" then
             weather_data = "Service not available at the moment."
             yawn.widget:set_text("N/A")
         if text == "" then
             weather_data = "Service not available at the moment."
             yawn.widget:set_text("N/A")


@@ -152,7 +153,7 @@ function yawn.show(t_out)
     yawn.hide()
 
     notification = naughty.notify({
     yawn.hide()
 
     notification = naughty.notify({

-        preset = notification_preset,
+        preset = yawn_notification_preset,

         text = weather_data,
         icon = sky,
         timeout = t_out
         text = weather_data,
         icon = sky,
         timeout = t_out