realaravinth/forgejo-federation
1
0
Fork 0
Code Issues Pull requests 6 Projects Releases Packages Wiki Activity

IsBranchExist: return false if provided name is empty (#8485)

Browse source 
* IsBranchExist: return false if provided name is empty

* Ensure that the reference returned is actually of a valid type
This commit is contained in:
zeripath 2019-10-13 17:29:08 +01:00 committed by Lauris BH
parent f858b89b13
commit c888ebfba7
1 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
modules/git/repo_branch.go
View file

@ -28,8 +28,14 @@ func IsBranchExist(repoPath, name string) bool {
// IsBranchExist returns true if given branch exists in current repository. // IsBranchExist returns true if given branch exists in current repository.
func (repo *Repository) IsBranchExist(name string) bool { func (repo *Repository) IsBranchExist(name string) bool {
_, err := repo.gogitRepo.Reference(plumbing.ReferenceName(BranchPrefix+name), true) if name == "" {
return err == nil return false
}
reference, err := repo.gogitRepo.Reference(plumbing.ReferenceName(BranchPrefix+name), true)
if err != nil {
return false
}
return reference.Type() != plumbing.InvalidReference
} }
// Branch represents a Git branch. // Branch represents a Git branch.