Merge pull request #5 from Mirasaki/dev

Add player data limit customization

config/.env.example

@@ -8,6 +8,8 @@ TEST_SERVER_GUILD_ID=
 CFTOOLS_API_APPLICATION_ID=
 CFTOOLS_API_SECRET=
 CFTOOLS_SERVER_API_ID=
+# Amount of players to display on the leaderboard. 10 min, 25 max
+CFTOOLS_API_PLAYER_DATA_COUNT=15
 
 # Debug variables
 DEBUG_ENABLED=true

src/commands/dayz/leaderboard.js

@@ -7,7 +7,8 @@ const { parseSnakeCaseArray, colorResolver } = require('../../util');
 // Destructure from our process env
 const {
   DEBUG_LEADERBOARD_API_DATA,
-  NODE_ENV
+  NODE_ENV,
+  CFTOOLS_API_PLAYER_DATA_COUNT
 } = process.env;
 
 // Mapping our leaderboard stat options
@@ -89,14 +90,25 @@ module.exports = {
       mappedStat = Statistic.KILLS;
     }
 
+    // Getting our player data count
+    let playerLimit = Number(CFTOOLS_API_PLAYER_DATA_COUNT);
+    if (
+      isNaN(playerLimit)
+      || playerLimit < 10
+      || playerLimit > 25
+    ) {
+      // Overwrite the provided player limit back to default if invalid
+      playerLimit = 15;
+    }
+
     let res;
     try {
       // Fetching our leaderboard data from the CFTools API
       res = await cftClient
         .getLeaderboard({
           order: 'ASC',
           statistic: mappedStat,
-          limit: 15
+          limit: playerLimit
         });
     } catch (err) {
       // Properly logging the error if it is encountered