summaryrefslogtreecommitdiffstats
path: root/models/system
diff options
context:
space:
mode:
authorLunny Xiao <xiaolunwen@gmail.com>2022-11-13 04:18:50 +0800
committerGitHub <noreply@github.com>2022-11-12 21:18:50 +0100
commit34283a74e85278fed2c9b70d6f8749dc6a4001ca (patch)
treef8fa43399cb723d55efc34dacf7947faf0ed5acf /models/system
parenta0a425a13ba587829a831aaecd8469d39d372111 (diff)
downloadgitea-34283a74e85278fed2c9b70d6f8749dc6a4001ca.tar.gz
gitea-34283a74e85278fed2c9b70d6f8749dc6a4001ca.zip
Allow detect whether it's in a database transaction for a context.Context (#21756)
Fix #19513 This PR introduce a new db method `InTransaction(context.Context)`, and also builtin check on `db.TxContext` and `db.WithTx`. There is also a new method `db.AutoTx` has been introduced but could be used by other PRs. `WithTx` will always open a new transaction, if a transaction exist in context, return an error. `AutoTx` will try to open a new transaction if no transaction exist in context. That means it will always enter a transaction if there is no error. Co-authored-by: delvh <dev.lh@web.de> Co-authored-by: 6543 <6543@obermui.de>
Diffstat (limited to 'models/system')
-rw-r--r--models/system/appstate.go2
-rw-r--r--models/system/setting.go2
2 files changed, 2 insertions, 2 deletions
diff --git a/models/system/appstate.go b/models/system/appstate.go
index c11a2512ab..bc18c5181d 100644
--- a/models/system/appstate.go
+++ b/models/system/appstate.go
@@ -25,7 +25,7 @@ func init() {
// SaveAppStateContent saves the app state item to database
func SaveAppStateContent(key, content string) error {
- return db.WithTx(func(ctx context.Context) error {
+ return db.WithTx(db.DefaultContext, func(ctx context.Context) error {
eng := db.GetEngine(ctx)
// try to update existing row
res, err := eng.Exec("UPDATE app_state SET revision=revision+1, content=? WHERE id=?", content, key)
diff --git a/models/system/setting.go b/models/system/setting.go
index b4011b1b3e..22119ee5e6 100644
--- a/models/system/setting.go
+++ b/models/system/setting.go
@@ -196,7 +196,7 @@ func SetSetting(setting *Setting) error {
}
func upsertSettingValue(key, value string, version int) error {
- return db.WithTx(func(ctx context.Context) error {
+ return db.WithTx(db.DefaultContext, func(ctx context.Context) error {
e := db.GetEngine(ctx)
// here we use a general method to do a safe upsert for different databases (and most transaction levels)