]> git.madduck.net Git - etc/awesome.git/blobdiff - widgets/maildir.lua

madduck's git repository

Every one of the projects in this repository is available at the canonical URL git://git.madduck.net/madduck/pub/<projectpath> — see each project's metadata for the exact URL.

All patches and comments are welcome. Please squash your changes to logical commits before using git-format-patch and git-send-email to patches@git.madduck.net. If you'd read over the Git project's submission guidelines and adhered to them, I'd be especially grateful.

SSH access, as well as push access can be individually arranged.

If you use my repositories frequently, consider adding the following snippet to ~/.gitconfig and using the third clone URL listed for each project:

[url "git://git.madduck.net/madduck/"]
  insteadOf = madduck:

#250 missing file patch
[etc/awesome.git] / widgets / maildir.lua
index 24766018c76a03b4541f3fad138aae74fe30ea67..1f4cd455a2fe1dc413c2a3ea2b87efe0bab86158 100644 (file)
@@ -7,24 +7,16 @@
                                                   
 --]]
 
                                                   
 --]]
 
-local newtimer        = require("lain.helpers").newtimer
-local read_pipe       = require("lain.helpers").read_pipe
-local spairs          = require("lain.helpers").spairs
-
-local wibox           = require("wibox")
-
-local awful           = require("awful")
-local util            = require("lain.util")
-
-local io              = { popen  = io.popen }
-local os              = { getenv = os.getenv }
-local pairs           = pairs
-local string          = { len    = string.len,
-                          match  = string.match }
-
-local setmetatable    = setmetatable
-
--- Maildir check
+local awful        = require("awful")
+local wibox        = require("wibox")
+local helpers      = require("lain.helpers")
+local io           = { popen  = io.popen }
+local os           = { getenv = os.getenv }
+local string       = { format = string.format,
+                       match  = string.match }
+local setmetatable = setmetatable
+
+-- Maildir check (synchronous)
 -- lain.widgets.maildir
 local maildir = {}
 
 -- lain.widgets.maildir
 local maildir = {}
 
@@ -36,59 +28,46 @@ local function worker(args)
     local settings     = args.settings or function() end
     local ext_mail_cmd = args.external_mail_cmd
 
     local settings     = args.settings or function() end
     local ext_mail_cmd = args.external_mail_cmd
 
-    maildir.widget = wibox.widget.textbox('')
+    maildir.widget = wibox.widget.textbox()
+
+    function maildir.update()
+        if ext_mail_cmd then awful.spawn(ext_mail_cmd) end
 
 
-    function update()
-        if ext_mail_cmd ~= nil
-        then
-            awful.util.spawn(ext_mail_cmd)
-        end
         -- Find pathes to mailboxes.
         -- Find pathes to mailboxes.
-        local p = io.popen("find " .. mailpath ..
-                           " -mindepth 1 -maxdepth 2 -type d" ..
-                           " -not -name .git")
+        local p = io.popen(string.format("find %s -mindepth 1 -maxdepth 2 -type d -not -name .git", mailpath))
         local boxes = {}
         repeat
             line = p:read("*l")
         local boxes = {}
         repeat
             line = p:read("*l")
-            if line ~= nil
-            then
+            if line then
                 -- Find all files in the "new" subdirectory. For each
                 -- file, print a single character (no newline). Don't
                 -- match files that begin with a dot.
                 -- Afterwards the length of this string is the number of
                 -- new mails in that box.
                 -- Find all files in the "new" subdirectory. For each
                 -- file, print a single character (no newline). Don't
                 -- match files that begin with a dot.
                 -- Afterwards the length of this string is the number of
                 -- new mails in that box.
-                local mailstring = read_pipe("find " .. line ..
-                                    "/new -mindepth 1 -type f " ..
-                                    "-not -name '.*' -printf a")
+                local mailstring = helpers.read_pipe(string.format("find %s /new -mindepth 1 -type f -not -name '.*' -printf a", line))
 
                 -- Strip off leading mailpath.
 
                 -- Strip off leading mailpath.
-                local box = string.match(line, mailpath .. "/(.*)")
-                local nummails = string.len(mailstring)
-                if nummails > 0
-                then
+                local box      = string.match(line, mailpath .. "/(.*)")
+                local nummails = #mailstring
+
+                if nummails > 0 then
                     boxes[box] = nummails
                 end
             end
                     boxes[box] = nummails
                 end
             end
-        until line == nil
-
+        until not line
         p:close()
 
         p:close()
 
-        newmail = "no mail"
-        -- Count the total number of mails irrespective of where it was found
-        total = 0
+        local newmail = "no mail"
+        local total = 0
 
 
-        for box, number in spairs(boxes)
-        do
+        for box, number in helpers.spairs(boxes) do
             -- Add this box only if it's not to be ignored.
             -- Add this box only if it's not to be ignored.
-            if not util.element_in_table(box, ignore_boxes)
-            then
+            if not helpers.element_in_table(box, ignore_boxes) then
                 total = total + number
                 total = total + number
-                if newmail == "no mail"
-                then
-                    newmail = box .. "(" .. number .. ")"
+                if newmail == "no mail" then
+                    newmail = string.format("%s(%s)", box, number)
                 else
                 else
-                    newmail = newmail .. ", " ..
-                              box .. "(" .. number .. ")"
+                    newmail = string.format("%s, %s(%s)", newmail, box, number)
                 end
             end
         end
                 end
             end
         end
@@ -97,8 +76,9 @@ local function worker(args)
         settings()
     end
 
         settings()
     end
 
-    newtimer(mailpath, timeout, update, true)
-    return maildir.widget
+    maildir.timer = helpers.newtimer(mailpath, timeout, maildir.update, true, true)
+
+    return setmetatable(maildir, { __index = maildir.widget })
 end
 
 return setmetatable(maildir, { __call = function(_, ...) return worker(...) end })
 end
 
 return setmetatable(maildir, { __call = function(_, ...) return worker(...) end })