Don't prompt user if we already know the game

KSP-CKAN · Dec 30, 2021 · 90be65d · 90be65d
1 parent a6e0bec
commit 90be65d
Showing 1 changed file with 15 additions and 11 deletions.
diff --git a/GUI/Dialogs/CloneFakeGameDialog.cs b/GUI/Dialogs/CloneFakeGameDialog.cs
@@ -145,22 +145,26 @@ private async void buttonOK_Click(object sender, EventArgs e)
 
                 try
                 {
-                    IGame guessedGame = manager.DetermineGame(new DirectoryInfo(existingPath), user);
-                    if (guessedGame == null)
+                    GameInstance instanceToClone = null;
+                    if (!manager.Instances.TryGetValue(comboBoxKnownInstance.SelectedItem as string, out instanceToClone)
+                        || existingPath != instanceToClone.GameDir().Replace('/', Path.DirectorySeparatorChar))
                     {
-                        // User cancelled, let them try again
-                        reactivateDialog();
-                        return;
-                    }
-                    await Task.Run(() =>
-                    {
-                        GameInstance instanceToClone = new GameInstance(
-                            guessedGame,
+                        IGame sourceGame = manager.DetermineGame(new DirectoryInfo(existingPath), user);
+                        if (sourceGame == null)
+                        {
+                            // User cancelled, let them try again
+                            reactivateDialog();
+                            return;
+                        }
+                        instanceToClone = new GameInstance(
+                            sourceGame,
                             existingPath,
                             "irrelevant",
                             user
                         );
-
+                    }
+                    await Task.Run(() =>
+                    {
                         if (instanceToClone.Valid)
                         {
                             manager.CloneInstance(instanceToClone, newName, newPath);