Change review content column to type text in db (#9229)

Author: davidsvantesson

models/migrations/migrations.go

@@ -274,6 +274,8 @@ var migrations = []Migration{
 	NewMigration("Add comment_id on table notification", addCommentIDOnNotification),
 	// v109 -> v110
 	NewMigration("add can_create_org_repo to team", addCanCreateOrgRepoColumnForTeam),
+	// v110 -> v111
+	NewMigration("change review content type to text", changeReviewContentToText),
 }
 
 // Migrate database to current version

models/migrations/v110.go

@@ -0,0 +1,36 @@
+// Copyright 2019 The Gitea Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package migrations
+
+import (
+	"xorm.io/core"
+	"xorm.io/xorm"
+)
+
+func changeReviewContentToText(x *xorm.Engine) error {
+
+	if x.Dialect().DBType() == core.MYSQL {
+		_, err := x.Exec("ALTER TABLE review MODIFY COLUMN content TEXT")
+		return err
+	}
+
+	if x.Dialect().DBType() == core.ORACLE {
+		_, err := x.Exec("ALTER TABLE review MODIFY content TEXT")
+		return err
+	}
+
+	if x.Dialect().DBType() == core.MSSQL {
+		_, err := x.Exec("ALTER TABLE review ALTER COLUMN content TEXT")
+		return err
+	}
+
+	if x.Dialect().DBType() == core.POSTGRES {
+		_, err := x.Exec("ALTER TABLE review ALTER COLUMN content TYPE TEXT")
+		return err
+	}
+
+	// SQLite doesn't support ALTER COLUMN, and it seem to already make String to _TEXT_ default so no migration needed
+	return nil
+}

models/review.go

@@ -52,7 +52,7 @@ type Review struct {
 	ReviewerID int64  `xorm:"index"`
 	Issue      *Issue `xorm:"-"`
 	IssueID    int64  `xorm:"index"`
-	Content    string
+	Content    string `xorm:"TEXT"`
 
 	CreatedUnix timeutil.TimeStamp `xorm:"INDEX created"`
 	UpdatedUnix timeutil.TimeStamp `xorm:"INDEX updated"`