Fix enterprise login.

grokys · grokys · commit 5e324ef37968 · 2017-12-07T10:53:52.000+01:00
Was throwing a `NullReferenceException` after recent merge.
diff --git a/src/GitHub.App/ViewModels/Dialog/LoginToGitHubForEnterpriseViewModel.cs b/src/GitHub.App/ViewModels/Dialog/LoginToGitHubForEnterpriseViewModel.cs
@@ -58,7 +58,7 @@ public LoginToGitHubForEnterpriseViewModel(
             this.WhenAnyValue(x => x.EnterpriseUrl, x => x.EnterpriseUrlValidator.ValidationResult)
                 .Throttle(TimeSpan.FromMilliseconds(500))
                 .ObserveOn(RxApp.MainThreadScheduler)
-                .Subscribe(x => EnterpriseUrlChanged(x.Item1, x.Item2.IsValid));
+                .Subscribe(x => EnterpriseUrlChanged(x.Item1, x.Item2?.IsValid ?? false));
 
             NavigateLearnMore = ReactiveCommand.CreateAsyncObservable(_ =>
             {
@@ -120,7 +120,7 @@ async void EnterpriseUrlChanged(string url, bool valid)
 
             try
             {
-                if (valid)
+                if (!string.IsNullOrWhiteSpace(url) && valid)
                 {
                     IsEnterpriseInstance = SupportsUserNameAndPassword = null;
 

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ public LoginToGitHubForEnterpriseViewModel(`
`58`	`58`	`this.WhenAnyValue(x => x.EnterpriseUrl, x => x.EnterpriseUrlValidator.ValidationResult)`
`59`	`59`	`.Throttle(TimeSpan.FromMilliseconds(500))`
`60`	`60`	`.ObserveOn(RxApp.MainThreadScheduler)`
`61`		`- .Subscribe(x => EnterpriseUrlChanged(x.Item1, x.Item2.IsValid));`
	`61`	`+ .Subscribe(x => EnterpriseUrlChanged(x.Item1, x.Item2?.IsValid ?? false));`
`62`	`62`
`63`	`63`	`NavigateLearnMore = ReactiveCommand.CreateAsyncObservable(_ =>`
`64`	`64`	`{`
`@@ -120,7 +120,7 @@ async void EnterpriseUrlChanged(string url, bool valid)`
`120`	`120`
`121`	`121`	`try`
`122`	`122`	`{`
`123`		`- if (valid)`
	`123`	`+ if (!string.IsNullOrWhiteSpace(url) && valid)`
`124`	`124`	`{`
`125`	`125`	`IsEnterpriseInstance = SupportsUserNameAndPassword = null;`
`126`	`126`