Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Changing UpdateRepositoryInfo to ClearRepositoryInfo #411

Merged
StanleyGoldman merged 4 commits into from
Nov 20, 2017
Merged

Changing UpdateRepositoryInfo to ClearRepositoryInfo #411

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
1aaf329
Changing UpdateRepositoryInfo to ClearRepositoryInfo
StanleyGoldman Nov 7, 2017
88c114e
CloneUrl and Name can be set to null because the property getter will…
StanleyGoldman Nov 14, 2017
8c2130c
Merge branch 'master' into fixes/update-repository-info
StanleyGoldman Nov 14, 2017
17b1c1e
Merge branch 'master' into fixes/update-repository-info
StanleyGoldman Nov 20, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 4 additions & 14 deletions src/GitHub.Api/Git/Repository.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -442,7 +442,7 @@ private void RepositoryManager_OnCurrentBranchAndRemoteUpdated(ConfigBranch? bra
{
CurrentConfigRemote = remote;
CurrentRemote = GetGitRemote(remote.Value);
UpdateRepositoryInfo();
ClearRepositoryInfo();
}
}) { Affinity = TaskAffinity.UI }.Start();
}
Expand Down Expand Up @@ -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)
Expand Down