added a null check to onlineManifest in VersionCheck.Resolvers so Resolvers don't throw exception when something goes wrong in FetchManifest (#89)

mohas · web-flow · commit b49f6d62a558 · 2020-04-23T12:47:16.000+02:00
diff --git a/Source/ModManager/Dependencies/VersionCheck.cs b/Source/ModManager/Dependencies/VersionCheck.cs
@@ -49,7 +49,7 @@ public override List<FloatMenuOption> Resolvers
             {
 
                 var options = Utilities.NewOptionsList;
-                if ( !onlineManifest.downloadUri.NullOrEmpty() )
+                if ( onlineManifest != null && !onlineManifest.downloadUri.NullOrEmpty() )
                     options.Add( new FloatMenuOption( I18n.OpenDownloadUri( onlineManifest.downloadUri ),
                                                       () => SteamUtility.OpenUrl( onlineManifest.downloadUri ) ) );
                 else

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public override List<FloatMenuOption> Resolvers`
`49`	`49`	`{`
`50`	`50`
`51`	`51`	`var options = Utilities.NewOptionsList;`
`52`		`- if ( !onlineManifest.downloadUri.NullOrEmpty() )`
	`52`	`+ if ( onlineManifest != null && !onlineManifest.downloadUri.NullOrEmpty() )`
`53`	`53`	`options.Add( new FloatMenuOption( I18n.OpenDownloadUri( onlineManifest.downloadUri ),`
`54`	`54`	`() => SteamUtility.OpenUrl( onlineManifest.downloadUri ) ) );`
`55`	`55`	`else`