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

Commit 2238a80

Browse files
Fixing branch selection when not named in config
1 parent 6e77812 commit 2238a80

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

src/GitHub.Api/Git/RepositoryManager.cs

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -346,6 +346,7 @@ private void SetupWatcher()
346346
private void UpdateHead()
347347
{
348348
var head = repositoryPaths.DotGitHead.ReadAllLines().FirstOrDefault();
349+
Logger.Trace("UpdateHead: {0}", head ?? "[NULL]");
349350
UpdateCurrentBranchAndRemote(head);
350351
}
351352

@@ -428,6 +429,11 @@ private void UpdateCurrentBranchAndRemote(string head)
428429
{
429430
var branchName = head.Substring(head.IndexOf("refs/heads/") + "refs/heads/".Length);
430431
branch = config.GetBranch(branchName);
432+
433+
if (!branch.HasValue)
434+
{
435+
branch = new ConfigBranch { Name = branchName };
436+
}
431437
}
432438

433439
var defaultRemote = "origin";
@@ -452,7 +458,10 @@ private void UpdateCurrentBranchAndRemote(string head)
452458
}
453459
}
454460

461+
Logger.Trace("OnCurrentBranchUpdated: {0}", branch.HasValue ? branch.Value.ToString() : "[NULL]");
455462
OnCurrentBranchUpdated?.Invoke(branch);
463+
464+
Logger.Trace("OnCurrentRemoteUpdated: {0}", remote.HasValue ? remote.Value.ToString() : "[NULL]");
456465
OnCurrentRemoteUpdated?.Invoke(remote);
457466
}
458467

@@ -476,13 +485,13 @@ private void Watcher_OnLocalBranchChanged(string name)
476485

477486
private void UpdateConfigData(bool resetConfig = false)
478487
{
488+
Logger.Trace("UpdateConfigData reset:{0}", resetConfig);
489+
479490
if (resetConfig)
480491
{
481492
config.Reset();
482493
}
483494

484-
Logger.Trace("RefreshConfigData");
485-
486495
LoadBranchesFromConfig();
487496
LoadRemotesFromConfig();
488497
UpdateHead();
@@ -494,6 +503,8 @@ private void LoadBranchesFromConfig()
494503

495504
var branches = new Dictionary<string, ConfigBranch>();
496505
LoadBranchesFromConfig(branches, repositoryPaths.BranchesPath, config.GetBranches().Where(x => x.IsTracking), "");
506+
507+
Logger.Trace("OnLocalBranchListUpdated {0} branches", branches.Count);
497508
OnLocalBranchListUpdated?.Invoke(branches);
498509
}
499510

@@ -542,6 +553,7 @@ private void LoadRemotesFromConfig()
542553
}
543554
}
544555

556+
Logger.Trace("OnRemoteBranchListUpdated {0} remotes", remotes.Count);
545557
OnRemoteBranchListUpdated?.Invoke(remotes, remoteBranches);
546558
}
547559

0 commit comments

Comments
 (0)