mirror of
https://framagit.org/tykayn/mastodon.git
synced 2023-08-25 08:33:12 +02:00
1155dc0835
Fixes #12690 The `strong_migrations` update from ba2eac8824a85aa9541f8070ed7bcd22b9982cc8 introduced a check for `change_column_null` specific to Postgres. This rejects old migrations. This commit just wraps old migrations with `safety_assured` to bypass this check. Alternatives would have been to: - Disable that check entirely (a possibility added in that same `strong_migrations` version) for Mastodon, but it makes sense to write new migrations without such a strong lock. - Rewrite the old migrations to do it in a way that do not require an exclusive lock. I thought fixing those old migrations for performance wasn't worth the pain. Also, if I understand correctly, the next version of `strong_migrations` is going to include a helper to do that. We could update those migrations at that point.
20 lines
668 B
Ruby
20 lines
668 B
Ruby
class FixNullBooleans < ActiveRecord::Migration[5.1]
|
|
def change
|
|
safety_assured do
|
|
change_column_default :domain_blocks, :reject_media, false
|
|
change_column_null :domain_blocks, :reject_media, false, false
|
|
|
|
change_column_default :imports, :approved, false
|
|
change_column_null :imports, :approved, false, false
|
|
|
|
change_column_null :statuses, :sensitive, false, false
|
|
change_column_null :statuses, :reply, false, false
|
|
|
|
change_column_null :users, :admin, false, false
|
|
|
|
change_column_default :users, :otp_required_for_login, false
|
|
change_column_null :users, :otp_required_for_login, false, false
|
|
end
|
|
end
|
|
end
|