Follow the old logic

This commit is contained in:
Lunny Xiao 2024-08-26 10:37:31 -07:00
parent fe9efa90f8
commit 4bf86e9594
No known key found for this signature in database
GPG Key ID: C3B7C91B632F738A

View File

@ -47,11 +47,12 @@ func TransferOwnership(ctx context.Context, doer, newOwner *user_model.User, rep
log.Error("lock.Lock(): %v", err) log.Error("lock.Lock(): %v", err)
return fmt.Errorf("lock.Lock: %w", err) return fmt.Errorf("lock.Lock: %w", err)
} }
defer releaser()
if err := transferOwnership(ctx, doer, newOwner.Name, repo); err != nil { if err := transferOwnership(ctx, doer, newOwner.Name, repo); err != nil {
releaser()
return err return err
} }
releaser()
newRepo, err := repo_model.GetRepositoryByID(ctx, repo.ID) newRepo, err := repo_model.GetRepositoryByID(ctx, repo.ID)
if err != nil { if err != nil {
@ -372,11 +373,12 @@ func ChangeRepositoryName(ctx context.Context, doer *user_model.User, repo *repo
log.Error("lock.Lock(): %v", err) log.Error("lock.Lock(): %v", err)
return fmt.Errorf("lock.Lock: %w", err) return fmt.Errorf("lock.Lock: %w", err)
} }
defer releaser()
if err := changeRepositoryName(ctx, repo, newRepoName); err != nil { if err := changeRepositoryName(ctx, repo, newRepoName); err != nil {
releaser()
return err return err
} }
releaser()
repo.Name = newRepoName repo.Name = newRepoName
notify_service.RenameRepository(ctx, doer, repo, oldRepoName) notify_service.RenameRepository(ctx, doer, repo, oldRepoName)