Inactive players can now be added via ID from the home page

Author: thordy

CHANGELOG.md

@@ -6,6 +6,7 @@ A preview of major changes can be found in the Wiki ([Latest Changes](https://gi
 - Option to see `All` or `Unique` tournament statistics per player
 - Ability to "Remote Start" matches on a selected venue
 - Split tournaments into "Tournaments" and "Seasons"
+- Inactive players can now be added via ID from the home page
 
 #### Changed
 - Updated to latest version of dependencies

routes/index.js

@@ -14,7 +14,7 @@ const screensaverTemplate = template.load(require.resolve('../src/pages/screensa
 /* GET home page. */
 router.get('/', function (req, res, next) {
     axios.all([
-        axios.get(`${req.app.locals.kcapp.api}/player/active`),
+        axios.get(`${req.app.locals.kcapp.api}/player`),
         axios.get(`${req.app.locals.kcapp.api}/match/modes`),
         axios.get(`${req.app.locals.kcapp.api}/owetype`),
         axios.get(`${req.app.locals.kcapp.api}/match/types`),

src/pages/index/components/new-game-form/new-game-form.component.js

@@ -10,7 +10,7 @@ module.exports = {
     onCreate(input) {
         this.state = {
             selected: [],
-            players: _.reject(input.players, (player) => { return player.is_bot; }),
+            players: _.reject(input.players, (player) => { return player.is_bot || !player.is_active; }),
             input: input,
             officeId: 0,
             venues: input.venues,
@@ -186,18 +186,19 @@ module.exports = {
                 }
                 this.state.playerId = '';
 
-                let player = _.find(this.state.players, function (player) {
-                    return player.id == playerId;
-                });
+                let player = _.find(this.state.players, (player) => player.id == playerId);
                 if (player) {
                     // Player is not already added, so add it
                     this.addPlayer(null, { input: { player: player } });
                 } else {
                     // Player is already added, so remove it
-                    player = _.find(this.state.selected, function (player) {
-                        return player.id == playerId;
-                    });
+                    player = _.find(this.state.selected, (player) => player.id == playerId);
                     if (!player) {
+                        // Check all players, since it could be a inactive player
+                        let player = _.find(this.input.players, (player) => player.id == playerId);
+                        if (player) {
+                            this.addPlayer(null, { input: { player: player } });
+                        }
                         return;
                     }
                     this.removePlayer(null, { input: { player: player } });