Correctly compare repository full name.

grokys · grokys · commit e7144520b5dc · 2017-11-09T17:30:28.000+01:00
Fixes opening links on enterprise instances. Previously we were comparing `repo.FullName == ActiveRepo.Name` which always failed as `repo.FullName` returns `owner/name` whereas `ActiveRepo.Name` just returns the `name` portion. Construct a full name for the active repo and check that. Fixes #1291
diff --git a/src/GitHub.VisualStudio/Menus/MenuBase.cs b/src/GitHub.VisualStudio/Menus/MenuBase.cs
@@ -131,7 +131,8 @@ protected async Task<bool> IsGitHubRepo()
             if (!isdotcom)
             {
                 var repo = await SimpleApiClient.GetRepository();
-                return (repo.FullName == ActiveRepo.Name || repo.Id == 0) && SimpleApiClient.IsEnterprise();
+                var activeRepoFullName = ActiveRepo.Owner + '/' + ActiveRepo.Name;
+                return (repo.FullName == activeRepoFullName || repo.Id == 0) && SimpleApiClient.IsEnterprise();
             }
             return isdotcom;
         }

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,8 @@ protected async Task<bool> IsGitHubRepo()`
`131`	`131`	`if (!isdotcom)`
`132`	`132`	`{`
`133`	`133`	`var repo = await SimpleApiClient.GetRepository();`
`134`		`- return (repo.FullName == ActiveRepo.Name \|\| repo.Id == 0) && SimpleApiClient.IsEnterprise();`
	`134`	`+ var activeRepoFullName = ActiveRepo.Owner + '/' + ActiveRepo.Name;`
	`135`	`+ return (repo.FullName == activeRepoFullName \|\| repo.Id == 0) && SimpleApiClient.IsEnterprise();`
`135`	`136`	`}`
`136`	`137`	`return isdotcom;`
`137`	`138`	`}`