phinx
phinx copied to clipboard
SQL Server adapter uses nvarchar(max) instead of ntext and always updates default constraint
The first commit of this PR handles #1972, which is to remove the use of NTEXT and replace it with NVARCHAR(MAX). The existing NTEXT and TEXT parsing to the Phinx TEXT type still exists for compatibility reasons.
The second commit forces default constraints to always be dropped and readded, as SQL Server requires this to be removed if the column is ever changed.
Any update? Or shall we close this?
Went ahead and updated these. Sorry on the delay.
Do we want to continue and merge? @MasterOdin
Ping @MasterOdin
I wonder if we might need a test case here to back things up.