fix: Fix panic when converting schema changes to string (#1885)

bbernays · web-flow · commit 8274f172ebf6 · 2024-09-02T15:42:13.000Z
`previous` is never set so shouldn't be stringified
diff --git a/schema/table.go b/schema/table.go
@@ -231,7 +231,7 @@ func (t TableColumnChange) String() string {
 	case TableColumnChangeTypeRemoveUniqueConstraint:
 		return fmt.Sprintf("column: %s, previous: %s", t.ColumnName, t.Previous)
 	case TableColumnChangeTypeMoveToCQOnly:
-		return fmt.Sprintf("multi-column: %s, type: %s, previous: %s", t.ColumnName, t.Type, t.Previous)
+		return fmt.Sprintf("multi-column: %s, type: %s", t.ColumnName, t.Type)
 	default:
 		return fmt.Sprintf("column: %s, type: %s, current: %s, previous: %s", t.ColumnName, t.Type, t.Current, t.Previous)
 	}

Original file line number	Diff line number	Diff line change
`@@ -231,7 +231,7 @@ func (t TableColumnChange) String() string {`
`231`	`231`	`case TableColumnChangeTypeRemoveUniqueConstraint:`
`232`	`232`	`return fmt.Sprintf("column: %s, previous: %s", t.ColumnName, t.Previous)`
`233`	`233`	`case TableColumnChangeTypeMoveToCQOnly:`
`234`		`- return fmt.Sprintf("multi-column: %s, type: %s, previous: %s", t.ColumnName, t.Type, t.Previous)`
	`234`	`+ return fmt.Sprintf("multi-column: %s, type: %s", t.ColumnName, t.Type)`
`235`	`235`	`default:`
`236`	`236`	`return fmt.Sprintf("column: %s, type: %s, current: %s, previous: %s", t.ColumnName, t.Type, t.Current, t.Previous)`
`237`	`237`	`}`