Apply suggestions from code review

Author: 6543

models/migrations/v167.go

@@ -11,6 +11,11 @@ import (
 )
 
 func recreateUserTableToFixDefaultValues(x *xorm.Engine) error {
+	type User struct {
+		ID                  int64 `xorm:"pk autoincr"`
+		KeepActivityPrivate bool  `xorm:"NOT NULL DEFAULT false"`
+	}
+
 	if _, err := x.Where(builder.IsNull{"keep_activity_private"}).
 		Cols("keep_activity_private").
 		Update(User{KeepActivityPrivate: false}); err != nil {
@@ -29,11 +34,6 @@ func recreateUserTableToFixDefaultValues(x *xorm.Engine) error {
 		return err
 	}
 
-	type User struct {
-		ID                  int64 `xorm:"pk autoincr"`
-		KeepActivityPrivate bool  `xorm:"NOT NULL DEFAULT false"`
-	}
-
 	sess := x.NewSession()
 	defer sess.Close()
 	if err := sess.Begin(); err != nil {