Don't alter column with full column data type, close #3339
This commit is contained in:
		
							parent
							
								
									496db1f13e
								
							
						
					
					
						commit
						0273856e4d
					
				| @ -297,10 +297,12 @@ func (m Migrator) DropColumn(value interface{}, name string) error { | |||||||
| func (m Migrator) AlterColumn(value interface{}, field string) error { | func (m Migrator) AlterColumn(value interface{}, field string) error { | ||||||
| 	return m.RunWithValue(value, func(stmt *gorm.Statement) error { | 	return m.RunWithValue(value, func(stmt *gorm.Statement) error { | ||||||
| 		if field := stmt.Schema.LookUpField(field); field != nil { | 		if field := stmt.Schema.LookUpField(field); field != nil { | ||||||
|  | 			fileType := clause.Expr{SQL: m.DataTypeOf(field)} | ||||||
| 			return m.DB.Exec( | 			return m.DB.Exec( | ||||||
| 				"ALTER TABLE ? ALTER COLUMN ? TYPE ?", | 				"ALTER TABLE ? ALTER COLUMN ? TYPE ?", | ||||||
| 				clause.Table{Name: stmt.Table}, clause.Column{Name: field.DBName}, m.DB.Migrator().FullDataTypeOf(field), | 				clause.Table{Name: stmt.Table}, clause.Column{Name: field.DBName}, fileType, | ||||||
| 			).Error | 			).Error | ||||||
|  | 
 | ||||||
| 		} | 		} | ||||||
| 		return fmt.Errorf("failed to look up field with name: %s", field) | 		return fmt.Errorf("failed to look up field with name: %s", field) | ||||||
| 	}) | 	}) | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user
	 Jinzhu
						Jinzhu