github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[jOOQ/jOOQ#18450] Diff should drop PRIMARY KEY first, then NOT NULL constraint

Browse Source
This commit is contained in:
Lukas Eder 2025-05-12 14:53:51 +02:00
parent f1d983d40d
commit 35b7961c75
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
jOOQ/src/main/java/org/jooq/impl/Diff.java
View File

@ -981,14 +981,20 @@ final class Diff {
// [#18044] DROP CONSTRAINT / INDEX before everything, ADD CONSTRAINT / INDEX after everything
// [#18383] FOREIGN KEY must be dropped before other constraints, or added after other constraints
// [#18450] DROP NOT NULL must happen after dropping constraints, SET NOT NULL before adding constraints
if (q instanceof AlterTableImpl a) {
return a.$dropConstraint() instanceof QOM.ForeignKey || a.$dropConstraintType() == FOREIGN_KEY
return
a.$dropConstraint() instanceof QOM.ForeignKey || a.$dropConstraintType() == FOREIGN_KEY
? -3
: a.$dropConstraint() != null || a.$dropConstraintType() != null
? -2
: a.$dropConstraint() != null
: a.$alterColumnNullability() == Nullability.NULL
? -1
: a.$addConstraint() instanceof QOM.ForeignKey || a.$dropConstraintType() == FOREIGN_KEY
? 2
? 3
: a.$addConstraint() != null
? 2
: a.$alterColumnNullability() == Nullability.NOT_NULL
? 1
: 0;
}