x/spend-sparrow
2
0
Fork 0
Code Issues 7 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

fix: dont prematurely close transaction
All checks were successful
Build Docker Image / Build-Docker-Image (push) Successful in 4m25s
Details
Build and Push Docker Image / Build-And-Push-Docker-Image (push) Successful in 5m7s
Details

Browse Source
This commit was merged in pull request #140.
This commit is contained in:
Tim Wundenberg
2025-05-29 20:08:11 +02:00
parent c2b96145f3
commit 4fa605bd8f
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
internal/service/transaction.go
View File

@@ -56,7 +56,9 @@ func (s TransactionImpl) Add(tx *sqlx.Tx, user *types.User, transactionInput typ
} }
var err error var err error
ownsTransaction := false
if tx == nil { if tx == nil {
ownsTransaction = true
tx, err = s.db.Beginx() tx, err = s.db.Beginx()
err = db.TransformAndLogDbError("transaction Add", nil, err) err = db.TransformAndLogDbError("transaction Add", nil, err)
if err != nil { if err != nil {
@@ -104,10 +106,12 @@ func (s TransactionImpl) Add(tx *sqlx.Tx, user *types.User, transactionInput typ
} }
} }
err = tx.Commit() if ownsTransaction {
err = db.TransformAndLogDbError("transaction Add", nil, err) err = tx.Commit()
if err != nil { err = db.TransformAndLogDbError("transaction Add", nil, err)
return nil, err if err != nil {
return nil, err
}
} }
return transaction, nil return transaction, nil