From e89db8393dadfe0d7a367a4e2e8de022fd651ac5 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 24 Jan 2026 16:03:10 +0000 Subject: [PATCH] Fix transaction scope issues in repo.go UpdateRepository and DeleteRepository Co-authored-by: unknwon <2946214+unknwon@users.noreply.github.com> --- internal/database/repo.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/database/repo.go b/internal/database/repo.go index d27692652..2935e6c47 100644 --- a/internal/database/repo.go +++ b/internal/database/repo.go @@ -1618,7 +1618,7 @@ func updateRepository(e *gorm.DB, repo *Repository, visibilityChanged bool) (err func UpdateRepository(repo *Repository, visibilityChanged bool) (err error) { return db.Transaction(func(tx *gorm.DB) error { - if err := updateRepository(db, repo, visibilityChanged); err != nil { + if err := updateRepository(tx, repo, visibilityChanged); err != nil { return errors.Newf("updateRepository: %v", err) } return nil @@ -1729,7 +1729,7 @@ func DeleteRepository(ownerID, repoID int64) error { } if repo.NumForks > 0 { - if err = db.Exec("UPDATE `repository` SET fork_id=0,is_fork=? WHERE fork_id=?", false, repo.ID).Error; err != nil { + if err = tx.Exec("UPDATE `repository` SET fork_id=0,is_fork=? WHERE fork_id=?", false, repo.ID).Error; err != nil { log.Error("reset 'fork_id' and 'is_fork': %v", err) } }