Merge pull request #23 from pepabo/fix-warning

fix "method redefined"

Author: kenchan

lib/arisaid/configurable.rb

@@ -6,8 +6,6 @@ module Configurable
     OPTIONS_KEYS = %i(
       debug
       read_only
-      slack_team
-      slack_token
       save_token
       conf_prefix
       exit_status
@@ -64,6 +62,10 @@ def slack_token
       ENV['SLACK_API_TOKEN'] ||= slack_token!
     end
 
+    def slack_token=(token)
+      ENV['SLACK_API_TOKEN'] = token
+    end
+
     def slack_token!
       token = slack_token_by_pit
 

lib/arisaid/core_ext/hash.rb

@@ -1,4 +1,6 @@
 class Hash
+  alias_method :original_slice, :slice
+
   def slice(*keys)
     keys.each_with_object(self.class.new) { |k, hash| hash[k] = self[k] if has_key?(k) }
   end

lib/arisaid/userable.rb

@@ -1,33 +1,31 @@
 module Arisaid
   module Userable
-    attr_reader :users, :guests, :bots
-
     def users
-      @users ||= users!
+      @_users ||= users!
     end
 
     def users!
-      @users = all_users.select { |u|
+      @_users = all_users.select { |u|
         u.deleted == false && u.is_bot == false && u.is_restricted == false
       }
     end
 
     def guests
-      @guests ||= guests!
+      @_guests ||= guests!
     end
 
     def guests!
-      @guests = all_users.select { |u|
+      @_guests = all_users.select { |u|
         u.deleted == false && u.is_bot == false && u.is_restricted == true
       }
     end
 
     def bots
-      @bots ||= bots!
+      @_bots ||= bots!
     end
 
     def bots!
-      @bots = all_users.select { |u|
+      @_bots = all_users.select { |u|
         u.deleted == false && u.is_bot == true && u.is_restricted == false
       }
     end