Fixed issue where smartcard can add players twice

kcapp · Dec 11, 2024 · 8d4dad6 · 8d4dad6
1 parent d1b934e
commit 8d4dad6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/routes/matches.js b/routes/matches.js
@@ -241,7 +241,7 @@ router.get('/:id/result', function (req, res, next) {
 
 /* Method for starting a new match */
 router.post('/new', function (req, res, next) {
-    const players = req.body.players;
+    const players = [...new Set(req.body.players)];
     if (players === undefined || players.length === 0) {
         debug('No players specified, unable to start leg');
         res.status(400).send("No players specified").end();

diff --git a/src/pages/index/components/new-game-form/new-game-form.component.js b/src/pages/index/components/new-game-form/new-game-form.component.js
@@ -92,7 +92,7 @@ module.exports = {
                                     const player = _.find(this.input.players, (p) => {
                                         return p.id === playerId;
                                     });
-                                    if (player) {
+                                    if (player && this.state.selected.indexOf(player) === -1) {
                                         this.addPlayer(null, player);
                                     }
                                 });