From b116dc506b55a78ef47582271f10bb810f000c47 Mon Sep 17 00:00:00 2001 From: Unknwon Date: Sun, 12 Mar 2017 03:34:19 -0400 Subject: [PATCH] migrations/v16: use continue not return for bad records --- models/migrations/v16.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/models/migrations/v16.go b/models/migrations/v16.go index 7c3f4821c..1c23dc410 100644 --- a/models/migrations/v16.go +++ b/models/migrations/v16.go @@ -49,7 +49,7 @@ func updateRepositorySizes(x *xorm.Engine) (err error) { for _, repo := range repos { if repo.Name == "." || repo.Name == ".." { - return nil + continue } user := new(User) @@ -57,14 +57,14 @@ func updateRepositorySizes(x *xorm.Engine) (err error) { if err != nil { return fmt.Errorf("query owner of repository [repo_id: %d, owner_id: %d]: %v", repo.ID, repo.OwnerID, err) } else if !has { - return nil + continue } repoPath := filepath.Join(setting.RepoRootPath, strings.ToLower(user.Name), strings.ToLower(repo.Name)) + ".git" countObject, err := git.GetRepoSize(repoPath) if err != nil { log.Warn("GetRepoSize: %v", err) - return nil + continue } repo.Size = countObject.Size + countObject.SizePack