Fixed a bug where player from wrong game is shown if the name is the …

…same + optimisations
dRoskar · Jan 21, 2015 · a2bf4ea · a2bf4ea
1 parent e218132
commit a2bf4ea
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/BGCRanker/MainWindow.xaml.cs b/BGCRanker/MainWindow.xaml.cs
@@ -121,6 +121,7 @@ private void readData(String newGame)
             // reset collected data
             games.Clear();
             players.Clear();
+            gameNames.Clear();
 
             // get games
             String[] gameDirectories = Directory.GetDirectories(dataPath);
@@ -162,6 +163,7 @@ private void readData(String newGame)
 
                             player.Victories = 0;
                             player.VictoriesOld = 0;
+                            player.HasPrev = false;
                         }
                         else
                         {
@@ -242,6 +244,7 @@ public void comboSelectionChanged(String selectPlayer)
                 if (game.Name == (String) gamesComboBox.SelectedItem)
                 {
                     selectedGame = game;
+                    break;
                 }
             }
 
@@ -308,9 +311,10 @@ private void playersListBox_SelectionChanged(object sender, SelectionChangedEven
                     selectedPlayer = null;
                     foreach (Player player in players)
                     {
-                        if (player.Name == playersListBox.SelectedItem.ToString())
+                        if (player.Name == playersListBox.SelectedItem.ToString() && player.Game == selectedGame.Name)
                         {
                             selectedPlayer = player;
+                            break;
                         }
                     }