slack-smart-bot 1.9.1 → 1.11.0

data/lib/slack/smart-bot/commands/on_bot/general/bot_stats.rb

@@ -0,0 +1,400 @@
+class SlackSmartBot
+    # help: ----------------------------------------------
+    # help: `bot stats`
+    # helpmaster: `bot stats USER_NAME`
+    # help: `bot stats exclude masters`
+    # help: `bot stats exclude routines`
+    # help: `bot stats from YYYY/MM/DD`
+    # help: `bot stats from YYYY/MM/DD to YYYY/MM/DD`
+    # help: `bot stats CHANNEL`
+    # help: `bot stats CHANNEL from YYYY/MM/DD`
+    # help: `bot stats CHANNEL from YYYY/MM/DD to YYYY/MM/DD`
+    # help: `bot stats command COMMAND`
+    # helpmaster: `bot stats USER_NAME from YYYY/MM/DD to YYYY/MM/DD`
+    # helpmaster: `bot stats CHANNEL USER_NAME from YYYY/MM/DD to YYYY/MM/DD`
+    # help: `bot stats CHANNEL exclude masters from YYYY/MM/DD to YYYY/MM/DD`
+    # help: `bot stats members #CHANNEL`
+    # help: `bot stats exclude members #CHANNEL`
+    # help: `bot stats today`
+    # help: `bot stats exclude COMMAND_ID`
+    # help: `bot stats monthly`
+    # help: `bot stats alldata`
+    # help:    To see the bot stats
+    # helpmaster:    You can use this command only if you are a Master admin user and if you are in a private conversation with the bot
+    # helpmaster:    You need to set stats to true to generate the stats when running the bot instance.
+    # help:    members #CHANNEL will return stats for only members of the channel supplied
+    # help:    exclude members #CHANNEL will return stats for only members that are not members of the channel supplied
+    # help:    If alldata option supplied then it will be attached files including all data and not only the top 10.
+    # help:    Examples:
+    # help:      _bot stats #sales_
+    # helpmaster:      _bot stats @peter.wind_
+    # help:      _bot stats #sales from 2019/12/15 to 2019/12/31_
+    # help:      _bot stats #sales today_
+    # help:      _bot stats #sales from 2020-01-01 monthly_
+    # help:      _bot stats exclude routines masters from 2021/01/01 monthly_
+    # help:      _bot stats members #development from 2022/01/01 to 2022/01/31_
+    # help:    <https://github.com/MarioRuiz/slack-smart-bot#bot-management|more info>
+    # help: command_id: :bot_stats
+    # help:
+    def bot_stats(dest, from_user, typem, channel_id, from, to, user, st_command, exclude_masters, exclude_routines, exclude_command, monthly, all_data, members_channel, exclude_members_channel)
+        require 'csv'
+        if config.stats
+            message = []
+        else
+            message = ["You need to set stats to true to generate the stats when running the bot instance."]
+        end
+        save_stats(__method__)
+        if (from_user.id != user and (config.masters.include?(from_user.name) or @master_admin_users_id.include?(from_user.id)) and (typem==:on_dm or dest[0]=='D'))
+            on_dm_master = true #master admin user
+        else
+            on_dm_master = false
+        end
+        wrong = false
+        exclude_channel_members = false
+        include_channel_members = false
+        members_list = []
+        if exclude_members_channel!='' or members_channel!=''
+            if members_channel!=''
+                channel_members = members_channel
+                include_channel_members = true
+            else
+                channel_members = exclude_members_channel
+                exclude_channel_members = true                
+            end
+            get_channels_name_and_id() unless @channels_id.keys.include?(channel_members)
+
+            tm = get_channel_members(channel_members)
+            if tm.nil?
+                message << ":exclamation: Add the Smart Bot to *<##{channel_members}>* channel first."
+                wrong = true
+            else
+              tm.each do |m|
+                user_info = @users.select { |u| u.id == m or (u.key?(:enterprise_user) and u.enterprise_user.id == m) }[-1]
+                members_list << user_info.name unless user_info.is_app_user or user_info.is_bot
+              end
+            end
+        end
+        tzone_users = {}
+
+        unless wrong
+            if on_dm_master or (from_user.id == user) # normal user can only see own stats 
+                if !File.exist?("#{config.stats_path}.#{Time.now.strftime("%Y-%m")}.log")
+                    message<<'No stats'
+                else
+                    from = "#{Time.now.strftime("%Y-%m")}-01" if from == ''
+                    to = "#{Time.now.strftime("%Y-%m-%d")}" if to == ''
+                    from_short = from
+                    to_short = to
+                    from_file = from[0..3] + '-' + from[5..6]
+                    to_file = to[0..3] + '-' + to[5..6]
+                    from+= " 00:00:00 +0000"
+                    to+= " 23:59:59 +0000"
+                    rows = []
+                    rows_month = {}
+                    users_month = {}
+                    commands_month = {}
+                    users_id_name = {}
+                    users_name_id = {}
+                    count_users = {}
+                    count_channels_dest = {}
+        
+                    # to translate global and enterprise users since sometimes was returning different names/ids
+                    #if from[0..3]=='2020' # this was an issue only on that period
+                        Dir["#{config.stats_path}.*.log"].sort.each do |file|
+                            if file >= "#{config.stats_path}.#{from_file}.log" and file <= "#{config.stats_path}.#{to_file}.log"
+                                CSV.foreach(file, headers: true, header_converters: :symbol, converters: :numeric) do |row|
+                                    unless users_id_name.key?(row[:user_id])
+                                        users_id_name[row[:user_id]] = row[:user_name]
+                                        users_name_id[row[:user_name]] = row[:user_id]
+                                    end
+                                end
+                            end
+                        end
+                    #end
+        
+                    if user!=''
+                        user_info = @users.select{|u| u.id == user or (u.key?(:enterprise_user) and u.enterprise_user.id == user)}[-1]
+                        if user_info.nil? # for the case the user is populated from outside of slack
+                            user_name = user
+                            user_id = user
+                        else
+                            if users_id_name.key?(user_info.id)
+                                user_name = users_id_name[user_info.id]
+                            else
+                                user_name = user_info.name
+                            end
+                            if users_name_id.key?(user_info.name)
+                                user_id = users_name_id[user_info.name]
+                            else
+                                user_id = user_info.id
+                            end                    
+                        end
+                    end
+                    master_admins = config.masters.dup
+                    if users_id_name.size > 0
+                        config.masters.each do |u|
+                            if users_id_name.key?(u)
+                                master_admins << users_id_name[u]
+                            elsif users_name_id.key?(u)
+                                master_admins << users_name_id[u]
+                            end
+                        end
+                    end
+        
+                    Dir["#{config.stats_path}.*.log"].sort.each do |file|
+                        if file >= "#{config.stats_path}.#{from_file}.log" and file <= "#{config.stats_path}.#{to_file}.log"
+                            CSV.foreach(file, headers: true, header_converters: :symbol, converters: :numeric) do |row|
+                                if (include_channel_members and members_list.include?(row[:user_name])) or 
+                                    (exclude_channel_members and !members_list.include?(row[:user_name])) or
+                                    (!include_channel_members and !exclude_channel_members)
+
+                                    row[:date] = row[:date].to_s
+                                    if row[:dest_channel_id].to_s[0]=='D'
+                                        row[:dest_channel] = 'DM'
+                                    elsif row[:dest_channel].to_s == ''
+                                        row[:dest_channel] = row[:dest_channel_id]
+                                    end
+                                    if users_name_id.size > 0
+                                        row[:user_name] = users_id_name[row[:user_id]]
+                                        row[:user_id] = users_name_id[row[:user_name]]
+                                    else
+                                        users_id_name[row[:user_id]] ||= row[:user_name]
+                                    end
+                                    if !exclude_masters or (exclude_masters and !master_admins.include?(row[:user_name]) and 
+                                                            !master_admins.include?(row[:user_id]) and
+                                                            !@master_admin_users_id.include?(row[:user_id]))
+                                        if !exclude_routines or (exclude_routines and !row[:user_name].match?(/^routine\//) )
+                                            if exclude_command == '' or (exclude_command!='' and row[:command]!=exclude_command)
+                                                if st_command == '' or (st_command != '' and row[:command] == st_command)
+                                                    if row[:bot_channel_id] == channel_id or channel_id == '' or row[:dest_channel_id] == channel_id
+                                                        if row[:date] >= from and row[:date] <= to
+                                                            count_users[row[:user_id]] ||= 0
+                                                            count_users[row[:user_id]] += 1
+                                                            if user=='' or (user!='' and row[:user_name] == user_name) or (user!='' and row[:user_id] == user_id)
+                                                                rows << row.to_h
+                                                                count_channels_dest[row[:dest_channel]] ||= 0
+                                                                count_channels_dest[row[:dest_channel]] += 1
+                                                                if monthly
+                                                                    rows_month[row[:date][0..6]] = 0 unless rows_month.key?(row[:date][0..6])
+                                                                    users_month[row[:date][0..6]] = [] unless users_month.key?(row[:date][0..6])
+                                                                    commands_month[row[:date][0..6]] = [] unless commands_month.key?(row[:date][0..6])
+                                                                    rows_month[row[:date][0..6]] += 1
+                                                                    users_month[row[:date][0..6]] << row[:user_id]
+                                                                    commands_month[row[:date][0..6]] << row[:command]
+                                                                end
+                                                            end
+                                                        end
+                                                    end
+                                                end
+                                            end
+                                        end
+                                    end
+                                end
+                            end
+                        end
+                    end
+                    total = rows.size
+                    if exclude_masters
+                        message << 'Excluding master admins'
+                    end
+                    if exclude_routines
+                        message << 'Excluding routines'
+                    end
+                    if exclude_command != ''
+                        message << "Excluding command #{exclude_command}"
+                    end
+                    if st_command != ''
+                        message << "Including only command #{st_command}"
+                    end
+                    if include_channel_members
+                        message << "Including only members of <##{members_channel}>"
+                    end
+                    if exclude_channel_members
+                        message << "Including only members that are not members of <##{exclude_members_channel}>"
+                    end
+                    if user!=''
+                        if user==from_user.id
+                            message << "Bot stats for <@#{user}>"
+                        else
+                            message << "Showing only user <@#{user}>"
+                        end
+                    end
+                    if channel_id == ''
+                        message << "*Total calls*: #{total} from #{from_short} to #{to_short}"
+                    else
+                        message << "*Total calls <##{channel_id}>*: #{total} from #{from_short} to #{to_short}"
+                    end
+                    unless count_users.size == 0 or total == 0 or user == ''
+                        my_place = (count_users.sort_by(&:last).reverse.to_h.keys.index(user_id)+1)
+                        message <<"\tYou are the *\# #{my_place}* of *#{count_users.size}* users"
+                    end
+                    if total > 0
+                        if monthly 
+                            if on_dm_master
+                                message << '*Totals by month / commands / users (%new)*'
+                            else
+                                message << '*Totals by month / commands*'
+                            end
+
+                            all_users = []
+                            new_users = []
+                            rows_month.each do |k,v|
+                                if all_users.empty?
+                                    message_new_users = ''
+                                else
+                                    new_users = (users_month[k]-all_users).uniq
+                                    message_new_users = "(#{new_users.size*100/users_month[k].uniq.size}%)"
+                                end
+                                all_users += users_month[k]
+                                if on_dm_master
+                                    message << "\t#{k}: #{v} (#{(v.to_f*100/total).round(2)}%) / #{commands_month[k].uniq.size} / #{users_month[k].uniq.size} #{message_new_users}"
+                                else
+                                    message << "\t#{k}: #{v} (#{(v.to_f*100/total).round(2)}%) / #{commands_month[k].uniq.size}"
+                                end
+                            end
+                        end
+        
+                        if channel_id == ''
+                            message << "*SmartBots*"
+                            channels = rows.bot_channel.uniq.sort
+                            channels.each do |channel|
+                                count = rows.count {|h| h.bot_channel==channel}
+                                channel_info = @channels_list.select { |c| c.name.to_s.downcase == channel.to_s.downcase}[-1]
+                                if @channels_id.key?(channel) and !channel_info.is_private
+                                    c = "<##{@channels_id[channel]}>"
+                                else
+                                    c = channel
+                                end
+                                message << "\t#{c}: #{count} (#{(count.to_f*100/total).round(2)}%)"
+                            end
+                        end
+                        channels_dest_attachment = []
+                        count_channels_dest = count_channels_dest.sort_by(&:last).reverse.to_h
+                        if count_channels_dest.size > 10
+                            message << "*From Channel* - #{count_channels_dest.size} (Top 10)"
+                        else
+                            message << "*From Channel* - #{count_channels_dest.size}"
+                        end
+
+                        count_channels_dest.keys[0..9].each do |ch|
+                            channel_info = @channels_list.select { |c| c.name.to_s.downcase == ch.to_s.downcase}[-1]
+                            if @channels_id.key?(ch) and !channel_info.is_private
+                                c = "<##{@channels_id[ch]}>"
+                            else
+                                c = ch
+                            end
+                            message << "\t#{c}: #{count_channels_dest[ch]} (#{(count_channels_dest[ch].to_f*100/total).round(2)}%)"
+                        end
+                        if count_channels_dest.size > 10 and all_data
+                            count_channels_dest.each do |ch, value|
+                                channel_info = @channels_list.select { |c| c.name.to_s.downcase == ch.to_s.downcase}[-1]
+                                channels_dest_attachment << "\t##{ch}: #{value} (#{(value.to_f*100/total).round(2)}%)"
+                            end
+                        end
+                        
+                        users_attachment = []
+                        if user==''
+                            users = rows.user_id.uniq.sort
+                            rows.user_id.each do |usr|
+                                user_info = @users.select { |u| u.id == usr or (u.key?(:enterprise_user) and u.enterprise_user.id == usr) }[-1]
+                                unless user_info.nil? or user_info.is_app_user or user_info.is_bot
+                                    tzone_users[user_info.tz_label] ||= 0
+                                    tzone_users[user_info.tz_label] += 1
+                                end
+                            end
+                            if users.size > 10
+                                message << "*Users* - #{users.size} (Top 10)"
+                            else
+                                message << "*Users* - #{users.size}"
+                            end
+                            count_user = {}
+                            users.each do |user|
+                                count = rows.count {|h| h.user_id==user}
+                                count_user[user] = count
+                            end
+                            i = 0
+                            total_without_routines = total
+                            count_user.sort_by {|k,v| -v}.each do |user, count|
+                                i+=1
+                                if user.include?('routine/')
+                                    user_link = users_id_name[user]
+                                    total_without_routines -= count
+                                else
+                                    user_link = "<@#{user}>"
+                                end
+                                if i <= 10
+                                    message << "\t#{user_link}: #{count} (#{(count.to_f*100/total).round(2)}%)"
+                                end
+                                if users.size > 10 and all_data
+                                    users_attachment << "\t#{users_id_name[user]}: #{count} (#{(count.to_f*100/total).round(2)}%)"
+                                end
+                            end
+
+                            if tzone_users.size > 0
+                                message << "*Time Zones*"
+                                total_known = 0
+                                tzone_users.each do |tzone, num|
+                                    message << "\t#{tzone}: #{num} (#{(num.to_f*100/total_without_routines).round(2)}%)"
+                                    total_known+=num
+                                end
+                                total_unknown = total_without_routines - total_known
+                                message << "\tUnknown: #{total_unknown} (#{(total_unknown.to_f*100/total_without_routines).round(2)}%)" if total_unknown > 0
+                            end
+
+                        end
+                        commands_attachment = []
+
+                        if st_command == ''
+                            commands = rows.command.uniq.sort
+                            count_command = {}
+                            commands.each do |command|
+                                count = rows.count {|h| h.command==command}
+                                count_command[command] = count
+                            end
+
+                            if commands.size > 10
+                                message << "*Commands* - #{commands.size} (Top 10)"
+                            else
+                                message << "*Commands* - #{commands.size}"
+                            end
+
+                            i = 0
+                            count_command.sort_by {|k,v| -v}.each do |command, count|
+                                i+=1
+                                if i <= 10
+                                    message << "\t#{command}: #{count} (#{(count.to_f*100/total).round(2)}%)"
+                                end
+                                if commands.size > 10 and all_data
+                                    commands_attachment << "\t#{command}: #{count} (#{(count.to_f*100/total).round(2)}%)"
+                                end
+                            end
+                        end
+        
+                        message << "*Message type*"
+                        types = rows.type_message.uniq.sort
+                        types.each do |type|
+                            count = rows.count {|h| h.type_message==type}
+                            message << "\t#{type}: #{count} (#{(count.to_f*100/total).round(2)}%)"
+                        end
+
+                        if on_dm_master
+                            message << "*Last activity*: #{rows[-1].date} #{rows[-1].bot_channel} #{rows[-1].type_message} #{rows[-1].user_name} #{rows[-1].command}"
+                        end
+                        if users_attachment.size>0
+                            send_file(dest, "", 'users.txt', "", 'text/plain', "text", content: users_attachment.join("\n"))
+                        end
+                        if commands_attachment.size>0
+                            send_file(dest, "", 'commands.txt', "", 'text/plain', "text", content: commands_attachment.join("\n"))
+                        end
+                        if channels_dest_attachment.size>0
+                            send_file(dest, "", 'channels_dest.txt', "", 'text/plain', "text", content: channels_dest_attachment.join("\n"))
+                        end
+                    end
+                end
+            else
+                message<<"Only Master admin users on a private conversation with the bot can see this kind of bot stats."
+            end
+        end
+        respond "#{message.join("\n")}", dest
+    end
+end

data/lib/slack/smart-bot/commands/{general → on_bot/general}/bot_status.rb

@@ -3,14 +3,13 @@ class SlackSmartBot
   # helpadmin: `bot status`
   # helpadmin:    Displays the status of the bot
   # helpadmin:    If on master channel and admin user also it will display info about bots created
+  # helpadmin:    <https://github.com/MarioRuiz/slack-smart-bot#bot-management|more info>
+  # helpadmin: command_id: :bot_status
   # helpadmin:
   def bot_status(dest, user)
     save_stats(__method__)
     get_bots_created()
-    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)
       gems_remote = `gem list slack-smart-bot --remote`
       version_remote = gems_remote.to_s().scan(/slack-smart-bot \((\d+\.\d+\.\d+)/).join
       version_message = ""

data/lib/slack/smart-bot/commands/on_bot/general/leaderboard.rb

@@ -0,0 +1,201 @@
+class SlackSmartBot
+    # help: ----------------------------------------------
+    # help: `leaderboard`
+    # help: `ranking`
+    # help: `podium`
+    # help: `leaderboard from YYYY/MM/DD`
+    # help: `leaderboard from YYYY/MM/DD to YYYY/MM/DD`
+    # help: `leaderboard PERIOD`
+    # help:    It will present some useful information about the use of the SmartBot in the period specified.
+    # help:    If no 'from' and 'to' specified then it will be considered 'last week'
+    # help:    PERIOD: today, yesterday, last week, this week, last month, this month, last year, this year
+    # help:    The results will exclude master admins and routines.
+    # help:    For a more detailed data use the command `bot stats`
+    # help:    Examples:
+    # help:      _leaderboard_
+    # help:      _podium from 2021/05/01_
+    # help:      _leaderboard from 2021/05/01 to 2021/05/31_
+    # help:      _ranking today_
+    # help:    <https://github.com/MarioRuiz/slack-smart-bot#bot-management|more info>
+    # help: command_id: :leaderboard
+    # help: 
+    
+    def leaderboard(from, to, period)
+        exclude_masters = true
+        exclude_routines = true
+        require 'csv'
+        if config.stats
+            message = []
+        else
+            message = ["You need to set stats to true to generate the stats when running the bot instance."]
+        end
+        save_stats(__method__)
+        if !File.exist?("#{config.stats_path}.#{Time.now.strftime("%Y-%m")}.log")
+            message<<'No stats'
+        else
+            if period == ''
+                message << "*Leaderboard SmartBot <##@channel_id> from #{from} to #{to}*\n"
+            else
+                message << "*Leaderboard SmartBot <##@channel_id> #{period}* (#{from} -> #{to})\n"
+            end
+            from_short = from
+            to_short = to
+            from_file = from[0..3] + '-' + from[5..6]
+            to_file = to[0..3] + '-' + to[5..6]
+            from+= " 00:00:00 +0000"
+            to+= " 23:59:59 +0000"
+            rows = []
+            users_id_name = {}
+            users_name_id = {}
+            count_users = {}
+            count_channels_dest = {}
+            count_commands_uniq_user = {}
+
+            # to translate global and enterprise users since sometimes was returning different names/ids
+            if from[0..3]=='2020' # this was an issue only on that period
+                Dir["#{config.stats_path}.*.log"].sort.each do |file|
+                    if file >= "#{config.stats_path}.#{from_file}.log" or file <= "#{config.stats_path}.#{to_file}.log"
+                        CSV.foreach(file, headers: true, header_converters: :symbol, converters: :numeric) do |row|
+                            unless users_id_name.key?(row[:user_id])
+                                users_id_name[row[:user_id]] = row[:user_name]
+                            end
+                            unless users_name_id.key?(row[:user_name])
+                                users_name_id[row[:user_name]] = row[:user_id]
+                            end
+    
+                        end
+                    end
+                end
+            end
+
+            master_admins = config.masters.dup
+            if users_id_name.size > 0
+                config.masters.each do |u|
+                    if users_id_name.key?(u)
+                        master_admins << users_id_name[u]
+                    elsif users_name_id.key?(u)
+                        master_admins << users_name_id[u]
+                    end
+                end
+            end
+
+            Dir["#{config.stats_path}.*.log"].sort.each do |file|
+                if file >= "#{config.stats_path}.#{from_file}.log" and file <= "#{config.stats_path}.#{to_file}.log"
+                    CSV.foreach(file, headers: true, header_converters: :symbol, converters: :numeric) do |row|
+                        row[:date] = row[:date].to_s
+                        if row[:dest_channel_id].to_s[0]=='D'
+                            row[:dest_channel] = 'DM'
+                        elsif row[:dest_channel].to_s == ''
+                            row[:dest_channel] = row[:dest_channel_id]
+                        end
+                        if users_name_id.size > 0
+                            row[:user_name] = users_id_name[row[:user_id]]
+                            row[:user_id] = users_name_id[row[:user_name]]
+                        else
+                            users_id_name[row[:user_id]] ||= row[:user_name]
+                        end
+                        if !exclude_masters or (exclude_masters and !master_admins.include?(row[:user_name]) and 
+                                                !master_admins.include?(row[:user_id]) and
+                                                !@master_admin_users_id.include?(row[:user_id]))
+                            if !exclude_routines or (exclude_routines and !row[:user_name].match?(/^routine\//) )
+                                if row[:bot_channel_id] == @channel_id
+                                    if row[:date] >= from and row[:date] <= to
+                                        count_users[row[:user_id]] ||= 0
+                                        count_users[row[:user_id]] += 1
+                                        rows << row.to_h
+                                        count_channels_dest[row[:dest_channel]] ||= 0
+                                        count_channels_dest[row[:dest_channel]] += 1
+                                        count_commands_uniq_user[row[:user_id]] ||= []
+                                        count_commands_uniq_user[row[:user_id]] << row[:command] unless count_commands_uniq_user[row[:user_id]].include?(row[:command])
+                                    end
+                                end
+                            end
+                        end
+                    end
+                end
+            end
+
+            total = rows.size
+
+            if total > 0
+
+                users = rows.user_id.uniq.sort
+                count_user = {}
+                users.each do |user|
+                    count = rows.count {|h| h.user_id==user}
+                    count_user[user] = count
+                end
+                mtc = nil
+                mtu = []
+                i = 0
+                count_user.sort_by {|k,v| -v}.each do |user, count|
+                    if i >= 3
+                        break
+                    elsif mtc.nil? or mtc == count or i < 3
+                        mtu << "*<@#{users_id_name[user]}>* (#{count})"
+                        mtc = count
+                    else 
+                        break
+                    end
+                    i+=1
+                end
+                message << "\t :boom: Users that called more commands: \n\t\t\t\t#{mtu.join("\n\t\t\t\t")}"
+
+                mtc = nil
+                mtu = []
+                i = 0
+                count_commands_uniq_user.sort_by {|k,v| -v.size}.each do |user, cmds|
+                    if i >= 3
+                        break
+                    elsif mtc.nil? or mtc == cmds.size or i < 3
+                        mtu << "*<@#{users_id_name[user]}>* (#{cmds.size})"
+                        mtc = cmds.size
+                    else 
+                        break
+                    end
+                    i+=1
+                end
+                message << "\t :stethoscope: Users that called more different commands: \n\t\t\t\t#{mtu.join("\n\t\t\t\t")}"
+                
+                commands_attachment = []
+
+                commands = rows.command.uniq.sort
+                count_command = {}
+                commands.each do |command|
+                    count = rows.count {|h| h.command==command}
+                    count_command[command] = count
+                end
+                
+                mtu = []
+                count_command.sort_by {|k,v| -v}[0..2].each do |command, count|
+                    mtu << "*`#{command.gsub('_',' ')}`* (#{count})"
+                end
+                message << "\t :four_leaf_clover: Most used commands: \n\t\t\t\t#{mtu.join("\n\t\t\t\t")}"
+
+                count_channels_dest = count_channels_dest.sort_by(&:last).reverse.to_h
+                count_channels_dest.keys[0..0].each do |ch|
+                    if ch=='DM'
+                        message << "\t :star: Most used channel: *DM* (#{(count_channels_dest[ch].to_f*100/total).round(2)}%)"
+                    else
+                        message << "\t :star: Most used channel: *<##{@channels_id[ch]}>* (#{(count_channels_dest[ch].to_f*100/total).round(2)}%)"
+                    end
+                end
+                
+                types = rows.type_message.uniq.sort
+                count_type = {}
+                types.each do |type|
+                    count = rows.count {|h| h.type_message==type}
+                    count_type[type] = count
+                end
+                
+                count_type.sort_by {|k,v| -v}[0..0].each do |type, count|
+                    message << "\t :house_with_garden: Most calls came from *#{type}* (#{(count.to_f*100/total).round(2)}%)"
+                end
+            else
+                message << 'No data yet'
+            end
+        end
+        respond "#{message.join("\n")}"
+
+    end
+end

data/lib/slack/smart-bot/commands/{general → on_bot/general}/stop_using_rules.rb

@@ -1,10 +1,14 @@
 class SlackSmartBot
   # help: ----------------------------------------------
   # help: `stop using rules from CHANNEL`
+  # help: `stop using CHANNEL`
   # help:    it will stop using the rules from the specified channel.
+  # help:    <https://github.com/MarioRuiz/slack-smart-bot#using-rules-from-other-channels|more info>
+  # help: command_id: :stop_using_rules
   # help:
   def stop_using_rules(dest, channel, user, dchannel)
     save_stats(__method__)
+    channel.gsub!('#','') # for the case the channel name is in plain text including #
     if @channels_id.key?(channel)
       channel_id = @channels_id[channel]
     else
@@ -12,11 +16,12 @@ class SlackSmartBot
     end
 
     if dest[0] == "C" or dest[0] == "G" #channel
-      if @rules_imported.key?(user.id) and @rules_imported[user.id].key?(dchannel)
-        if @rules_imported[user.id][dchannel] != channel_id
+      if @rules_imported.key?(user.name) and @rules_imported[user.name].key?(dchannel)
+        if @rules_imported[user.name][dchannel] != channel_id
           respond "You are not using those rules.", dest
         else
-          @rules_imported[user.id].delete(dchannel)
+          @rules_imported[user.name].delete(dchannel)
+          sleep 0.5
           update_rules_imported()
           respond "You won't be using those rules from now on.", dest
 
@@ -27,11 +32,12 @@ class SlackSmartBot
         respond "You were not using those rules.", dest
       end
     else #direct message
-      if @rules_imported.key?(user.id) and @rules_imported[user.id].key?(user.id)
-        if @rules_imported[user.id][user.id] != channel_id
+      if @rules_imported.key?(user.name) and @rules_imported[user.name].key?(user.name)
+        if @rules_imported[user.name][user.name] != channel_id
           respond "You are not using those rules.", dest
         else
-          @rules_imported[user.id].delete(user.id)
+          @rules_imported[user.name].delete(user.name)
+          sleep 0.5
           update_rules_imported()
           respond "You won't be using those rules from now on.", dest