fix #288

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

diff --git a/widgets/base.lua b/widgets/base.lua
index 0431d03a8891f15696ff674aa47e501cf7a86b2a..9d7b71097326fba0aaa99218bb54350b4c74c0a4 100644 (file)
--- a/widgets/base.lua
+++ b/widgets/base.lua
@@ -6,32 +6,34 @@
                                                   
 --]]
 
                                                   
 --]]
 

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

 local wibox        = require("wibox")
 local wibox        = require("wibox")

-
-local io           = io

 local setmetatable = setmetatable
 
 local setmetatable = setmetatable
 

--- Basic template for simple widgets 
+-- Basic template for custom widgets

 -- lain.widgets.base
 -- lain.widgets.base

-local base = {}

 
 local function worker(args)
 
 local function worker(args)

-    local args     = args or {}
-    local timeout  = args.timeout or 5
-    local cmd      = args.cmd or ""
-    local settings = args.settings or function() end
-
-    base.widget = wibox.widget.textbox('')
-
-    function update()
-        output = io.popen(cmd):read("*all")
-        widget = base.widget
-        settings()
+    local base      = helpers.make_widget_textbox()
+    local args      = args or {}
+    local timeout   = args.timeout or 5
+    local nostart   = args.nostart or false
+    local stoppable = args.stoppable or false
+    local cmd       = args.cmd or ""
+    local settings  = args.settings or function() end
+
+    function base.update()
+        output = helpers.read_pipe(cmd)
+        if output ~= base.prev then
+            widget = base.widget
+            settings()
+            base.prev = output
+        end

     end
 
     end
 

-    newtimer(cmd, timeout, update)
-    return base.widget
+    base.timer = helpers.newtimer(cmd, timeout, base.update, nostart, stoppable)
+
+    return base

 end
 
 end
 

-return setmetatable(base, { __call = function(_, ...) return worker(...) end })
+return setmetatable({}, { __call = function(_, ...) return worker(...) end })