Merge pull request #411 from github-for-unity/fixes/update-repository-info

StanleyGoldman · web-flow · commit 9268a4d0097a · 2017-11-20T18:13:47.000-05:00
Changing UpdateRepositoryInfo to ClearRepositoryInfo
diff --git a/src/GitHub.Api/Git/Repository.cs b/src/GitHub.Api/Git/Repository.cs
@@ -442,7 +442,7 @@ private void RepositoryManager_OnCurrentBranchAndRemoteUpdated(ConfigBranch? bra
                 {
                         CurrentConfigRemote = remote;
                         CurrentRemote = GetGitRemote(remote.Value);
-                        UpdateRepositoryInfo();
+                        ClearRepositoryInfo();
                 }
             }) { Affinity = TaskAffinity.UI }.Start();
         }
@@ -485,20 +485,10 @@ private void UpdateLocalBranches()
             LocalBranches = LocalConfigBranches.Values.Select(GetLocalGitBranch).ToArray();
         }
 
-        private void UpdateRepositoryInfo()
+        private void ClearRepositoryInfo()
         {
-            if (CurrentRemote.HasValue)
-            {
-                CloneUrl = new UriString(CurrentRemote.Value.Url);
-                Name = CloneUrl.RepositoryName;
-                Logger.Trace("CloneUrl: {0}", CloneUrl.ToString());
-            }
-            else
-            {
-                CloneUrl = null;
-                Name = LocalPath.FileName;
-                Logger.Trace("CloneUrl: [NULL]");
-            }
+            CloneUrl = null;
+            Name = null;
         }
 
         private void RepositoryManager_OnLocalBranchRemoved(string name)

Original file line number	Diff line number	Diff line change
`@@ -442,7 +442,7 @@ private void RepositoryManager_OnCurrentBranchAndRemoteUpdated(ConfigBranch? bra`
`442`	`442`	`{`
`443`	`443`	`CurrentConfigRemote = remote;`
`444`	`444`	`CurrentRemote = GetGitRemote(remote.Value);`
`445`		`- UpdateRepositoryInfo();`
	`445`	`+ ClearRepositoryInfo();`
`446`	`446`	`}`
`447`	`447`	`}) { Affinity = TaskAffinity.UI }.Start();`
`448`	`448`	`}`
`@@ -485,20 +485,10 @@ private void UpdateLocalBranches()`
`485`	`485`	`LocalBranches = LocalConfigBranches.Values.Select(GetLocalGitBranch).ToArray();`
`486`	`486`	`}`
`487`	`487`
`488`		`- private void UpdateRepositoryInfo()`
	`488`	`+ private void ClearRepositoryInfo()`
`489`	`489`	`{`
`490`		`- if (CurrentRemote.HasValue)`
`491`		`- {`
`492`		`- CloneUrl = new UriString(CurrentRemote.Value.Url);`
`493`		`- Name = CloneUrl.RepositoryName;`
`494`		`- Logger.Trace("CloneUrl: {0}", CloneUrl.ToString());`
`495`		`- }`
`496`		`- else`
`497`		`- {`
`498`		`- CloneUrl = null;`
`499`		`- Name = LocalPath.FileName;`
`500`		`- Logger.Trace("CloneUrl: [NULL]");`
`501`		`- }`
	`490`	`+ CloneUrl = null;`
	`491`	`+ Name = null;`
`502`	`492`	`}`
`503`	`493`
`504`	`494`	`private void RepositoryManager_OnLocalBranchRemoved(string name)`