qwtf_discord_bot 5.4.5 → 5.4.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 55e755d887d064bcf3067b5360e2b2c30d0f88f6fffd0c3e43f7255ff392ee4f
-  data.tar.gz: 3efe1df2e2ecf4ba8e1af9099e5969e36d77c08d56e5422a21cef7332573ddf2
+  metadata.gz: c5bfd247ff3e811ab845a1cef86f043b93f54a6f30769885547d0ba1601c15ef
+  data.tar.gz: 72aebecdfaa233cd62e4d36b25c474a319dae5a1fa12d8ea8711536f23cf9549
 SHA512:
-  metadata.gz: 8509a4b5ea6b5c57fcf987524673b2b3f89488104e0680575b9e948ef4253d00aaeac67c02bdc29c7857bd9908dc253de8843a87e4ffb882fa61f6bdeaf23442
-  data.tar.gz: a4452e760d59de5175d52e90f7940653d1f5a595ed01f4b4e0a8249e3a5f07a929c1806d84e851f93c5b5e546a7906782b365b5f062c1e9bf9f2806641340335
+  metadata.gz: 89c358f811c6c0d17b31fb26e1a8d6234bf170f82bd5a820da5d8474052a42594f1adeafcd6fa387488147559370533a10d73bf50526bdfedb5b948325f78f7d
+  data.tar.gz: e2eb01bcd75ff2ab9251452c6cfac97b381d1199d5fddc5a146ee451def5e56747b4528e2977fd15e2d3d732a63cac03a1dcaa38a37cecaa6a75758075bdf770

data/README.md

@@ -90,14 +90,14 @@ This responds to discord messages:
   - `!join`
   - `!leave`
   - `!teamsize <no_of_players>`
-  - `!kick <@player>`
+  - `!kick <@player> [@player2]`
   - `!team <team_no>`
   - `!unteam`
   - `!addmap <map_name>`
   - `!removemap <map_name>`
   - `!maps`
-  - `!map <map_name>`
-  - `!choose`
+  - `!map [map_name]`
+  - `!choose [n]`
   - `!win <team_no>`
   - `!draw`
   - `!notify <@role>`

data/VERSION

@@ -1 +1 @@
-5.4.5
+5.4.10

data/lib/qwtf_discord_bot/qwtf_discord_bot_pug.rb

@@ -7,7 +7,7 @@ class QwtfDiscordBotPug # :nodoc:
   include QwtfDiscordBot
 
   MSG_SNIPPET_DELIMITER = ' · '
-  TEAM_NAMES = { 0 => "No team", 1 => "Blue", 2 => "Red" }
+  TEAM_NAMES = { 1 => "Blue", 2 => "Red" }
 
   def run
     bot = Discordrb::Commands::CommandBot.new(
@@ -26,12 +26,17 @@ class QwtfDiscordBotPug # :nodoc:
     )
 
     bot.command :help do |event, *args|
-      "Pug commands: `!status`, `!join`, `!team <team_no>`, `!unteam`, `!leave`, `!kick <@player>`, `!win <team_no>`, `!draw`, `!end`, `!teamsize <no_of_players>`, `!addmap <map_name>`, `!removemap <map_name>`, `!maps`, `!map <map_name>`, `!choose`, `!notify <@role>`"
+      "Pug commands: `!status`, `!join`, `!team <team_no> [@player1] [@player2]`, `!unteam`, `!leave`, `!kick <@player>`, `!win <team_no>`, `!draw`, `!end`, `!teamsize <no_of_players>`, `!addmap <map_name>`, `!removemap <map_name>`, `!maps`, `!map [map_name]`, `!choose [n]`, `!notify <@role>`"
     end
 
     bot.command :join do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message("You've already joined", e.channel) if pug.joined?(e.user_id)
+        if pug.joined?(e.user_id)
+          return send_embedded_message(
+            description: "You've already joined",
+            channel: e.channel
+          )
+        end
 
         join_pug(e, pug)
         start_pug(pug, e) if pug.full?
@@ -40,84 +45,95 @@ class QwtfDiscordBotPug # :nodoc:
 
     bot.command :choose do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message("Can't choose teams until PUG is full", e.channel) unless pug.full?
+        if pug.joined_players.count.odd?
+          return send_embedded_message(
+            description: "Can't choose teams with odd number of players",
+            channel: event.channel
+          )
+        end
 
-        pug.joined_players.each do |player_id|
-          pug.join_team(team_no: 0, player_id: player_id)
+        if args.any? && args.first.to_i < 1
+          return send_embedded_message(
+            description: "Choose a number higher than 0; e.g. `!choose 2`",
+            channel: e.channel
+          )
         end
 
-        start_pug(pug, e)
+        iteration = if args.any?
+                      args.first.to_i - 1
+                    else
+                      0
+                    end
+
+
+        message_obj = choose_fair_teams(pug: pug, event: e, iteration: iteration)
+        status(pug: pug, event: e, message_obj: message_obj) if message_obj
       end
     end
 
     bot.command :status do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message('No PUG has been started. `!join` to create', e.channel) unless pug.active?
-
-        footer = [
-          pug.game_map,
-          "#{pug.player_slots} joined",
-        ].compact.join(MSG_SNIPPET_DELIMITER)
-
-        send_embedded_message(nil, e.channel) do |embed|
-          embed.footer = Discordrb::Webhooks::EmbedFooter.new(
-            text: footer
+        if !pug.active?
+          return send_embedded_message(
+            description: "No PUG has been started. `!join` to create",
+            channel: e.channel
           )
-
-          pug.teams.each do |team_no, player_ids|
-            team_display_names = player_ids.map do |player_id|
-              e.display_name_for(player_id)
-            end
-
-            embed.add_field(
-              Discordrb::Webhooks::EmbedField.new(
-                {
-                  inline: true,
-                  name: team_name(team_no),
-                  value: team_display_names.join("\n")
-                }
-              )
-            )
-          end
         end
+
+        status(pug: pug, event: e)
       end
     end
 
     bot.command :teamsize do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message("Team size is #{pug.teamsize}", e.channel) unless args.any?
-
-        new_teamsize = args[0].to_i
-        return send_embedded_message('Team size should be a number higher than 0', e.channel) unless new_teamsize > 0
-
-        if new_teamsize
-          pug.teamsize = new_teamsize
-
-          send_embedded_message(
-            [
-              "Team size set to #{pug.teamsize}",
+        unless args.any?
+          return send_embedded_message(
+            description: [
+              "Each team has #{pug.teamsize} players",
               "#{pug.player_slots} joined"
             ].join(MSG_SNIPPET_DELIMITER),
-            e.channel
+            channel: e.channel
           )
+        end
 
-          start_pug(pug, e) if pug.full?
-        else
-          send_embedded_message(
-            [
-              "Current team size is #{pug.teamsize}",
-              "#{pug.player_slots} joined"
-            ].join(MSG_SNIPPET_DELIMITER),
-            e.channel
+        new_teamsize = args[0].to_i
+
+        if new_teamsize < 1
+          return send_embedded_message(
+            description: "Team size should be 1 or more",
+            channel: e.channel
           )
         end
+
+        pug.teamsize = new_teamsize
+
+        send_embedded_message(
+          description: [
+            "Each team has #{pug.teamsize} players",
+            "#{pug.player_slots} joined"
+          ].join(MSG_SNIPPET_DELIMITER),
+          channel: e.channel
+        )
+
+        start_pug(pug, e) if pug.full?
       end
     end
 
     bot.command :leave do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message(no_active_pug_message, e.channel) unless pug.active?
-        return send_embedded_message("You're not in the PUG", e.channel) unless pug.joined?(e.user_id)
+        unless pug.active?
+          return send_embedded_message(
+            description: no_active_pug_message,
+            channel: e.channel
+          )
+        end
+
+        unless pug.joined?(e.user_id)
+          return send_embedded_message(
+            description: "You're not in the PUG",
+            channel: e.channel
+          )
+        end
 
         pug.leave(e.user_id)
 
@@ -126,11 +142,12 @@ class QwtfDiscordBotPug # :nodoc:
           "#{pug.player_slots} remain"
         ]
 
-        snippets << "#{pug.slots_left} more #{pug.notify_roles}" if pug.slots_left == 1
+        message = "#{pug.slots_left} more #{pug.notify_roles}" if pug.slots_left == 1
 
         send_embedded_message(
-          snippets.join(MSG_SNIPPET_DELIMITER),
-          e.channel
+          message: message,
+          description: snippets.join(MSG_SNIPPET_DELIMITER),
+          channel: e.channel
         )
 
         end_pug(pug, e.channel) if pug.empty?
@@ -139,12 +156,26 @@ class QwtfDiscordBotPug # :nodoc:
 
     bot.command :kick do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message("Kick who? e.g. `!kick @#{e.display_name}`", e.channel) unless args.any?
-        return send_embedded_message(no_active_pug_message, e.channel) unless pug.active?
+        unless args.any?
+          return send_embedded_message(
+            description: "Kick who? e.g. `!kick @#{e.display_name}`",
+            channel: e.channel
+          )
+        end
+
+        unless pug.active?
+          return send_embedded_message(
+            description: no_active_pug_message,
+            channel: e.channel
+          )
+        end
 
         args.each do |arg|
           unless arg.match(/<@!\d+>/)
-            send_embedded_message("#{arg} isn't a valid mention", e.channel)
+            send_embedded_message(
+              description: "#{arg} isn't a valid mention",
+              channel: e.channel
+            )
             next
           end
 
@@ -152,7 +183,10 @@ class QwtfDiscordBotPug # :nodoc:
           display_name = e.display_name_for(user_id) || arg
 
           unless pug.joined?(user_id)
-            send_embedded_message("#{display_name} isn't in the PUG", e.channel)
+            send_embedded_message(
+              description: "#{display_name} isn't in the PUG",
+              channel: e.channel
+            )
             next
           end
 
@@ -163,11 +197,12 @@ class QwtfDiscordBotPug # :nodoc:
             "#{pug.player_slots} remain"
           ]
 
-          snippets << "#{pug.slots_left} more #{pug.notify_roles}" if pug.slots_left == 1
+          message = "#{pug.slots_left} more #{pug.notify_roles}" if pug.slots_left == 1
 
           send_embedded_message(
-            snippets.join(MSG_SNIPPET_DELIMITER),
-            e.channel
+            message: message,
+            description: snippets.join(MSG_SNIPPET_DELIMITER),
+            channel: e.channel
           )
 
           break end_pug(pug, e.channel) if pug.empty?
@@ -177,47 +212,63 @@ class QwtfDiscordBotPug # :nodoc:
 
     bot.command :team do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message("Which team? E.G. `!team 1`", e.channel) unless args.any?
-        return send_embedded_message("Choose a team between 0 and 2", e.channel) unless ["0", "1", "2"].any?(args.first)
+        unless args.any?
+          return send_embedded_message(
+            description: "Which team? E.G. `!team 1`",
+            channel: e.channel
+          )
+        end
+
+        unless ["1", "2"].any?(args.first)
+          return send_embedded_message(
+            description: "Choose `!team 1`, `!team 2`, or `!unteam` to leave team",
+            channel: e.channel
+          )
+        end
 
         team_no = args.first.to_i
         pug_already_full = pug.full?
 
         if args.count == 1
           user_id = e.user_id
-          return send_embedded_message("You're already in #{TEAM_NAMES[team_no]}", e.channel) if pug.team(team_no).include?(user_id)
+
+          if pug.team(team_no).include?(user_id)
+            return send_embedded_message(
+              description: "You're already in #{TEAM_NAMES[team_no]}",
+              channel: e.channel
+            )
+          end
 
           join_pug(e, pug) unless pug.joined?(user_id)
           pug.join_team(team_no: team_no, player_id: user_id)
 
           send_embedded_message(
-            [
+            description: [
               "#{e.display_name} joins #{TEAM_NAMES[team_no]}",
               "#{pug.team_player_count(team_no)}/#{pug.teamsize}"
-            ].join(MSG_SNIPPET_DELIMITER), e.channel
+            ].join(MSG_SNIPPET_DELIMITER),
+            channel: e.channel
           )
         else
           args[1..-1].each do |mention|
             unless mention.match(/<@!\d+>/)
-              send_embedded_message("#{arg} isn't a valid mention", e.channel)
+              send_embedded_message(
+                description: "#{arg} isn't a valid mention",
+                channel: e.channel
+              )
               next
             end
 
             user_id = mention_to_user_id(mention)
             display_name = e.display_name_for(user_id) || arg
-
-            unless pug.joined?(user_id)
-              send_embedded_message("#{display_name} isn't in the PUG", e.channel)
-              next
-            end
-
             pug.join_team(team_no: team_no, player_id: user_id)
 
             send_embedded_message(
-              [
+              description: [
                 "#{display_name} joins #{TEAM_NAMES[team_no]}",
                 "#{pug.team_player_count(team_no)}/#{pug.teamsize}"
-              ].join(MSG_SNIPPET_DELIMITER), e.channel
+              ].join(MSG_SNIPPET_DELIMITER),
+              channel: e.channel
             )
           end
         end
@@ -229,26 +280,66 @@ class QwtfDiscordBotPug # :nodoc:
     bot.command :unteam do |event, *args|
       setup_pug(event) do |e, pug|
         user_id = e.user_id
-        return send_embedded_message('No PUG has been started. `!join` to create', e.channel) unless pug.active?
-        return send_embedded_message("You aren't in this PUG", e.channel) unless pug.joined?(user_id)
-        return send_embedded_message("You aren't in a team", e.channel) if pug.team(0).include?(user_id)
+
+        unless pug.active?
+          return send_embedded_message(
+            description: 'No PUG has been started. `!join` to create',
+            channel: e.channel
+          )
+        end
+
+        unless pug.joined?(user_id)
+          return send_embedded_message(
+            description: "You aren't in this PUG",
+            channel: e.channel
+          )
+        end
+
+        if pug.team(0).include?(user_id)
+          return send_embedded_message(
+            description: "You aren't in a team",
+            channel: e.channel
+          )
+        end
 
         pug.join_team(team_no: 0, player_id: user_id)
-        send_embedded_message("#{e.display_name} has no team", e.channel)
+
+        send_embedded_message(
+          description: "#{e.display_name} leaves team",
+          channel: e.channel
+        )
       end
     end
 
     bot.command :win do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message(no_active_pug_message, e.channel) unless pug.active?
-        return send_embedded_message("Specify winning team; e.g. `!win 1`", e.channel) unless args.any?
-        return send_embedded_message("Invalid team number", e.channel) unless ["1", "2"].any?(args.first)
+        unless pug.active?
+          return send_embedded_message(
+            description: no_active_pug_message,
+            channel: e.channel
+          )
+        end
+
+        unless args.any?
+          return send_embedded_message(
+            description: "Specify winning team; e.g. `!win 1`",
+            channel: e.channel
+          )
+        end
+
+        unless ["1", "2"].any?(args.first)
+          return send_embedded_message(
+            description: "Invalid team number",
+            channel: e.channel
+          )
+        end
 
         winning_team_no = args.first.to_i
 
         if pug.actual_teams.count < 2
           return send_embedded_message(
-            "There must be at least two teams with players to submit a result", e.channel
+            description: "There must be at least two teams with players to submit a result",
+            channel: e.channel
           )
         end
 
@@ -274,17 +365,26 @@ class QwtfDiscordBotPug # :nodoc:
           }.to_json
         )
 
-        send_embedded_message("#{TEAM_NAMES[winning_team_no]} wins. [Ratings](http://ratings.fortressone.org)", e.channel)
+        send_embedded_message(
+          description: "#{TEAM_NAMES[winning_team_no]} wins. [Ratings](http://ratings.fortressone.org)",
+          channel: e.channel
+        )
       end
     end
 
     bot.command :draw do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message(no_active_pug_message, e.channel) unless pug.active?
+        unless pug.active?
+          return send_embedded_message(
+            description: no_active_pug_message,
+            channel: e.channel
+          )
+        end
 
         if pug.actual_teams.count < 2
           return send_embedded_message(
-            "There must be at least two teams with players to submit a result", e.channel
+            description: "There must be at least two teams with players to submit a result",
+            channel: e.channel
           )
         end
 
@@ -309,13 +409,21 @@ class QwtfDiscordBotPug # :nodoc:
           }.to_json
         )
 
-        send_embedded_message("Match drawn. [Ratings](http://ratings.fortressone.org)", e.channel)
+        send_embedded_message(
+          description: "Match drawn. [Ratings](http://ratings.fortressone.org)",
+          channel: e.channel
+        )
       end
     end
 
     bot.command :end do |event, *args|
       setup_pug(event) do |e, pug|
-        return send_embedded_message(no_active_pug_message, e.channel) unless pug.active?
+        unless pug.active?
+          return send_embedded_message(
+            description: no_active_pug_message,
+            channel: e.channel
+          )
+        end
 
         end_pug(pug, e.channel)
       end
@@ -324,47 +432,105 @@ class QwtfDiscordBotPug # :nodoc:
     bot.command :addmap do |event, *args|
       setup_pug(event) do |e, pug|
         maps = args
-        return send_embedded_message("What map? e.g. `!addmap 2fort5r`", e.channel) unless maps.any?
+
+        unless maps.any?
+          return send_embedded_message(
+            description: "What map? e.g. `!addmap 2fort5r`",
+            channel: e.channel
+          )
+        end
 
         pug.add_maps(maps)
-        send_embedded_message("#{maps.join(', ')} added to maps", e.channel)
+
+        send_embedded_message(
+          description: "#{maps.join(', ')} added to maps",
+          channel: e.channel)
       end
     end
 
     bot.command :removemap do |event, *args|
       setup_pug(event) do |e, pug|
         maps = args
-        return send_embedded_message("What map? e.g. `!removemap 2fort5r`", e.channel) unless maps.any?
+
+        unless maps.any?
+          return send_embedded_message(
+            description: "What map? e.g. `!removemap 2fort5r`",
+            channel: e.channel
+          )
+        end
 
         pug.remove_maps(maps)
-        send_embedded_message("#{maps.join(', ')} removed from maps", e.channel)
+
+        send_embedded_message(
+          description: "#{maps.join(', ')} removed from maps",
+          channel: e.channel
+        )
       end
     end
 
     bot.command :maps do |event, *args|
       setup_pug(event) do |e, pug|
         maps = pug.maps
-        return send_embedded_message('No maps have been added. `!addmap`', e.channel) unless maps.any?
+        unless maps.any?
+          return send_embedded_message(
+            description: 'No maps have been added. `!addmap`',
+            channel: e.channel
+          )
+        end
 
-        send_embedded_message(maps.join(', '), e.channel)
+        send_embedded_message(
+          description: maps.join(', '),
+          channel: e.channel
+        )
       end
     end
 
     bot.command :map do |event, *args|
       setup_pug(event) do |e, pug|
         maps = pug.maps
-        return send_embedded_message('No maps have been added. `!addmap`', e.channel) unless maps.any?
-        return send_embedded_message(no_active_pug_message, e.channel) unless pug.active?
+
+        unless maps.any?
+          return send_embedded_message(
+            description: 'No maps have been added. `!addmap`',
+            channel: e.channel
+          )
+        end
+
+        unless pug.active?
+          return send_embedded_message(
+            description: no_active_pug_message,
+            channel: e.channel
+          )
+        end
 
         if args.empty?
-          return send_embedded_message('No map has been set for the current PUG', e.channel) unless pug.game_map
-          send_embedded_message("Current map is #{pug.game_map}", e.channel)
+          unless pug.game_map
+            return send_embedded_message(
+              description: 'No map has been set for the current PUG',
+              channel: e.channel
+            )
+          end
+
+          send_embedded_message(
+            description: "Current map is #{pug.game_map}",
+            channel: e.channel
+          )
         else
           game_map = args.first
-          return send_embedded_message("#{game_map} isn't in the map list. `!addmap` to add it.", e.channel) unless maps.include?(game_map)
+
+          unless maps.include?(game_map)
+            return send_embedded_message(
+              description: "#{game_map} isn't in the map list. `!addmap` to add it.",
+              channel: e.channel
+            )
+          end
 
           pug.game_map = game_map
-          send_embedded_message("Map set to #{game_map}", e.channel)
+
+          send_embedded_message(
+            description: "Map set to #{game_map}",
+            channel: e.channel
+          )
         end
       end
     end
@@ -380,7 +546,10 @@ class QwtfDiscordBotPug # :nodoc:
                 "Notification role set to #{roles}"
               end
 
-        send_embedded_message(msg, e.channel)
+        send_embedded_message(
+          description: msg,
+          channel: e.channel
+        )
       end
     end
 
@@ -390,6 +559,8 @@ class QwtfDiscordBotPug # :nodoc:
   private
 
   def team_name(team_no)
+    return "No team" if team_no == 0
+
     [team_no, TEAM_NAMES[team_no]].join(" ")
   end
 
@@ -401,13 +572,18 @@ class QwtfDiscordBotPug # :nodoc:
     pug.join(e.user_id)
 
     if pug.joined_player_count == 1
-      snippets = ["#{e.display_name} creates a PUG", pug.player_slots, pug.notify_roles]
+      snippets = ["#{e.display_name} creates a PUG", "#{pug.player_slots} joined"]
+      message = pug.notify_roles
     else
-      snippets = ["#{e.display_name} joins the PUG", pug.player_slots]
-      snippets << "#{pug.slots_left} more #{pug.notify_roles}" if pug.slots_left.between?(1, 3)
+      snippets = ["#{e.display_name} joins the PUG", "#{pug.player_slots} joined"]
+      message = "#{pug.slots_left} more #{pug.notify_roles}" if pug.slots_left.between?(1, 3)
     end
 
-    send_embedded_message(snippets.join(MSG_SNIPPET_DELIMITER), e.channel)
+    send_embedded_message(
+      message: message,
+      description: snippets.join(MSG_SNIPPET_DELIMITER),
+      channel: e.channel
+    )
   end
 
   def setup_pug(event)
@@ -417,31 +593,97 @@ class QwtfDiscordBotPug # :nodoc:
     nil # stop discordrb printing return value
   end
 
-  def start_pug(pug, event)
-    if !pug.actual_teams.any?
-      send_embedded_message("Choosing fair teams...", event.channel)
-      teams = get_fair_teams(pug.joined_players)
+  def choose_fair_teams(pug:, event:, iteration: 0)
+    if !pug.full?
+      return send_embedded_message(
+        description: "Can't choose teams until PUG is full",
+        channel: event.channel
+      ) && nil
+    end
+
+    message_obj = send_embedded_message(
+      description: "Choosing fair teams...",
+      channel: event.channel
+    )
+
+    combinations = get_fair_teams(pug.joined_players)
+    teams = combinations[iteration]
+
+    if !teams
+      return send_embedded_message(
+        description: "There are only #{combinations.count} possible combinations",
+        channel: event.channel,
+        message_obj: message_obj
+      ) && nil
+    end
+
+    teams.each do |team_no, player_ids|
+      player_ids.each do |player_id|
+        pug.join_team(team_no: team_no, player_id: player_id)
+      end
+    end
 
-      teams.each do |team_no, player_ids|
-        player_ids.each do |player_id|
-          pug.join_team(team_no: team_no, player_id: player_id)
+    message_obj
+  end
+
+  def status(pug:, event:, message_obj: nil)
+    footer = [
+      pug.game_map,
+      "#{pug.player_slots} joined"
+    ].compact.join(MSG_SNIPPET_DELIMITER)
+
+    send_embedded_message(
+      description: nil,
+      channel: event.channel,
+      message_obj: message_obj
+    ) do |embed|
+      embed.footer = Discordrb::Webhooks::EmbedFooter.new(
+        text: footer
+      )
+
+      pug.teams.each do |team_no, player_ids|
+        team_display_names = player_ids.map do |player_id|
+          event.display_name_for(player_id)
         end
+
+        embed.add_field(
+          Discordrb::Webhooks::EmbedField.new(
+            {
+              inline: true,
+              name: team_name(team_no),
+              value: team_display_names.join("\n")
+            }
+          )
+        )
       end
     end
+  end
+
+  def start_pug(pug, event)
+    choose_fair_teams(pug: pug, event: event) unless pug.actual_teams.any?
 
     footer = [
       pug.game_map,
       "#{pug.player_slots} joined",
     ].compact.join(MSG_SNIPPET_DELIMITER)
 
-    send_embedded_message("Time to play!", event.channel) do |embed|
+    mentions = pug.joined_players.map do |player_id|
+      event.mention_for(player_id)
+    end
+
+    mention_line = "Time to play! #{mentions.join(" ")}"
+
+    send_embedded_message(
+      message: mention_line,
+      channel: event.channel
+    ) do |embed|
       embed.footer = Discordrb::Webhooks::EmbedFooter.new(
         text: footer
       )
 
       pug.teams.each do |team_no, player_ids|
         team_mentions = player_ids.map do |player_id|
-          event.mention_for(player_id)
+          event.display_name_for(player_id)
         end
 
         embed.add_field(
@@ -459,24 +701,38 @@ class QwtfDiscordBotPug # :nodoc:
 
   def end_pug(pug, channel_id)
     pug.end_pug
-    send_embedded_message('PUG ended', channel_id)
+
+    send_embedded_message(
+      description: 'PUG ended',
+      channel: channel_id
+    )
   end
 
   def no_active_pug_message
     "There's no active PUG"
   end
 
-  def send_embedded_message(message, channel)
+  def send_embedded_message(message: nil, description: nil, channel:, message_obj: nil)
     embed = Discordrb::Webhooks::Embed.new
-    embed.description = message
+    embed.description = description
     yield(embed) if block_given?
-    channel.send_embed(nil, embed) && puts(message)
+
+    if message_obj
+      message_obj.edit(message, embed).tap do
+        puts(message)
+      end
+    else
+      channel.send_embed(message, embed).tap do
+        puts(message)
+      end
+    end
   end
 
   def post_results(json)
     uri = URI("#{ENV['RATINGS_API_URL']}matches/")
     req = Net::HTTP::Post.new(uri, 'Content-Type' => 'application/json')
     req.body = json
+
     Net::HTTP.start(uri.hostname, uri.port) do |http|
       http.request(req)
     end
@@ -487,9 +743,11 @@ class QwtfDiscordBotPug # :nodoc:
     params = { 'players[]' => players }
     uri.query = URI.encode_www_form(params)
     req = Net::HTTP::Get.new(uri)
+
     res = Net::HTTP.start(uri.hostname, uri.port) do |http|
       http.request(req)
     end
-    JSON.parse(res.body).first.to_h
+
+    JSON.parse(res.body).map(&:to_h)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: qwtf_discord_bot
 version: !ruby/object:Gem::Version
-  version: 5.4.5
+  version: 5.4.10
 platform: ruby
 authors:
 - Sheldon Johnson
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-10-26 00:00:00.000000000 Z
+date: 2020-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: discordrb