slack-smart-bot 1.9.2 → 1.10.0

data/lib/slack/smart-bot/commands/on_bot/add_shortcut.rb

@@ -18,16 +18,13 @@ class SlackSmartBot
   # help:        _sc spanish account_
   # help:        _shortcut Spanish Account_
   # help:        _Spanish Account_
+  # help:    <https://github.com/MarioRuiz/slack-smart-bot#shortcuts|more info>
   # help:
   def add_shortcut(dest, user, typem, for_all, shortcut_name, command, command_to_run, global)
     save_stats(__method__)
     unless typem == :on_extended
       from = user.name
-      if config[:allow_access].key?(__method__) and !config[:allow_access][__method__].include?(user.name) and !config[:allow_access][__method__].include?(user.id) and 
-        (!user.key?(:enterprise_user) or ( user.key?(:enterprise_user) and !config[:allow_access][__method__].include?(user[:enterprise_user].id)))
-        respond "You don't have access to use this command, please contact an Admin to be able to use it: <@#{config.admins.join(">, <@")}>"
-      else
-
+      if has_access?(__method__, user)
         if global 
           if !config.on_master_bot or typem != :on_master
             respond "It is only possible to add global shortcuts from Master channel"

data/lib/slack/smart-bot/commands/on_bot/admin/add_routine.rb

@@ -3,7 +3,9 @@ class SlackSmartBot
   # helpadmin: `add routine NAME every NUMBER PERIOD COMMAND`
   # helpadmin: `add routine NAME every NUMBER PERIOD #CHANNEL COMMAND`
   # helpadmin: `add routine NAME every NUMBER PERIOD`
+  # helpadmin: `add bgroutine NAME every NUMBER PERIOD`
   # helpadmin: `add silent routine NAME every NUMBER PERIOD`
+  # helpadmin: `add silent bgroutine NAME every NUMBER PERIOD`
   # helpadmin: `create routine NAME every NUMBER PERIOD`
   # helpadmin: `add routine NAME at TIME COMMAND`
   # helpadmin: `add routine NAME at TIME #CHANNEL COMMAND`
@@ -14,23 +16,27 @@ class SlackSmartBot
   # helpadmin: `create routine NAME at TIME`
   # helpadmin:    It will execute the command/rule supplied. Only for Admin and Master Admins.
   # helpadmin:    If no COMMAND supplied, then it will be necessary to attach a file with the code to be run and add this command as message to the file. ONLY for MASTER ADMINS.
+  # helpadmin:    In case *bgroutine* then the results of the run won't be published. To see the results call: `see result routine NAME`
   # helpadmin:    In case *silent* provided then when executed will be only displayed if the routine returns a message
   # helpadmin:    NAME: one word to identify the routine
   # helpadmin:    NUMBER: Integer
   # helpadmin:    PERIOD: days, d, hours, h, minutes, mins, min, m, seconds, secs, sec, s
   # helpadmin:    TIME: time at format HH:MM:SS
-  # helpadmin:    DAYWEEK: Monday, Tuesday, Wednesday, Thursday, Friday, Saturday, Sunday. And their plurals.
+  # helpadmin:    DAYWEEK: Monday, Tuesday, Wednesday, Thursday, Friday, Saturday, Sunday. And their plurals. Also possible to be used 'weekdays' and 'weekends'
   # helpadmin:    #CHANNEL: the destination channel where the results will be published. If not supplied then the bot channel by default or a DM if the command is run from a DM.
   # helpadmin:    COMMAND: any valid smart bot command or rule
+  # helpadmin:    It is possible to add a script directly. Only master admins can do it.
   # helpadmin:    Examples:
-  # helpadmin:      _add routine example every 30s ruby puts 'a'_
-  # helpadmin:      _add routine example every 3 days ruby puts 'a'_
-  # helpadmin:      _add routine example at 17:05 ruby puts 'a'_
+  # helpadmin:      _add routine example every 30s !ruby puts 'a'_
+  # helpadmin:      _add bgroutine example every 3 days !ruby puts 'a'_
+  # helpadmin:      _add routine example at 17:05 !ruby puts 'a'_
   # helpadmin:      _create silent routine Example every 12 hours !Run customer tests_
-  # helpadmin:      _add routine example on Mondays at 05:00 !run customer tests_
+  # helpadmin:      _add bgroutine example on Mondays at 05:00 !run customer tests_
   # helpadmin:      _add routine example on Tuesdays at 09:00 #SREChannel !run db cleanup_
+  # helpadmin:      _add routine example on weekdays at 22:00 suggest command_
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#routines|more info>
   # helpadmin:
-  def add_routine(dest, from, user, name, type, number_time, period, command_to_run, files, silent, channel)
+  def add_routine(dest, from, user, name, type, number_time, period, command_to_run, files, silent, channel, routine_type)
     save_stats(__method__)
     if files.nil? or files.size == 0 or (files.size > 0 and config.masters.include?(from))
       if config.admins.include?(from)
@@ -61,8 +67,9 @@ class SlackSmartBot
               every = "#{number_time} seconds"
               every_in_seconds = number_time.to_i
             else # time
-              if type != 'at'
+              if type != 'at' and type!='weekday' and type!='weekend'
                 dayweek = type.downcase
+
                 days = ['sunday','monday','tuesday','wednesday','thursday','friday','saturday']
                 incr = days.index(dayweek) - Time.now.wday
                 if incr < 0 
@@ -72,6 +79,14 @@ class SlackSmartBot
                 end
                 days = incr/(24*60*60)
                 every_in_seconds = 7 * 24 * 60 * 60 # one week
+              elsif type=='weekend'
+                dayweek = type.downcase
+                days = 0
+                every_in_seconds = 24 * 60 * 60 # one day
+              elsif type=='weekday'
+                dayweek = type.downcase
+                days = 0
+                every_in_seconds = 24 * 60 * 60 # one day
               else
                 days = 0
                 every_in_seconds = 24 * 60 * 60 # one day
@@ -95,8 +110,14 @@ class SlackSmartBot
               if files[0].filetype == "ruby" and files[0].name.scan(/[^\.]+(\.\w+$)/).join == ''
                 file_path += ".rb"
               end
-              http = NiceHttp.new(host: "https://files.slack.com", headers: { "Authorization" => "Bearer #{config[:token]}" }, log_headers: :partial)
-              http.get(files[0].url_private_download, save_data: file_path)
+              if files[0].key?(:content)
+                File.open(file_path, 'w') do |f| 
+                  f.write files[0].content
+                end
+              else
+                http = NiceHttp.new(host: "https://files.slack.com", headers: { "Authorization" => "Bearer #{config[:token]}" }, log_headers: :partial)
+                http.get(files[0].url_private_download, save_data: file_path)
+              end
               system("chmod +x #{file_path}")
             end
             get_channels_name_and_id() unless @channels_name.keys.include?(channel) or @channels_id.keys.include?(channel)
@@ -114,10 +135,10 @@ class SlackSmartBot
                                              every: every, every_in_seconds: every_in_seconds, at: at, dayweek: dayweek, file_path: file_path, 
                                              command: command_to_run.to_s.strip, silent: silent,
                                              next_run: next_run.to_s, dest: channel_id, last_run: "", last_elapsed: "", 
-                                             running: false }
+                                             running: false, routine_type: routine_type}
             update_routines
             respond "Added routine *`#{name}`* to the channel", dest
-            create_routine_thread(name)
+            create_routine_thread(name, @routines[@channel_id][name])
           end
         end
       else

data/lib/slack/smart-bot/commands/on_bot/admin/extend_rules.rb

@@ -4,6 +4,7 @@ class SlackSmartBot
   # helpadmin: `extend rules to CHANNEL_NAME`
   # helpadmin: `use rules on CHANNEL_NAME`
   # helpadmin:    It will allow to use the specific rules from this channel on the CHANNEL_NAME
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#extending-rules-to-other-channels|more info>
   # helpadmin:
   def extend_rules(dest, user, from, channel, typem)
     save_stats(__method__)
@@ -15,6 +16,7 @@ class SlackSmartBot
       else
         #todo: add pagination for case more than 1000 channels on the workspace
         channels = get_channels()
+        channel = @channels_name[channel] if @channels_name.key?(channel)
 
         channel_found = channels.detect { |c| c.name == channel }
         get_channels_name_and_id()

data/lib/slack/smart-bot/commands/on_bot/admin/pause_bot.rb

@@ -5,6 +5,7 @@ class SlackSmartBot
   # helpadmin: `pause this bot`
   # helpadmin:    the bot will pause so it will listen only to admin commands
   # helpadmin:    You can use this command only if you are an admin user
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#bot-management|more info>
   # helpadmin:
   def pause_bot(dest, from)
     save_stats(__method__)
@@ -12,11 +13,12 @@ class SlackSmartBot
       respond "This bot is paused from now on. You can start it again: start this bot", dest
       respond "zZzzzzZzzzzZZZZZZzzzzzzzz", dest
       @status = :paused
-      @bots_created[@channel_id][:status] = :paused
-      update_bots_file()
       unless config.on_master_bot
+        @bots_created[@channel_id][:status] = :paused 
+        update_bots_file()
         send_msg_channel config.master_channel, "Changed status on #{config.channel} to :paused"
       end
+      save_status :off, :paused, 'The admin paused this bot'
     else
       respond "Only admin users can put me on pause", dest
     end

data/lib/slack/smart-bot/commands/on_bot/admin/pause_routine.rb

@@ -6,6 +6,7 @@ class SlackSmartBot
   # helpadmin:    NAME: one word to identify the routine
   # helpadmin:    Examples:
   # helpadmin:      _pause routine example_
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#routines|more info>
   # helpadmin:
   def pause_routine(dest, from, name)
     save_stats(__method__)

data/lib/slack/smart-bot/commands/on_bot/admin/remove_routine.rb

@@ -9,13 +9,12 @@ class SlackSmartBot
   # helpadmin:    NAME: one word to identify the routine
   # helpadmin:    Examples:
   # helpadmin:      _kill routine example_
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#routines|more info>
   # helpadmin:
   def remove_routine(dest, from, name)
     save_stats(__method__)
     if config.admins.include?(from) #admin user
-      if !config.on_master_bot and dest[0] == "D"
-        respond "It's only possible to remove routines from MASTER channel from a direct message with the bot.", dest
-      elsif @routines.key?(@channel_id) and @routines[@channel_id].key?(name)
+      if @routines.key?(@channel_id) and @routines[@channel_id].key?(name)
         @routines[@channel_id][name][:thread].exit
         @routines[@channel_id].delete(name)
         update_routines()

data/lib/slack/smart-bot/commands/on_bot/admin/run_routine.rb

@@ -8,6 +8,7 @@ class SlackSmartBot
   # helpadmin:    NAME: one word to identify the routine
   # helpadmin:    Examples:
   # helpadmin:      _run routine example_
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#routines|more info>
   # helpadmin:
 
   def run_routine(dest, from, name)
@@ -16,6 +17,7 @@ class SlackSmartBot
       if !config.on_master_bot and dest[0] == "D"
         respond "It's only possible to run routines from MASTER channel from a direct message with the bot.", dest
       elsif @routines.key?(@channel_id) and @routines[@channel_id].key?(name)
+        File.delete "#{config.path}/routines/#{@channel_id}/#{name}_output.txt" if File.exists?("#{config.path}/routines/#{@channel_id}/#{name}_output.txt")
         if @routines[@channel_id][name][:file_path] != ""
           if @routines[@channel_id][name][:file_path].match?(/\.rb$/i)
             ruby = "ruby "
@@ -40,7 +42,10 @@ class SlackSmartBot
           treat_message({ channel: @routines[@channel_id][name][:dest],
                          user: @routines[@channel_id][name][:creator_id],
                          text: @routines[@channel_id][name][:command],
-                         files: nil })
+                         files: nil,
+                         routine_name: name, 
+                         routine_type: @routines[@channel_id][name][:routine_type],
+                         routine: true })
         end
         @routines[@channel_id][name][:last_elapsed] = (Time.now - started)
         @routines[@channel_id][name][:last_run] = started.to_s

data/lib/slack/smart-bot/commands/on_bot/admin/see_result_routine.rb

@@ -0,0 +1,32 @@
+class SlackSmartBot
+
+  # helpadmin: ----------------------------------------------
+  # helpadmin: `see routine result NAME`
+  # helpadmin: `see result routine NAME`
+  # helpadmin: `result routine NAME`
+  # helpadmin:    It will display the last result of the routine run.
+  # helpadmin:    You can use this command only if you are an admin user
+  # helpadmin:    NAME: one word to identify the routine
+  # helpadmin:    Examples:
+  # helpadmin:      _see routine result example_
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#routines|more info>
+  # helpadmin:
+  def see_result_routine(dest, from, name)
+    save_stats(__method__)
+    if config.admins.include?(from) #admin user
+      if @routines.key?(@channel_id) and @routines[@channel_id].key?(name)
+        if File.exist?("#{config.path}/routines/#{@channel_id}/#{name}_output.txt")
+          msg = "*Results from routine run #{File.mtime("#{config.path}/routines/#{@channel_id}/#{name}_output.txt")}*\n"
+          msg += File.read("#{config.path}/routines/#{@channel_id}/#{name}_output.txt")
+          respond msg, dest
+        else
+          respond "The routine *`#{name}`* doesn't have any result yet.", dest
+        end
+      else
+        respond "There isn't a routine with that name: *`#{name}`*.\nCall `see routines` to see added routines", dest
+      end
+    else
+      respond "Only admin users can see the routines results", dest
+    end
+  end
+end

data/lib/slack/smart-bot/commands/on_bot/admin/see_routines.rb

@@ -5,6 +5,7 @@ class SlackSmartBot
   # helpadmin:    It will show the routines of the channel
   # helpadmin:    In case of `all` and on the master channel, it will show all the routines from all channels
   # helpadmin:    You can use this command only if you are an admin user
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#routines|more info>
   # helpadmin:
   def see_routines(dest, from, user, all)
     save_stats(__method__)
@@ -23,8 +24,8 @@ class SlackSmartBot
           routines = @routines.dup
         end
       else
-        if @rules_imported.key?(user.id) and @rules_imported[user.id].key?(user.id) and dest[0] == "D"
-          file_conf = IO.readlines("#{config.path}/routines/routines_#{@rules_imported[user.id][user.id]}.rb").join
+        if @rules_imported.key?(user.name) and @rules_imported[user.name].key?(user.name) and dest[0] == "D"
+          file_conf = IO.readlines("#{config.path}/routines/routines_#{@rules_imported[user.name][user.name]}.rb").join
           routines = eval(file_conf)
         else
           routines = @routines.dup
@@ -57,6 +58,7 @@ class SlackSmartBot
             msg << "\tCommand: #{v[:command]}" unless v[:command].to_s.strip == ''
             msg << "\tFile: #{v[:file_path]}" unless v[:file_path] == ''
             msg << "\tSilent: #{v[:silent]}" unless !v[:silent]
+            msg << "\tType: #{v[:routine_type]}" if v[:routine_type].to_s == 'bgroutine'
             respond msg.join("\n"), dest
           end
         end

data/lib/slack/smart-bot/commands/on_bot/admin/start_bot.rb

@@ -5,17 +5,19 @@ class SlackSmartBot
   # helpadmin: `start this bot`
   # helpadmin:    the bot will start to listen
   # helpadmin:    You can use this command only if you are an admin user
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#bot-management|more info>
   # helpadmin:
   def start_bot(dest, from)
     save_stats(__method__)
     if config.admins.include?(from) #admin user
       respond "This bot is running and listening from now on. You can pause again: pause this bot", dest
       @status = :on
-      @bots_created[@channel_id][:status] = :on
-      update_bots_file()
       unless config.on_master_bot
+        @bots_created[@channel_id][:status] = :on
+        update_bots_file()
         send_msg_channel config.master_channel, "Changed status on #{config.channel} to :on"
       end
+      save_status :on, :started, 'The admin started this bot'
     else
       respond "Only admin users can change my status", dest
     end

data/lib/slack/smart-bot/commands/on_bot/admin/start_routine.rb

@@ -7,6 +7,7 @@ class SlackSmartBot
   # helpadmin:    NAME: one word to identify the routine
   # helpadmin:    Examples:
   # helpadmin:      _start routine example_
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#routines|more info>
   # helpadmin:
 
   def start_routine(dest, from, name)

data/lib/slack/smart-bot/commands/on_bot/admin/stop_using_rules_on.rb

@@ -3,6 +3,7 @@ class SlackSmartBot
   # helpadmin: ----------------------------------------------
   # helpadmin: `stop using rules on CHANNEL_NAME`
   # helpadmin:    it will stop using the extended rules on the specified channel.
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#extending-rules-to-other-channels|more info>
   # helpadmin:
 
   def stop_using_rules_on(dest, user, from, channel, typem)
@@ -12,6 +13,7 @@ class SlackSmartBot
         respond "Only admins can extend or stop using the rules. Admins on this channel: #{config.admins}", dest
       else
         get_bots_created()
+        channel = @channels_name[channel] if @channels_name.key?(channel)
         if @bots_created[@channel_id][:extended].include?(channel)
           @bots_created[@channel_id][:extended].delete(channel)
           update_bots_file()

data/lib/slack/smart-bot/commands/on_bot/admin_master/react_to.rb

@@ -0,0 +1,32 @@
+class SlackSmartBot
+
+    # helpadmin: ----------------------------------------------
+    # helpadmin: `react to #CHANNEL_NAME THREAD_ID EMOJIS`
+    # helpadmin:    It will send the specified reactions as SmartBot
+    # helpadmin:    You can use this command only if you are a Master admin user and if you are in a private conversation with the bot
+    # helpadmin:     Examples:
+    # helpadmin:       _react to #sales 1622550707.012100 :thumbsup:_
+    # helpadmin:       _react to #sales p1622550707012100 :thumbsup:_
+    # helpadmin:       _react to #sales p1622550707012100 :thumbsup: :heavy_check_mark: :bathtub:_
+    # helpadmin:
+    def react_to(dest, from, typem, to, thread_ts, emojis)
+      save_stats(__method__)
+      if config.masters.include?(from) and typem==:on_dm #master admin user
+        succs = []
+        emojis.split(' ').each do |emoji|
+          succs << (react emoji, thread_ts, to)
+        end
+        succs.uniq!
+        if succs.size == 1 and succs[0] == true
+          react :heavy_check_mark
+        elsif succs.size == 2
+          react :exclamation
+        else
+          react :x
+        end
+      else
+        respond "Only master admin users on a `pr`ivate conversation with the SmartBot can send reactions as SmartBot.", dest
+      end
+    end
+end
+  

data/lib/slack/smart-bot/commands/on_bot/admin_master/send_message.rb

@@ -0,0 +1,24 @@
+class SlackSmartBot
+
+    # helpadmin: ----------------------------------------------
+    # helpadmin: `send message to @USER_NAME : MESSAGE`
+    # helpadmin: `send message to #CHANNEL_NAME : MESSAGE`
+    # helpadmin: `send message to #CHANNEL_NAME THREAD_ID : MESSAGE`
+    # helpadmin:    It will send the specified message as SmartBot
+    # helpadmin:    You can use this command only if you are a Master admin user and if you are in a private conversation with the bot
+    # helpadmin:
+    def send_message(dest, from, typem, to, thread_ts, message)
+      save_stats(__method__)
+      if config.masters.include?(from) and typem==:on_dm #master admin user
+        succ = (respond message, to, thread_ts: thread_ts, web_client: true)
+        if succ
+          react :heavy_check_mark
+        else
+          react :x
+        end
+      else
+        respond "Only master admin users on a private conversation with the SmartBot can send messages as SmartBot.", dest
+      end
+    end
+end
+  

data/lib/slack/smart-bot/commands/on_bot/delete_repl.rb

@@ -5,14 +5,12 @@ class SlackSmartBot
   # help: `remove repl SESSION_NAME`
   # help:     Will delete the specified REPL
   # help:     Only the creator of the REPL or an admin can delete REPLs
+  # help:     <https://github.com/MarioRuiz/slack-smart-bot#repl|more info>
   # help:
   def delete_repl(dest, user, session_name)
     #todo: add tests
     save_stats(__method__)
-    if config[:allow_access].key?(__method__) and !config[:allow_access][__method__].include?(user.name) and !config[:allow_access][__method__].include?(user.id) and 
-      (!user.key?(:enterprise_user) or ( user.key?(:enterprise_user) and !config[:allow_access][__method__].include?(user[:enterprise_user].id)))
-      respond "You don't have access to use this command, please contact an Admin to be able to use it: <@#{config.admins.join(">, <@")}>"
-    else
+    if has_access?(__method__, user)
       if @repls.key?(session_name)
         Dir.mkdir("#{config.path}/repl") unless Dir.exist?("#{config.path}/repl")
         if config.admins.include?(user.name) or @repls[session_name].creator_name == user.name

data/lib/slack/smart-bot/commands/on_bot/delete_shortcut.rb

@@ -5,16 +5,14 @@ class SlackSmartBot
   # help: `delete global sc NAME`
   # help:    It will delete the shortcut with the supplied name
   # help:    'global' or 'generic' can only be used on Master channel.
+  # help:    <https://github.com/MarioRuiz/slack-smart-bot#shortcuts|more info>
   # help:
 
   def delete_shortcut(dest, user, shortcut, typem, command, global)
     save_stats(__method__)
     unless typem == :on_extended
       from = user.name
-      if config[:allow_access].key?(__method__) and !config[:allow_access][__method__].include?(user.name) and !config[:allow_access][__method__].include?(user.id) and 
-        (!user.key?(:enterprise_user) or ( user.key?(:enterprise_user) and !config[:allow_access][__method__].include?(user[:enterprise_user].id)))
-        respond "You don't have access to use this command, please contact an Admin to be able to use it: <@#{config.admins.join(">, <@")}>"
-      else
+      if has_access?(__method__, user)
         deleted = false
 
         if global

data/lib/slack/smart-bot/commands/on_bot/get_repl.rb

@@ -4,14 +4,12 @@ class SlackSmartBot
   # help: `get irb SESSION_NAME`
   # help: `get live SESSION_NAME`
   # help:     Will get the Ruby commands sent on that SESSION_NAME.
+  # help:     <https://github.com/MarioRuiz/slack-smart-bot#repl|more info>
   # help:
   def get_repl(dest, user, session_name)
     #todo: add tests
     save_stats(__method__)
-    if config[:allow_access].key?(__method__) and !config[:allow_access][__method__].include?(user.name) and !config[:allow_access][__method__].include?(user.id) and 
-      (!user.key?(:enterprise_user) or ( user.key?(:enterprise_user) and !config[:allow_access][__method__].include?(user[:enterprise_user].id)))
-      respond "You don't have access to use this command, please contact an Admin to be able to use it: <@#{config.admins.join(">, <@")}>"
-    else
+    if has_access?(__method__, user)
       Dir.mkdir("#{config.path}/repl") unless Dir.exist?("#{config.path}/repl")
       Dir.mkdir("#{config.path}/repl/#{@channel_id}") unless Dir.exist?("#{config.path}/repl/#{@channel_id}")
       if File.exist?("#{config.path}/repl/#{@channel_id}/#{session_name}.run")

data/lib/slack/smart-bot/commands/on_bot/repl.rb

@@ -19,7 +19,7 @@ class SlackSmartBot
   # help:     Send _quit_, _bye_ or _exit_ to finish the session.
   # help:     Send puts, print, p or pp if you want to print out something when using `run repl` later.
   # help:     After 30 minutes of no communication with the Smart Bot the session will be dismissed.
-  # help:     If you declare on your rules file a method called `project_folder` returning the path for the project folder, the code will be executed from that folder. 
+  # help:     If you declare on your rules file a method called 'project_folder' returning the path for the project folder, the code will be executed from that folder. 
   # help:     By default it will be automatically loaded the gems: string_pattern, nice_hash and nice_http
   # help:     To pre-execute some ruby when starting the session add the code to .smart-bot-repl file on the project root folder defined on project_folder
   # help:     If you want to see the methods of a class or module you created use _ls TheModuleOrClass_
@@ -29,14 +29,12 @@ class SlackSmartBot
   # help:       _repl CreateCustomer: "It creates a random customer for testing" LOCATION=spain HOST='https://10.30.40.50:8887'_
   # help:       _repl delete_logs_
   # help:       _private repl random-ssn_
+  # help:     <https://github.com/MarioRuiz/slack-smart-bot#repl|more info>
   # help:
   def repl(dest, user, session_name, env_vars, rules_file, command, description, type)
     #todo: add more tests
     from = user.name
-    if config[:allow_access].key?(__method__) and !config[:allow_access][__method__].include?(user.name) and !config[:allow_access][__method__].include?(user.id) and 
-      (!user.key?(:enterprise_user) or ( user.key?(:enterprise_user) and !config[:allow_access][__method__].include?(user[:enterprise_user].id)))
-      respond "You don't have access to use this command, please contact an Admin to be able to use it: <@#{config.admins.join(">, <@")}>"
-    else
+    if has_access?(__method__, user)
       if !@repl_sessions.key?(from)
         save_stats(__method__)
         Dir.mkdir("#{config.path}/repl") unless Dir.exist?("#{config.path}/repl")
@@ -147,7 +145,7 @@ class SlackSmartBot
                   rescue Exception => resp_repl
                     error = true
                   end
-                  if resp_repl.to_s != \"\"
+                  unless error
                     if code_to_run_repl.match?(/^\s*p\s+/i)
                       open(\"' + File.expand_path(config.path) + '/repl/' + @channel_id + '/' + session_name + '.output\", \"a+\") {|f|
                         f.puts \"\`\`\`\n#{resp_repl.inspect}\n\`\`\`\"
@@ -157,7 +155,7 @@ class SlackSmartBot
                         f.puts \"\`\`\`\n#{resp_repl.ai}\n\`\`\`\"
                       }
                     end
-                    unless error or !add_to_run_repl
+                    unless !add_to_run_repl
                       open(\"' + File.expand_path(config.path) + '/repl/' + @channel_id + '/' + session_name + '.run\", \"a+\") {|f|
                         f.puts code_to_run_repl
                       }