From 960c322586eceb9598bb0a9985a8dd987dc74807 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Fri, 22 Oct 2021 00:10:49 +0800 Subject: Refactor update checker to use AppState (#17387) We have the `AppState` module now, it can store app related data easily. We do not need to create separate tables for each feature. So the update checker can use `AppState` instead of a one-row dedicate table. And the code of update checker is moved from `models` to `modules`. --- models/migrations/v199.go | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) (limited to 'models/migrations/v199.go') diff --git a/models/migrations/v199.go b/models/migrations/v199.go index 64b21172c1..4351ba4fa8 100644 --- a/models/migrations/v199.go +++ b/models/migrations/v199.go @@ -5,19 +5,10 @@ package migrations import ( - "fmt" - "xorm.io/xorm" ) -func addRemoteVersionTable(x *xorm.Engine) error { - type RemoteVersion struct { - ID int64 `xorm:"pk autoincr"` - Version string `xorm:"VARCHAR(50)"` - } - - if err := x.Sync2(new(RemoteVersion)); err != nil { - return fmt.Errorf("Sync2: %v", err) - } +func addRemoteVersionTableNoop(x *xorm.Engine) error { + // we used to use a table `remote_version` to store information for updater, now we use `AppState`, so this migration task is a no-op now. return nil } -- cgit v1.2.3