Ticket #3383: lobby_coding_convention_v1.patch

binaries/data/mods/public/gui/lobby/lobby.js

@@ +1 @@
+const g_MapTypesText = [translateWithContext("map", "Skirmish"), translateWithContext("map", "Random"), translate("Scenario")];
+const g_MapTypes = ["skirmish", "random", "scenario"];
+const g_ShowTimestamp = Engine.ConfigDB_GetValue("user", "lobby.chattimestamp") == "true";
+const g_ModPrefix = "@";
+const g_SpecialKey = Math.random();
+const g_SpamBlockLength = 30; // Block spammers for 30 seconds.
+
 var g_ChatMessages = [];
 var g_Name = "unknown";
 var g_GameList = {}
 var g_GameListSortBy = "name";
 var g_PlayerListSortBy = "name";
 var g_GameListOrder = 1; // 1 for ascending sort, and -1 for descending
 var g_PlayerListOrder = 1;
-var g_specialKey = Math.random();
 // This object looks like {"name":[numMessagesSinceReset, lastReset, timeBlocked]} when in use.
-var g_spamMonitor = {};
-var g_timestamp = Engine.ConfigDB_GetValue("user", "lobby.chattimestamp") == "true";
-var g_mapSizes = {};
-const g_mapTypesText = [translateWithContext("map", "Skirmish"), translateWithContext("map", "Random"), translate("Scenario")];
-const g_mapTypes = ["skirmish", "random", "scenario"];
-var g_userRating = ""; // Rating of user, defaults to Unrated
-var g_modPrefix = "@";
-// Block spammers for 30 seconds.
-var SPAM_BLOCK_LENGTH = 30;
+var g_SpamMonitor = {};
+var g_MapSizes = {};
+var g_UserRating = ""; // Rating of user, defaults to Unrated
 
 ////////////////////////////////////////////////////////////////////////////////////////////////
 
 function init(attribs)
 {
@@ -25 +25 @@
     initMusic();
     global.music.setState(global.music.states.MENU);
 
     g_Name = Engine.LobbyGetNick();
 
-    g_mapSizes = initMapSizes();
-    g_mapSizes.shortNames.splice(0, 0, translateWithContext("map size", "Any"));
-    g_mapSizes.tiles.splice(0, 0, "");
+    g_MapSizes = initMapSizes();
+    g_MapSizes.shortNames.splice(0, 0, translateWithContext("map size", "Any"));
+    g_MapSizes.tiles.splice(0, 0, "");
 
     var mapSizeFilter = Engine.GetGUIObjectByName("mapSizeFilter");
-    mapSizeFilter.list = g_mapSizes.shortNames;
-    mapSizeFilter.list_data = g_mapSizes.tiles;
+    mapSizeFilter.list = g_MapSizes.shortNames;
+    mapSizeFilter.list_data = g_MapSizes.tiles;
 
     var playersNumberFilter = Engine.GetGUIObjectByName("playersNumberFilter");
     playersNumberFilter.list = [translateWithContext("player number", "Any"),2,3,4,5,6,7,8];
     playersNumberFilter.list_data = ["",2,3,4,5,6,7,8];
 
     var mapTypeFilter = Engine.GetGUIObjectByName("mapTypeFilter");
-    mapTypeFilter.list = [translateWithContext("map", "Any")].concat(g_mapTypesText);
-    mapTypeFilter.list_data = [""].concat(g_mapTypes);
+    mapTypeFilter.list = [translateWithContext("map", "Any")].concat(g_MapTypesText);
+    mapTypeFilter.list_data = [""].concat(g_MapTypes);
 
     Engine.LobbySetPlayerPresence("available");
     Engine.SendGetGameList();
     Engine.SendGetBoardList();
     updatePlayerList();
@@ -208 +208 @@
     });
     for (var i = 0; i < cleanPlayerList.length; i++)
     {
         // Identify current user's rating.
         if (cleanPlayerList[i].name == g_Name && cleanPlayerList[i].rating)
-            g_userRating = cleanPlayerList[i].rating;
+            g_UserRating = cleanPlayerList[i].rating;
         // Add a "-" for unrated players.
         if (!cleanPlayerList[i].rating)
             cleanPlayerList[i].rating = "-";
         // Colorize.
         var [name, status, rating] = formatPlayerListEntry(cleanPlayerList[i].name, cleanPlayerList[i].presence, cleanPlayerList[i].rating, cleanPlayerList[i].role);
@@ -236 +236 @@
 
 /**
  * Display the profile of the selected player.
  * Displays N/A for all stats until updateProfile is called when the stats
  *  are actually received from the bot.
- * 
+ *
  * @param caller From which screen is the user requesting data from?
  */
 function displayProfile(caller)
 {
     var playerList, rating;
@@ -261 +261 @@
         Engine.GetGUIObjectByName("profileArea").hidden = true;
         return;
     }
     Engine.GetGUIObjectByName("profileArea").hidden = false;
 
-    Engine.SendGetProfile(playerList.list[playerList.selected]);    
+    Engine.SendGetProfile(playerList.list[playerList.selected]);
 
     var user = playerList.list_name[playerList.selected];
     var role = Engine.LobbyGetPlayerRole(playerList.list[playerList.selected]);
     var userList = Engine.GetGUIObjectByName("playersBox");
     if (role && caller == "lobbylist")
@@ -306 +306 @@
             Engine.GetGUIObjectByName("profileErrorText").hidden = false;
             return;
         }
         Engine.GetGUIObjectByName("profileWindowArea").hidden = false;
         Engine.GetGUIObjectByName("profileErrorText").hidden = true;
-        
+
         if (attributes[0].rating != "")
-            user = sprintf(translate("%(nick)s (%(rating)s)"), { nick: user, rating: attributes[0].rating });
+            user = sprintf(translate("%(nick)s (%(rating)s)"), { "nick": user, "rating": attributes[0].rating });
 
         Engine.GetGUIObjectByName("profileUsernameText").caption = user;
         Engine.GetGUIObjectByName("profileRankText").caption = attributes[0].rank;
         Engine.GetGUIObjectByName("profileHighestRatingText").caption = attributes[0].highestRating;
         Engine.GetGUIObjectByName("profileTotalGamesText").caption = attributes[0].totalGamesPlayed;
         Engine.GetGUIObjectByName("profileWinsText").caption = attributes[0].wins;
         Engine.GetGUIObjectByName("profileLossesText").caption = attributes[0].losses;
 
         var winRate = (attributes[0].wins / attributes[0].totalGamesPlayed * 100).toFixed(2);
         if (attributes[0].totalGamesPlayed != 0)
-            Engine.GetGUIObjectByName("profileRatioText").caption = sprintf(translate("%(percentage)s%%"), { percentage: winRate });
+            Engine.GetGUIObjectByName("profileRatioText").caption = sprintf(translate("%(percentage)s%%"), { "percentage": winRate });
         else
             Engine.GetGUIObjectByName("profileRatioText").caption = translateWithContext("Used for an undefined winning rate", "-");
         return;
     }
     else if (!Engine.GetGUIObjectByName("leaderboard").hidden)
         playerList = Engine.GetGUIObjectByName("leaderboardBox");
     else
         playerList = Engine.GetGUIObjectByName("playersBox");
-    
+
     if (attributes[0].rating == "-2")
         return;
     // Make sure the stats we have received coincide with the selected player.
     if (attributes[0].player != playerList.list[playerList.selected])
         return;
     user = playerList.list_name[playerList.selected];
     if (attributes[0].rating != "")
-        user = sprintf(translate("%(nick)s (%(rating)s)"), { nick: user, rating: attributes[0].rating });
+        user = sprintf(translate("%(nick)s (%(rating)s)"), { "nick": user, "rating": attributes[0].rating });
 
     Engine.GetGUIObjectByName("usernameText").caption = user;
     Engine.GetGUIObjectByName("rankText").caption = attributes[0].rank;
     Engine.GetGUIObjectByName("highestRatingText").caption = attributes[0].highestRating;
     Engine.GetGUIObjectByName("totalGamesText").caption = attributes[0].totalGamesPlayed;
     Engine.GetGUIObjectByName("winsText").caption = attributes[0].wins;
     Engine.GetGUIObjectByName("lossesText").caption = attributes[0].losses;
 
     var winRate = (attributes[0].wins / attributes[0].totalGamesPlayed * 100).toFixed(2);
     if (attributes[0].totalGamesPlayed != 0)
-        Engine.GetGUIObjectByName("ratioText").caption = sprintf(translate("%(percentage)s%%"), { percentage: winRate });
+        Engine.GetGUIObjectByName("ratioText").caption = sprintf(translate("%(percentage)s%%"), { "percentage": winRate });
     else
         Engine.GetGUIObjectByName("ratioText").caption = translateWithContext("Used for an undefined winning rate", "-");
 }
 
 /**
@@ -467 +467 @@
                 name = '[color="255 0 0"]' + name + '[/color]';
             list_name.push(name);
             list_ip.push(g.ip);
             list_mapName.push(translate(g.niceMapName));
             list_mapSize.push(translatedMapSize(g.mapSize));
-            let idx = g_mapTypes.indexOf(g.mapType);
-            list_mapType.push(idx != -1 ? g_mapTypesText[idx] : "");
+            let idx = g_MapTypes.indexOf(g.mapType);
+            list_mapType.push(idx != -1 ? g_MapTypesText[idx] : "");
             list_nPlayers.push(g.nbp + "/" +g.tnbp);
             list.push(name);
             list_data.push(c);
         }
         c++;
@@ -524 +524 @@
     case "offline":
         color = "0 0 0";
         status = translate("Offline");
         break;
     default:
-        warn(sprintf("Unknown presence '%(presence)s'", { presence: presence }));
+        warn(sprintf("Unknown presence '%(presence)s'", { "presence": presence }));
         color = "178 178 178";
         status = translateWithContext("lobby presence", "Unknown");
         break;
     }
     // Center the unrated symbol.
@@ -536 +536 @@
         rating = "    -";
     var formattedStatus = '[color="' + color + '"]' + status + "[/color]";
     var formattedRating = '[color="' + color + '"]' + rating + "[/color]";
     var role = Engine.LobbyGetPlayerRole(nickname);
     if (role == "moderator")
-        nickname = g_modPrefix + nickname;
+        nickname = g_ModPrefix + nickname;
     var formattedName = colorPlayerName(nickname);
 
     // Push this player's name and status onto the list
     return [formattedName, formattedStatus, formattedRating];
 }
@@ -552 +552 @@
 function translatedMapSize(mapSize)
 {
     if (+mapSize !== +mapSize) // NaN
         return translate(mapSize);
     else
-        return g_mapSizes.shortNames[g_mapSizes.tiles.indexOf(+mapSize)];
+        return g_MapSizes.shortNames[g_MapSizes.tiles.indexOf(+mapSize)];
 }
 
 /**
  * Populate the game info area with information on the current game selection.
  */
@@ -575 +575 @@
     var mapData;
     var g = Engine.GetGUIObjectByName("gamesBox").list_data[selected];
 
     // Load map data
     if (g_GameList[g].mapType == "random" && g_GameList[g].mapName == "random")
-        mapData = {"settings": {"Description": translate("A randomly selected map.")}};
+        mapData = { "settings": { "Description": translate("A randomly selected map.") } };
     else if (g_GameList[g].mapType == "random" && Engine.FileExists(g_GameList[g].mapName + ".json"))
         mapData = Engine.ReadJSONFile(g_GameList[g].mapName + ".json");
     else if (Engine.FileExists(g_GameList[g].mapName + ".xml"))
         mapData = Engine.LoadMapSettings(g_GameList[g].mapName + ".xml");
     else
-        // Warn the player if we can't find the map. 
-        warn(sprintf("Map '%(mapName)s' not found locally.", { mapName: g_GameList[g].mapName }));
+        // Warn the player if we can't find the map.
+        warn(sprintf("Map '%(mapName)s' not found locally.", { "mapName": g_GameList[g].mapName }));
 
     // Show the game info panel and join button.
     Engine.GetGUIObjectByName("gameInfo").hidden = false;
     Engine.GetGUIObjectByName("joinGameButton").hidden = false;
     Engine.GetGUIObjectByName("gameInfoEmpty").hidden = true;
@@ -594 +594 @@
     // Display the map name, number of players, the names of the players, the map size and the map type.
     Engine.GetGUIObjectByName("sgMapName").caption = translate(g_GameList[g].niceMapName);
     Engine.GetGUIObjectByName("sgNbPlayers").caption = g_GameList[g].nbp + "/" + g_GameList[g].tnbp;
     Engine.GetGUIObjectByName("sgPlayersNames").caption = g_GameList[g].players;
     Engine.GetGUIObjectByName("sgMapSize").caption = translatedMapSize(g_GameList[g].mapSize);
-    let idx = g_mapTypes.indexOf(g_GameList[g].mapType);
-    Engine.GetGUIObjectByName("sgMapType").caption = idx != -1 ? g_mapTypesText[idx] : "";
+    let idx = g_MapTypes.indexOf(g_GameList[g].mapType);
+    Engine.GetGUIObjectByName("sgMapType").caption = idx != -1 ? g_MapTypesText[idx] : "";
 
     // Display map description if it exists, otherwise display a placeholder.
     if (mapData && mapData.settings.Description)
         var mapDescription = translate(mapData.settings.Description);
     else
@@ -629 +629 @@
 
         // TODO: What about valid host names?
         // Check if it looks like an ip address
         if (sip.split('.').length != 4)
         {
-            addChatMessage({ "from": "system", "text": sprintf(translate("This game's address '%(ip)s' does not appear to be valid."), { ip: sip }) });
+            addChatMessage({ "from": "system", "text": sprintf(translate("This game's address '%(ip)s' does not appear to be valid."), { "ip": sip }) });
             return;
         }
 
         // Open Multiplayer connection window with join option.
-        Engine.PushGuiPage("page_gamesetup_mp.xml", { multiplayerGameType: "join", name: sname, ip: sip, rating: g_userRating });
+        Engine.PushGuiPage("page_gamesetup_mp.xml", { "multiplayerGameType": "join", "name": sname, "ip": sip, "rating": g_UserRating });
     }
 }
 
 /**
  * Start the hosting process.
  */
 function hostGame()
 {
     // Open Multiplayer connection window with host option.
-    Engine.PushGuiPage("page_gamesetup_mp.xml", { multiplayerGameType: "host", name: g_Name, rating: g_userRating });
+    Engine.PushGuiPage("page_gamesetup_mp.xml", { "multiplayerGameType": "host", "name": g_Name, "rating": g_UserRating });
 }
 
 ////////////////////////////////////////////////////////////////////////////////////////////////
 // Utils
 ////////////////////////////////////////////////////////////////////////////////////////////////
@@ -700 +700 @@
                 var [name, status, rating] = formatPlayerListEntry(nick, presence, "-");
                 playerList.push(name);
                 presenceList.push(status);
                 nickList.push(nick);
                 ratingList.push(String(rating));
-                addChatMessage({ "text": "/special " + sprintf(translate("%(nick)s has joined."), { nick: nick }), "key": g_specialKey });
+                addChatMessage({ "text": "/special " + sprintf(translate("%(nick)s has joined."), { "nick": nick }), "key": g_SpecialKey });
                 break;
             case "leave":
                 if (nickIndex == -1) // Left, but not present (TODO: warn about this?)
                     break;
                 playerList.splice(nickIndex, 1);
                 presenceList.splice(nickIndex, 1);
                 nickList.splice(nickIndex, 1);
                 ratingList.splice(nickIndex, 1);
-                addChatMessage({ "text": "/special " + sprintf(translate("%(nick)s has left."), { nick: nick }), "key": g_specialKey });
+                addChatMessage({ "text": "/special " + sprintf(translate("%(nick)s has left."), { "nick": nick }), "key": g_SpecialKey });
                 break;
             case "nick":
                 if (nickIndex == -1) // Changed nick, but not present (shouldn't ever happen)
                     break;
                 if (!isValidNick(message.data))
                 {
-                    addChatMessage({ "from": "system", "text": sprintf(translate("Invalid nickname: %(nick)s"), { nick: message.data })});
+                    addChatMessage({ "from": "system", "text": sprintf(translate("Invalid nickname: %(nick)s"), { "nick": message.data }) });
                     break;
                 }
                 var [name, status, rating] = formatPlayerListEntry(message.data, presence, stripColorCodes(ratingList[nickIndex])); // TODO: actually we don't want to change the presence here, so use what was used before
                 playerList[nickIndex] = name;
                 // presence stays the same
                 nickList[nickIndex] = message.data;
-                addChatMessage({ "text": "/special " + sprintf(translate("%(oldnick)s is now known as %(newnick)s."), { oldnick: nick, newnick: message.data }), "key": g_specialKey });
+                addChatMessage({ "text": "/special " + sprintf(translate("%(oldnick)s is now known as %(newnick)s."), { "oldnick": nick, "newnick": message.data }), "key": g_SpecialKey });
                 break;
             case "presence":
                 if (nickIndex == -1) // Changed presence, but not online (shouldn't ever happen)
                     break;
                 var [name, status, rating] = formatPlayerListEntry(nick, presence, stripColorCodes(ratingList[nickIndex]));
@@ -737 +737 @@
                 break;
             case "subject":
                 updateSubject(message.text);
                 break;
             default:
-                warn(sprintf("Unknown message.level '%(msglvl)s'", { msglvl: message.level }));
+                warn(sprintf("Unknown message.level '%(msglvl)s'", { "msglvl": message.level }));
                 break;
             }
             // Push new data to GUI
             playersBox.list_name = playerList;
             playersBox.list_status = presenceList;
             playersBox.list_rating = ratingList;
-            playersBox.list = nickList;     
+            playersBox.list = nickList;
             if (playersBox.selected >= playersBox.list.length)
                 playersBox.selected = -1;
             break;
         case "system":
             switch (message.level)
@@ -790 +790 @@
                 }
                 break;
             }
             break;
         default:
-            error(sprintf("Unrecognised message type %(msgtype)s", { msgtype: message.type }));
+            error(sprintf("Unrecognised message type %(msgtype)s", { "msgtype": message.type }));
         }
     }
 }
 
-/* Messages */
 function submitChatInput()
 {
     var input = Engine.GetGUIObjectByName("chatInput");
     var text = input.caption;
     if (text.length)
@@ -850 +849 @@
         break;
     case "say":
     case "me":
         return false;
     default:
-        addChatMessage({ "from":"system", "text": sprintf(translate("We're sorry, the '%(cmd)s' command is not supported."), { cmd: cmd})});
+        addChatMessage({ "from":"system", "text": sprintf(translate("We're sorry, the '%(cmd)s' command is not supported."), { "cmd": cmd }) });
     }
     return true;
 }
 
 /**
@@ -868 +867 @@
     if (msg.from)
     {
         // Display the moderator symbol in the chatbox.
         var playerRole = Engine.LobbyGetPlayerRole(msg.from);
         if (playerRole == "moderator")
-            msg.from = g_modPrefix + msg.from;
+            msg.from = g_ModPrefix + msg.from;
     }
     else
         msg.from = null;
     if (!msg.color)
         msg.color = null;
@@ -934 +933 @@
         var [command, message] = ircSplit(text);
         switch (command)
         {
             case "me":
                 // Translation: IRC message prefix when the sender uses the /me command.
-                var senderString = '[font="sans-bold-13"]' + sprintf(translate("* %(sender)s"), { sender: coloredFrom }) + '[/font]';
+                var senderString = '[font="sans-bold-13"]' + sprintf(translate("* %(sender)s"), { "sender": coloredFrom }) + '[/font]';
                 // Translation: IRC message issued using the ‘/me’ command.
-                var formattedMessage = sprintf(translate("%(sender)s %(action)s"), { sender: senderString, action: message });
+                var formattedMessage = sprintf(translate("%(sender)s %(action)s"), { "sender": senderString, "action": message });
                 break;
             case "say":
                 // Translation: IRC message prefix.
-                var senderString = '[font="sans-bold-13"]' + sprintf(translate("<%(sender)s>"), { sender: coloredFrom }) + '[/font]';
+                var senderString = '[font="sans-bold-13"]' + sprintf(translate("<%(sender)s>"), { "sender": coloredFrom }) + '[/font]';
                 // Translation: IRC message.
-                var formattedMessage = sprintf(translate("%(sender)s %(message)s"), { sender: senderString, message: message });
+                var formattedMessage = sprintf(translate("%(sender)s %(message)s"), { "sender": senderString, "message": message });
                 break
             case "special":
-                if (key === g_specialKey)
+                if (key === g_SpecialKey)
                     // Translation: IRC system message.
-                    var formattedMessage = '[font="sans-bold-13"]' + sprintf(translate("== %(message)s"), { message: message }) + '[/font]';
+                    var formattedMessage = '[font="sans-bold-13"]' + sprintf(translate("== %(message)s"), { "message": message }) + '[/font]';
                 else
                 {
                     // Translation: IRC message prefix.
-                    var senderString = '[font="sans-bold-13"]' + sprintf(translate("<%(sender)s>"), { sender: coloredFrom }) + '[/font]';
+                    var senderString = '[font="sans-bold-13"]' + sprintf(translate("<%(sender)s>"), { "sender": coloredFrom }) + '[/font]';
                     // Translation: IRC message.
-                    var formattedMessage = sprintf(translate("%(sender)s %(message)s"), { sender: senderString, message: message });
+                    var formattedMessage = sprintf(translate("%(sender)s %(message)s"), { "sender": senderString, "message": message });
                 }
                 break;
             default:
                 // This should never happen.
                 var formattedMessage = "";
         }
     }
     else
     {
         // Translation: IRC message prefix.
-        var senderString = '[font="sans-bold-13"]' + sprintf(translate("<%(sender)s>"), { sender: coloredFrom }) + '[/font]';
+        var senderString = '[font="sans-bold-13"]' + sprintf(translate("<%(sender)s>"), { "sender": coloredFrom }) + '[/font]';
         // Translation: IRC message.
-        var formattedMessage = sprintf(translate("%(sender)s %(message)s"), { sender: senderString, message: text });
+        var formattedMessage = sprintf(translate("%(sender)s %(message)s"), { "sender": senderString, "message": text });
     }
 
     // Build time header if enabled
-    if (g_timestamp)
+    if (g_ShowTimestamp)
     {
 
         var time;
         if (datetime)
         {
@@ -991 +990 @@
         // For a list of symbols that you can use, see:
         // https://sites.google.com/site/icuprojectuserguide/formatparse/datetime?pli=1#TOC-Date-Field-Symbol-Table
         var timeString = Engine.FormatMillisecondsIntoDateString(time.getTime(), translate("HH:mm"));
 
         // Translation: Time prefix as shown in the multiplayer lobby (when you enable it in the options page).
-        var timePrefixString = '[font="sans-bold-13"]' + sprintf(translate("\\[%(time)s]"), { time: timeString }) + '[/font]';
+        var timePrefixString = '[font="sans-bold-13"]' + sprintf(translate("\\[%(time)s]"), { "time": timeString }) + '[/font]';
 
         // Translation: IRC message format when there is a time prefix.
-        return sprintf(translate("%(time)s %(message)s"), { time: timePrefixString, message: formattedMessage });
+        return sprintf(translate("%(time)s %(message)s"), { "time": timePrefixString, "message": formattedMessage });
     }
     else
         return formattedMessage;
 }
 
@@ -1011 +1010 @@
 {
     // Integer time in seconds.
     var time = Math.floor(Date.now() / 1000);
 
     // Update or initialize the user in the spam monitor.
-    if (g_spamMonitor[from])
-        g_spamMonitor[from][0]++;
+    if (g_SpamMonitor[from])
+        g_SpamMonitor[from][0]++;
     else
-        g_spamMonitor[from] = [1, time, 0];
+        g_SpamMonitor[from] = [1, time, 0];
 }
 
 /**
  * Check if a message is spam.
  *
@@ -1030 +1029 @@
 {
     // Integer time in seconds.
     var time = Math.floor(Date.now() / 1000);
 
     // Initialize if not already in the database.
-    if (!g_spamMonitor[from])
-        g_spamMonitor[from] = [1, time, 0];
+    if (!g_SpamMonitor[from])
+        g_SpamMonitor[from] = [1, time, 0];
 
     // Block blank lines.
     if (text == " ")
         return true;
     // Block users who are still within their spam block period.
-    else if (g_spamMonitor[from][2] + SPAM_BLOCK_LENGTH >= time)
+    else if (g_SpamMonitor[from][2] + g_SpamBlockLength >= time)
         return true;
     // Block users who exceed the rate of 1 message per second for five seconds and are not already blocked. TODO: Make this smarter and block profanity.
-    else if (g_spamMonitor[from][0] == 6)
+    else if (g_SpamMonitor[from][0] == 6)
     {
-        g_spamMonitor[from][2] = time;
+        g_SpamMonitor[from][2] = time;
         if (from == g_Name)
             addChatMessage({ "from": "system", "text": translate("Please do not spam. You have been blocked for thirty seconds.") });
         return true;
     }
     // Return false if everything is clear.
@@ -1061 +1060 @@
 {
     // Integer time in seconds.
     var time = Math.floor(Date.now() / 1000);
 
     // Clear message count every 5 seconds.
-    for each (var stats in g_spamMonitor)
+    for (let i in g_SpamMonitor)
     {
-        if (stats[1] + 5 <= time)
+        if (g_SpamMonitor[i][1] + 5 <= time)
         {
-            stats[1] = time;
-            stats[0] = 0;
+            g_SpamMonitor[i][1] = time;
+            g_SpamMonitor[i][0] = 0;
         }
     }
-
 }
 
 /* Utilities */
 // Generate a (mostly) unique color for this player based on their name.
 // See http://stackoverflow.com/questions/3426404/create-a-hexadecimal-colour-based-on-a-string-with-jquery-javascript
@@ -1106 +1104 @@
     if (color) {
     color = color.split(".");
     return ('[color="' + playername.split("").map(function (c, i) color.slice(i * 3, i * 3 + 3).join(" ") + '"]' + c + '[/color][color="')
                 .join("") + '"]').slice(0, -10);
     }
-    return '[color="' + getPlayerColor(playername.replace(g_modPrefix, "")) + '"]' + playername + '[/color]';
+    return '[color="' + getPlayerColor(playername.replace(g_ModPrefix, "")) + '"]' + playername + '[/color]';
 }
 
 // Ensure `value` is between 0 and 1.
 function clampColorValue(value)
 {