fix: broken migration on postgres (resolve #127)

migrations/20210206_drop_badges_column_add_sharing_flags.go

@@ -14,7 +14,7 @@ func init() {
 			migrator := db.Migrator()
 
 			if !migrator.HasColumn(&models.User{}, "badges_enabled") {
-				// empty database, nothing to migrate
+				// empty database or already migrated, nothing to migrate
 				return nil
 			}
 

migrations/20210213_add_has_data_field.go

@@ -1,17 +1,38 @@
 package migrations
 
 import (
+	"github.com/emvi/logbuch"
 	"github.com/muety/wakapi/config"
+	"github.com/muety/wakapi/models"
 	"gorm.io/gorm"
 )
 
 func init() {
+	const name = "20210213-add_has_data_field"
 	f := migrationFunc{
-		name: "20210213_add_has_data_field",
+		name: name,
 		f: func(db *gorm.DB, cfg *config.Config) error {
-			if err := db.Exec("UPDATE users SET has_data = TRUE WHERE 1").Error; err != nil {
+			condition := "key = ?"
+			if cfg.Db.Dialect == config.SQLDialectMysql {
+				condition = "`key` = ?"
+			}
+			lookupResult := db.Where(condition, name).First(&models.KeyStringValue{})
+			if lookupResult.Error == nil && lookupResult.RowsAffected > 0 {
+				logbuch.Info("no need to migrate '%s'", name)
+				return nil
+			}
+
+			if err := db.Exec("UPDATE users SET has_data = TRUE WHERE TRUE").Error; err != nil {
 				return err
 			}
+
+			if err := db.Create(&models.KeyStringValue{
+				Key:   name,
+				Value: "done",
+			}).Error; err != nil {
+				return err
+			}
+
 			return nil
 		},
 	}

version.txt

@@ -1 +1 @@
-1.24.0
+1.24.1