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

* Add check for empty set when dropping indexes during migration
go-gitea · Oct 12, 2019 · f1fdd78 · f1fdd78
1 parent 5e759b6
commit f1fdd78
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/models/migrations/migrations.go b/models/migrations/migrations.go
@@ -404,9 +404,11 @@ func dropTableColumns(sess *xorm.Session, tableName string, columnNames ...strin
 		}
 		for _, index := range res {
 			indexName := index["column_name"]
-			_, err := sess.Exec(fmt.Sprintf("DROP INDEX `%s` ON `%s`", indexName, tableName))
-			if err != nil {
-				return err
+			if len(indexName) > 0 {
+				_, err := sess.Exec(fmt.Sprintf("DROP INDEX `%s` ON `%s`", indexName, tableName))
+				if err != nil {
+					return err
+				}
 			}
 		}