widget.imap: ensure mailcount is not nil before notifying; closes #400

[etc/awesome.git] / widget / imap.lua

diff --git a/widget/imap.lua b/widget/imap.lua
index 7c13223e9ed5eb54dbd3eb174458abbcd191ad96..7e83d36e46d798debfcfe30dcc69fd19e1c3cfde 100644 (file)
--- a/widget/imap.lua
+++ b/widget/imap.lua
@@ -9,8 +9,7 @@ local helpers  = require("lain.helpers")
 local naughty  = require("naughty")
 local wibox    = require("wibox")
 local awful    = require("awful")
 local naughty  = require("naughty")
 local wibox    = require("wibox")
 local awful    = require("awful")

-local string   = { format = string.format,
-                   gsub   = string.gsub }
+local string   = string

 local type     = type
 local tonumber = tonumber
 
 local type     = type
 local tonumber = tonumber
 


@@ -55,20 +54,17 @@ local function factory(args)
             mail_notification_preset.screen = awful.screen.focused()
         end
 
             mail_notification_preset.screen = awful.screen.focused()
         end
 

-        curl = string.format("%s --url imaps://%s:%s/INBOX -u %s:%q %s -k",
-               head_command, server, port, mail, password, request)
+        local curl = string.format("%s --url imaps://%s:%s/INBOX -u %s:%q %s -k",
+                     head_command, server, port, mail, password, request)

 
         helpers.async(curl, function(f)
 
         helpers.async(curl, function(f)

-            _, mailcount = string.gsub(f, "%d+", "")
+            mailcount = tonumber(f:match("UNSEEN (%d+)"))

             widget = imap.widget
             settings()
 
             widget = imap.widget
             settings()
 

-            if notify == "on" and mailcount >= 1 and mailcount > helpers.get_map(mail) then
-                if mailcount == 1 then
-                    nt = mail .. " has one new message"
-                else
-                    nt = mail .. " has <b>" .. mailcount .. "</b> new messages"
-                end
+            if notify == "on" and mailcount and mailcount >= 1 and mailcount > helpers.get_map(mail) then
+                local nt = mail .. " has <b>" .. mailcount .. "</b> new message"
+                if mailcount >= 1 then nt = nt .. "s" end

                 naughty.notify { preset = mail_notification_preset, text = nt }
             end
 
                 naughty.notify { preset = mail_notification_preset, text = nt }
             end