Merge #1447: New dependencies of upgrading mods better handled by GUI

* Postremus/#795:
  Add dependencies we are installing for a upgrade of a mod to the changeset fixes #795

CHANGELOG.md

@@ -11,6 +11,7 @@ All notable changes to this project will be documented in this file.
 - [GUI] Installing a `.ckan` virtual mod will crash the client with an NRE less often (Postremus, #1478)
 - [GUI] The "Installing mods" tab is now called the "Status log" tab, as it's used for upgrading and removing mods too. (plague006, #1460)
 - [GUI] Links to `ckan://` resources under Linux are more likely to be handled correctly. (Postremus, #1434)
+- [GUI] Mods upgrades with additional dependencies are better handled and displayed to the user. (Postremus, #1447)
 
 ### Features
 

GUI/MainModList.cs

@@ -365,6 +365,7 @@ public async Task<IEnumerable<ModChange>> ComputeChangeSetFromModList(
                 {
                     case GUIModChangeType.None:
                         break;
+                    case GUIModChangeType.Update:
                     case GUIModChangeType.Install:
                         //TODO: Fix
                         //This will give us a mod with a wrong version!
@@ -373,8 +374,6 @@ public async Task<IEnumerable<ModChange>> ComputeChangeSetFromModList(
                     case GUIModChangeType.Remove:
                         modules_to_remove.Add(change.Mod);
                         break;
-                    case GUIModChangeType.Update:
-                        break;
                     default:
                         throw new ArgumentOutOfRangeException();
                 }