pub-solar/forgejo
5
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Add check for empty set when dropping indexes during migration (#8471)

Browse source 
* Add check for empty set when dropping indexes during migration
This commit is contained in:
guillep2k 2019-10-12 01:55:07 -03:00 committed by zeripath
parent 5e759b60cc
commit f1fdd782d5
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
models/migrations/migrations.go
View file

@ -404,9 +404,11 @@ func dropTableColumns(sess *xorm.Session, tableName string, columnNames ...strin
} }
for _, index := range res { for _, index := range res {
indexName := index["column_name"] indexName := index["column_name"]
_, err := sess.Exec(fmt.Sprintf("DROP INDEX `%s` ON `%s`", indexName, tableName)) if len(indexName) > 0 {
if err != nil { _, err := sess.Exec(fmt.Sprintf("DROP INDEX `%s` ON `%s`", indexName, tableName))
return err if err != nil {
return err
}
} }
} }