]> source.dussan.org Git - gitea.git/commit
Prevent primary key update on migration (#26192)
authorKN4CK3R <admin@oldschoolhack.me>
Fri, 28 Jul 2023 07:54:31 +0000 (09:54 +0200)
committerGitHub <noreply@github.com>
Fri, 28 Jul 2023 07:54:31 +0000 (09:54 +0200)
commit7866a6e0e25cf87b298e046bb38f4d68c35529a7
tree324e75f1da5229cc5ede217866de3f991a8a953e
parent494ee56de28a52568413dfa30b5ddd99411fc2fc
Prevent primary key update on migration (#26192)

Fixes #25918

The migration fails on MSSQL because xorm tries to update the primary
key column. xorm prevents this if the column is marked as auto
increment:

https://gitea.com/xorm/xorm/src/commit/c622cdaf893fbfe3f40a6b79f6bc17ee10f53865/internal/statements/update.go#L38-L40

I think it would be better if xorm would check for primary key columns
here because updating such columns is bad practice. It looks like if
that auto increment check should do the same.

fyi @lunny
models/migrations/v1_20/v250.go