From f228007a38e0b9855f8eaf0e54ea9fecf6dabb1b Mon Sep 17 00:00:00 2001 From: Sage Pointer Date: Wed, 12 Sep 2018 21:28:05 +0200 Subject: [PATCH] Fix bug when repo remained bare if multiple branches pushed --- models/action.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/models/action.go b/models/action.go index f3f89d143f4a1..9050be22a6403 100644 --- a/models/action.go +++ b/models/action.go @@ -537,12 +537,14 @@ func CommitRepoAction(opts CommitRepoActionOptions) error { } refName := git.RefEndName(opts.RefFullName) - if repo.IsBare && refName != repo.DefaultBranch { + + // Change default branch and bare status only if pushed ref is non-empty branch. + if repo.IsBare && opts.NewCommitID != git.EmptySHA && strings.HasPrefix(opts.RefFullName, git.BranchPrefix) { repo.DefaultBranch = refName + repo.IsBare = false } // Change repository bare status and update last updated time. - repo.IsBare = repo.IsBare && opts.Commits.Len <= 0 if err = UpdateRepository(repo, false); err != nil { return fmt.Errorf("UpdateRepository: %v", err) }