just some tiny tweaks. giuseppe! you have been busy! :)

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

diff --git a/data/rbot/plugins/autoop.rb b/data/rbot/plugins/autoop.rb
index 9da5bd379d4224f076375fec9cab4f9310da9435..cc1e11a9a16ce887681bde799a26ea2732ba9bdd 100644 (file)
--- a/data/rbot/plugins/autoop.rb
+++ b/data/rbot/plugins/autoop.rb
@@ -1,6 +1,6 @@
 class AutoOP < Plugin
     def help(plugin, topic="")
-        "perform autoop based on hostmask - usage: addop <hostmask> [channel channel ...], rmop <hostmask> [channel], list - list current ops. If you don't specify which channels, all channels are assumed"
+        "perform autoop based on hostmask - usage: add <hostmask> [channel channel ...], rm <hostmask> [channel], list - list current ops. If you don't specify which channels, all channels are assumed"
     end
     
     def join(m)
@@ -24,24 +24,27 @@ class AutoOP < Plugin
         m.reply @bot.lang.get('dunno')
         return
       end
-      if (params[:channels] && @registry[params[:mask]] != nil)
+      if (!params[:channels].empty? && @registry[params[:mask]] != nil)
         params[:channels].each do |c|
           @registry[params[:mask]] = @registry[params[:mask]].reject {|ele| ele =~ /^#{c}$/i}
         end
-      elsif(!@registry.delete(params[:mask]))
-        m.reply @bot.lang.get('dunno')
+        if @registry[params[:mask]].empty?
+          @registry.delete(params[:mask])
+        end
       else
-        m.okay
+        @registry.delete(params[:mask])
       end
+      m.okay
     end
 
     def list(m, params)
-      if(@registry.length)
+      debug @registry.length
+      if(@registry.length > 0)
         @registry.each { |mask,channels|
           m.reply "#{mask} in #{channels.empty? ? 'all channels' : channels.join(', ')}"
         }
       else
-        m.reply "No entrys"
+        m.reply "No entries"
       end
     end
 end