Added the switch and refresh commands. One caveat is I noticed the switch command seemed to duplicate all the names on the ladder for some reason. Undo fixed it so it can be recovered but I wasnt able to reproduce or see how it could happen. My only hunch is the MMR was being updated at the same time but even that shouldnt break it.

Author: kirby561

Beef/Application.cs

@@ -10,7 +10,7 @@
 
 namespace Beef {
     class Application : ProfileInfoProvider, MmrListener {
-        private readonly String _version = "1.4.1";
+        private readonly String _version = "1.5";
         private BeefConfig _config;
         private String _botPrefix;
         private BeefUserConfigManager _userManager;
@@ -355,6 +355,31 @@ private void HandleCommand(SocketMessage userInput) {
 
                             MessageChannel(channel, "**" + playerOrRankToRename + "** has been renamed to **" + newName + "**").GetAwaiter().GetResult();
                         }
+                    } else if (arguments[1] == "switch") {
+                        if (!IsLeader(author)) {
+                            MessageChannel(channel, "You don't have permission to do that.").GetAwaiter().GetResult();
+                            return;
+                        }
+
+                        String playerOrRank1 = arguments[2];
+                        String playerOrRank2 = arguments[3];
+
+                        BeefEntry player1 = null;
+                        BeefEntry player2 = null;
+                        code = _presentationManager.SwitchPlayers(playerOrRank1, playerOrRank2, out player1, out player2);
+
+                        if (code.Ok()) {
+                            MessageChannel(channel, "**" + player1.PlayerName + "** has been switched with **" + player2.PlayerName + "**").GetAwaiter().GetResult();
+                        } else {
+                            if (code == ErrorCode.Player1DoesNotExist) {
+                                MessageChannel(channel, "**" + playerOrRank1 + "** isn't a valid rank or beef name.  When was the last time you had your eyes checked?").GetAwaiter().GetResult();
+                            } else if (code == ErrorCode.Player2DoesNotExist) {
+                                MessageChannel(channel, "**" + playerOrRank2 + "** isn't a valid rank or beef name.  When was the last time you had your eyes checked?").GetAwaiter().GetResult();
+                            } else {
+                                MessageChannel(channel, "Something went wrong.  I'm not sure what but I'm pretty sure it's your fault.").GetAwaiter().GetResult();
+                            }
+                            return;
+                        }
                     }
                 } else if (arguments.Length >= 2) {
                     if (arguments[1] == "version" && arguments.Length == 2) {
@@ -389,8 +414,6 @@ private void HandleCommand(SocketMessage userInput) {
                             MessageChannel(channel, "Removed **" + playerToRemove + "** from the ladder.").GetAwaiter().GetResult();
                         }
                     } else if (arguments[1] == "bracket" || arguments[1] == "list" || arguments[1] == "ladder") {
-                        IsLeader(userInput.Author);
-
                         List<BeefEntry> entries = _presentationManager.ReadBracket();
 
                         if (entries.Count == 0)
@@ -405,11 +428,25 @@ private void HandleCommand(SocketMessage userInput) {
                             }
                             // Send the help message to all if requested.  Otherwise
                             // just DM it to the user that asked.
-                            if (arguments.Length >= 3 && arguments[2] == "all")
+                            if (arguments.Length >= 3 && arguments[2] == "all") {
+                                if (!IsLeader(author)) {
+                                    MessageChannel(channel, "You don't have permission to do that.").GetAwaiter().GetResult();
+                                    return;
+                                }
+
                                 MessageChannel(channel, bracket).GetAwaiter().GetResult();
-                            else
+                            } else
                                 MessageUser(userInput.Author, bracket).GetAwaiter().GetResult();
                         }
+                    } else if (arguments[1].Equals("refresh")) {
+                        if (!IsLeader(author)) {
+                            MessageChannel(channel, "You don't have permission to do that.").GetAwaiter().GetResult();
+                            return;
+                        }
+
+                        _mmrReader.RequestUpdate();
+                        code = ErrorCode.Success;
+                        MessageChannel(channel, "MMR refresh requested.").GetAwaiter().GetResult();
                     } else if (arguments[1].Equals("undo")) {
                         if (!IsLeader(author)) {
                             MessageChannel(channel, "You don't have permission to do that.").GetAwaiter().GetResult();
@@ -454,6 +491,8 @@ private void HandleCommand(SocketMessage userInput) {
                         help += "\t **%beef% unregister <PlayerLadderName>** - Unregisters the given ladder name.\n";
                         help += "\t **%beef% link <PlayerLadderName> <PlayerBattleNetProfileLink>** - Links the given ladder name to the given Battle.net profile link.  This will enable their best MMR and race to be displayed on the ladder.\n";
                         help += "\t **%beef% unlink <PlayerLadderName>** - Unlinks the given ladder name from their associated Battle.net Profile.\n";
+                        help += "\t **%beef% switch <PlayerOrRank> <OtherPlayerOrRank>** - Switches the two players on the ladder leaving everyone else in place.\n";
+                        help += "\t **%beef% refresh** - Requests a refresh to the MMRs for each player.  Note that this can take a minute.\n";
                         help += "\t **%beef% undo** - Undoes the last change to the ladder (renames, wins, etc..).\n";
                         help += "\t **%beef% version** - Prints the version of BeefBot\n";
                         help = help.Replace("%beef%", _botPrefix + "beef");

Beef/ErrorCodes.cs

@@ -34,6 +34,10 @@ public enum ErrorCode {
         LadderDifferentSize,
         CouldNotRevertBackupFile,
 
+        // Switching
+        Player1DoesNotExist,
+        Player2DoesNotExist,
+
         // Commands
         CommandNotRecognized,
 

Beef/MmrReader/MmrReader.cs

@@ -78,7 +78,6 @@ private long GetNowInMs() {
         /// <returns>Returns the number of milliseconds until the next refresh or 0 if it should refresh now.</returns>
         private long GetTimeToNextRefresh(long nextRefreshTimeMs) {
             if (_manualUpdateRequested) {
-                _manualUpdateRequested = false;
                 return 0; // Update immediately if requested
             }
 
@@ -383,10 +382,12 @@ private void ReadMmrLoop() {
 
                 int timeToNextRefresh = (int)GetTimeToNextRefresh(nextRefreshTimeMs);
                 lock (_lock) {
-                    if (_manualUpdateRequested)
+                    if (_manualUpdateRequested) {
                         forceRefresh = true;
-                    else
+                        _manualUpdateRequested = false;
+                    } else {
                         Monitor.Wait(_lock, timeToNextRefresh);
+                    }
                 }
             }
 

Beef/PresentationManager.cs

@@ -296,6 +296,63 @@ private ErrorCode RenamePlayer(BeefEntry existingEntry, String newName) {
             return UpdatePresentationFromEntries(_entries, true);
         }
 
+        /// <summary>
+        /// Switches the 2 players in the beef ladder.
+        /// </summary>
+        /// <param name="nameOrRank1">A rank or beef name for the first player.</param>
+        /// <param name="nameOrRank2">A rank or beef name for the second player.</param>
+        /// <param name="player1">This is set to the BeefEntry if the first player is found.</param>
+        /// <param name="player2">This is set to the BeefEntry if the second player is found.</param>
+        /// <returns>Returns success if it succeeded or an error code identifying what happened.</returns>
+        public ErrorCode SwitchPlayers(String nameOrRank1, String nameOrRank2, out BeefEntry player1, out BeefEntry player2) {
+            // Init the out params
+            player1 = null;
+            player2 = null;
+
+            List<BeefEntry> entries = ReadBracket();
+            player1 = GetBeefEntryFromNameOrRank(nameOrRank1, entries);
+            if (player1 == null) {
+                return ErrorCode.Player1DoesNotExist;
+            }
+
+            player2 = GetBeefEntryFromNameOrRank(nameOrRank2, entries);
+            if (player2 == null) {
+                return ErrorCode.Player2DoesNotExist;
+            }
+
+            return SwitchPlayers(player1, player2);
+        }
+
+        private ErrorCode SwitchPlayers(BeefEntry player1, BeefEntry player2) {
+            // Assumes the beef ladder has already been updated
+            /*
+            List<BeefEntry> newList = new List<BeefEntry>(_entries.Count);
+            for (int i = 0; i < _entries.Count; i++) {
+                if (i == (player1.PlayerRank - 1)) {
+                    newList.Add(player2);
+                } else if (i == (player2.PlayerRank - 1)) {
+                    newList.Add(player1);
+                } else {
+                    newList.Add(_entries[i]);
+                }
+            }
+
+            int player1PrevRank = player1.PlayerRank;
+            String player1PrevObjectId = player1.
+            player1.PlayerRank = player2.PlayerRank;
+            player2.PlayerRank = player1PrevRank;
+
+            return UpdatePresentationFromEntries(newList, true);
+            */
+
+            String player1Name = player1.PlayerName;
+            String player2Name = player2.PlayerName;
+            player1.PlayerName = player2Name;
+            player2.PlayerName = player1Name;
+
+            return UpdatePresentationFromEntries(_entries, true);
+        }
+
         /// <summary>
         /// Removes the player from the ladder with the given name and shuffles everyone else down.
         /// </summary>
@@ -654,5 +711,32 @@ private String GetStringFromElements(IList<TextElement> elements) {
 
             return result;
         }
+
+        /// <summary>
+        /// Gets the BeefEntry corresponding to the given name or rank.
+        /// </summary>
+        /// <param name="nameOrRank">A Beef Name or rank on the ladder (RANK not INDEX)</param>
+        /// <param name="entries">The list of current entries to check</param>
+        /// <returns>Returns the entry that was found or null</returns>
+        private BeefEntry GetBeefEntryFromNameOrRank(String nameOrRank, List<BeefEntry> entries) {
+            int rank;
+            if (!int.TryParse(nameOrRank, out rank)) rank = -1;
+
+            if (rank == -1) {
+                foreach (BeefEntry entry in entries) {
+                    if (entry.PlayerName.Equals(nameOrRank)) {
+                        return entry;
+                    }
+                }
+                return null; // Not found
+            } else {
+                int index = rank - 1;
+                if (index < 0 || index >= entries.Count) {
+                    return null;
+                }
+
+                return entries[index];
+            }            
+        }
     }
 }