Browse Source

Migration code: errors are not to be forgotten

Peter Smit 10 years ago
parent
commit
4ef3245413
1 changed files with 6 additions and 2 deletions
  1. 6 2
      models/migrations/migrations.go

+ 6 - 2
models/migrations/migrations.go

@@ -20,7 +20,9 @@ var migrations = []migration{}
 
 // Migrate database to current version
 func Migrate(x *xorm.Engine) error {
-	x.Sync(new(Version))
+	if err := x.Sync(new(Version)); err != nil {
+		return err
+	}
 
 	currentVersion := &Version{Id: 1}
 	has, err := x.Get(currentVersion)
@@ -39,7 +41,9 @@ func Migrate(x *xorm.Engine) error {
 			return err
 		}
 		currentVersion.Version = v + int64(i) + 1
-		x.Id(1).Update(currentVersion)
+		if _, err = x.Id(1).Update(currentVersion); err != nil {
+			return err
+		}
 	}
 	return nil
 }