calendar: fix async hanging notifications; closes #289

[etc/awesome.git] / widgets / contrib / gpmdp.lua

diff --git a/widgets/contrib/gpmdp.lua b/widgets/contrib/gpmdp.lua
index fa514407f61589762fbc03cae64237b7e4dd6f95..37da43d06ad5e32c74f58917299c68cdc8b8be7f 100644 (file)
--- a/widgets/contrib/gpmdp.lua
+++ b/widgets/contrib/gpmdp.lua
@@ -6,31 +6,30 @@
                                                      
 --]]
 
                                                      
 --]]
 

-local helpers = require("lain.helpers")
-local json    = require("lain.util.dkjson")
-local pread   = require("awful.util").pread
-local naughty = require("naughty")
-local wibox   = require("wibox")
-local mouse   = mouse
-local os      = { getenv = os.getenv }
-
+local helpers      = require("lain.helpers")
+local json         = require("lain.util.dkjson")
+local focused      = require("awful.screen").focused
+local pread        = require("awful.util").pread
+local naughty      = require("naughty")
+local wibox        = require("wibox")
+local next         = next
+local os           = { getenv = os.getenv }
+local table        = table

 local setmetatable = setmetatable
 
 -- Google Play Music Desktop infos
 -- lain.widget.contrib.gpmdp
 local setmetatable = setmetatable
 
 -- Google Play Music Desktop infos
 -- lain.widget.contrib.gpmdp

-local gpmdp = {}
+local gpmdp = helpers.make_widget_textbox()

 
 local function worker(args)
     local args          = args or {}
     local timeout       = args.timeout or 2
     local notify        = args.notify or "off"
 
 local function worker(args)
     local args          = args or {}
     local timeout       = args.timeout or 2
     local notify        = args.notify or "off"

-    local followmouse   = args.followmouse or false
+    local followtag     = args.followtag or false

     local file_location = args.file_location or
                           os.getenv("HOME") .. "/.config/Google Play Music Desktop Player/json_store/playback.json"
     local settings      = args.settings or function() end
 
     local file_location = args.file_location or
                           os.getenv("HOME") .. "/.config/Google Play Music Desktop Player/json_store/playback.json"
     local settings      = args.settings or function() end
 

-    gpmdp.widget = wibox.widget.textbox('')
-

     gpmdp_notification_preset = {
         title   = "Now playing",
         timeout = 6
     gpmdp_notification_preset = {
         title   = "Now playing",
         timeout = 6


@@ -39,14 +38,13 @@ local function worker(args)
     helpers.set_map("gpmdp_current", nil)
 
     function gpmdp.update()
     helpers.set_map("gpmdp_current", nil)
 
     function gpmdp.update()

-        file, err = io.open(file_location, "r")
-        if not file
-        then
-            gpm_now = { running = false, playing = false }
+        local filelines = helpers.lines_from(file_location)
+
+        if not next(filelines) then
+            local gpm_now = { running = false, playing = false }

         else
         else

-            dict, pos, err = json.decode(file:read "*a", 1, nil)
-            file:close()
-            gpm_now = {}
+            dict, pos, err = json.decode(table.concat(filelines), 1, nil)
+            local gpm_now = {}

             gpm_now.artist    = dict.song.artist
             gpm_now.album     = dict.song.album
             gpm_now.title     = dict.song.title
             gpm_now.artist    = dict.song.artist
             gpm_now.album     = dict.song.album
             gpm_now.title     = dict.song.title


@@ -54,7 +52,7 @@ local function worker(args)
             gpm_now.playing   = dict.playing
         end
 
             gpm_now.playing   = dict.playing
         end
 

-        if (pread("pidof 'Google Play Music Desktop Player'") ~= '') then
+        if pread("pidof 'Google Play Music Desktop Player'") ~= '' then

             gpm_now.running = true
         else
             gpm_now.running = false
             gpm_now.running = true
         else
             gpm_now.running = false


@@ -64,32 +62,29 @@ local function worker(args)
         widget = gpmdp.widget
         settings()
 
         widget = gpmdp.widget
         settings()
 

-        if gpm_now.playing
-        then
-            if notify == "on" and gpm_now.title ~= helpers.get_map("gpmdp_current")
-            then
+        if gpm_now.playing then
+            if notify == "on" and gpm_now.title ~= helpers.get_map("gpmdp_current") then

                 helpers.set_map("gpmdp_current", gpm_now.title)
                 helpers.set_map("gpmdp_current", gpm_now.title)

-                os.execute("curl " .. gpm_now.cover_url .. " -o /tmp/gpmcover.png")

 
 

-                if followmouse then
-                    gpmdp_notification_preset.screen = mouse.screen
-                end
+                if followtag then gpmdp_notification_preset.screen = focused() end

 
 

-                gpmdp.id = naughty.notify({
-                    preset = gpmdp_notification_preset,
-                    icon = "/tmp/gpmcover.png",
-                    replaces_id = gpmdp.id,
-                }).id
+                helpers.async(string.format("curl %d -o /tmp/gpmcover.png", gpm_now.cover_url),
+                function(f)
+                    gpmdp.id = naughty.notify({
+                        preset = gpmdp_notification_preset,
+                        icon = "/tmp/gpmcover.png",
+                        replaces_id = gpmdp.id
+                    }).id
+                end)

             end
             end

-        elseif not gpm_now.running
-        then
+        elseif not gpm_now.running then

             helpers.set_map("gpmdp_current", nil)
         end
     end
 
             helpers.set_map("gpmdp_current", nil)
         end
     end
 

-    helpers.newtimer("gpmdp", timeout, gpmdp.update)
+    gpmdp.timer = helpers.newtimer("gpmdp", timeout, gpmdp.update, true, true)

 
 

-    return setmetatable(gpmdp, { __index = gpmdp.widget })
+    return gpmdp

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