Better ignoring of bots in kill stats.

sysrage · Oct 13, 2015 · 1046a23 · 1046a23
1 parent 4174e82
commit 1046a23
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/cu-chatbot.cfg.example b/cu-chatbot.cfg.example
@@ -95,6 +95,15 @@ module.exports = {
         'user'
     ],
 
+    // Names of bots to be ignored for leaderboard
+    botNames: [
+        'SuperFireBot',
+        'SuperWaterBot',
+        'SuperEarthBot',
+        '[Bot] WaterMage',
+        '[Bot] FireMage',
+    ],
+
     // Keyword regular expressions to watch for which match test CSE messages
     testKeywords: [
         ".*impromptu.*",

diff --git a/cu-chatbot.js b/cu-chatbot.js
@@ -329,7 +329,7 @@ var chatCommands = [
 
         // Remove bots from rankings
         for (var i = 0; i < pStats.length; i++) {
-            if (['SuperFireBot','SuperWaterBot','SuperEarthBot','[Bot] WaterMage','[Bot] FireMage'].indexOf(pStats[i].playerName) > -1) {
+            if (config.botNames.indexOf(pStats[i].playerName) > -1) {
                 pStats.splice(i, 1);
                 i--;
             }
@@ -1534,8 +1534,8 @@ function controlGame(server) {
                         getRoundKills(server, roundStartTime, roundEndTime, 0, function(roundKills) {
                             var playersInRound = [];
                             roundKills.forEach(function(rkEntry) {
-                                // Skip suicides
-                                if (rkEntry.killerName !== rkEntry.victimName) {
+                                // Skip suicides and bot kills
+                                if (rkEntry.killerName !== rkEntry.victimName && config.botNames.indexOf(rkEntry.victimName) === -1) {
                                     // Check to see if these players already had their gameCount increased for this round.
                                     var killerAlreadyCounted = false;
                                     var victimAlreadyCounted = false;

diff --git a/server.js b/server.js
@@ -324,7 +324,7 @@ var SampleApp = function() {
 
                     // Remove bots from rankings.
                     for (var i = 0; i < data.length; i++) {
-                        if (['SuperFireBot','SuperWaterBot','SuperEarthBot','[Bot] WaterMage','[Bot] FireMage'].indexOf(data[i].playerName) > -1) {
+                        if (config.botNames.indexOf(data[i].playerName) > -1) {
                             data.splice(i, 1);
                             i--;
                         }