MarcoBuster/gitea
1
0
Fork 0
mirror of https://github.com/go-gitea/gitea synced 2024-06-01 19:06:52 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix(transaction): changed the use of the transaction to WithTx instead of TxContext

Browse Source
This commit is contained in:
Daniel Carvalho 2024-05-17 17:02:35 +00:00
parent 3704d233eb
commit 805dbc7ed4
1 changed files with 21 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
models/repo/pin.go
View File

@ -33,33 +33,29 @@ func IsPinned(ctx context.Context, userID, repoID int64) bool {
}
func PinRepo(ctx context.Context, doer *user_model.User, repo *Repository, pin bool) error {
ctx, commiter, err := db.TxContext(ctx)
if err != nil {
return err
}
return db.WithTx(ctx, func(ctx context.Context) error {
pinned := IsPinned(ctx, doer.ID, repo.ID)
defer commiter.Close()
pinned := IsPinned(ctx, doer.ID, repo.ID)
if pin {
// Already pinned, nothing to do
if pinned {
return nil
}
if pin {
// Already pinned, nothing to do
if pinned {
return nil
if err := db.Insert(ctx, &Pin{UID: doer.ID, RepoID: repo.ID}); err != nil {
return err
}
} else {
// Not pinned, nothing to do
if !pinned {
return nil
}
if _, err := db.DeleteByBean(ctx, &Pin{UID: doer.ID, RepoID: repo.ID}); err != nil {
return err
}
}
if err = db.Insert(ctx, &Pin{UID: doer.ID, RepoID: repo.ID}); err != nil {
return err
}
} else {
// Not pinned, nothing to do
if !pinned {
return nil
}
if _, err = db.DeleteByBean(ctx, &Pin{UID: doer.ID, RepoID: repo.ID}); err != nil {
return err
}
}
return commiter.Commit()
return nil
})
}