plugin(lart): refactor to use registry to persist

[user/henk/code/ruby/rbot.git] / data / rbot / plugins / time.rb

diff --git a/data/rbot/plugins/time.rb b/data/rbot/plugins/time.rb
index 854e403de9fa3ab7570abe8bb30ea4daa63cb169..5c3189f60ad2d6b0164c215d036d4f22b33e1e97 100644 (file)
--- a/data/rbot/plugins/time.rb
+++ b/data/rbot/plugins/time.rb
@@ -66,26 +66,41 @@ class TimePlugin < Plugin
   end
 
   def showTime(m, params)
-    zone = params[:where].join('_')
-    if params[:where].size > 0 then
-      begin
-        m.reply getTime( m,  zone )
-      rescue TZInfo::InvalidTimezoneIdentifier
-        if @registry.has_key?( zone ) then
-          zone =  @registry[ zone ]
-          m.reply getTime( m,  zone )
-        else
-          parse(m, params)
-        end
-      end
+    nick = nil
+    zone = nil
+    speaker = false
+    case params[:where].size
+    when 0
+      nick = m.sourcenick
+      speaker = true
+    when 1
+      zone = params[:where].first
+      nick = m.channel.get_user(zone)
+      speaker = (nick == m.sourcenick)
     else
-      if @registry.has_key?( m.sourcenick) then
-        zone = @registry[ m.sourcenick ]
-        m.reply "#{m.sourcenick}: #{getTime( m,  zone )}"
+      zone = params[:where].join('_')
+    end
+
+    # now we have a non-nil nick iff we want user information
+    if nick
+      if @registry.has_key? nick
+        zone = @registry[nick]
       else
-        m.reply "#{m.sourcenick}: use time set <Continent>/<City> to set your time zone."
+        if speaker
+          msg = _("I don't know where you are, use %{pfx}time set <Continent>/<City> to let me know")
+        else
+          msg = _("I don't know where %{nick} is, (s)he should use %{pfx}time set <Continent>/<City> to let me know")
+        end
+        m.reply(msg % { :pfx => @bot.config['core.address_prefix'].first, :nick => nick })
+        return false
       end
     end
+
+    begin
+      m.reply getTime( m,  zone )
+    rescue TZInfo::InvalidTimezoneIdentifier
+      parse(m, params)
+    end
   end
 
   def setUserZone( m, params )
@@ -159,8 +174,12 @@ class TimePlugin < Plugin
       offset = (time - now).abs
       raise if offset < 0.1
     rescue => e
-      m.reply _("unintelligible time")
-      return
+      if str.match(/^\d+$/)
+        time = Time.at(str.to_i)
+      else
+        m.reply _("unintelligible time")
+        return
+      end
     end
 
     if zone = @registry[m.sourcenick]
@@ -177,8 +196,9 @@ class TimePlugin < Plugin
   def on_timezone(to_zone)
     original_zone = ENV["TZ"]
     ENV["TZ"] = to_zone
-    return yield
+    ret = yield
     ENV["TZ"] = original_zone
+    return ret
   end
 end