Removed duplicate names in deck_ordering bug

SquireTournamentServices · Nov 27, 2022 · 6123156 · 6123156
1 parent de55411
commit 6123156
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/squire_lib/src/players/mod.rs b/squire_lib/src/players/mod.rs
@@ -70,8 +70,9 @@ impl Player {
 
     /// Adds a deck to the player
     pub fn add_deck(&mut self, name: String, deck: Deck) {
-        self.deck_ordering.push(name.clone());
-        self.decks.insert(name, deck);
+        if self.decks.insert(name.clone(), deck).is_none() {
+            self.deck_ordering.push(name);
+        }
     }
 
     /// Gets a specific deck from the player