diff options
| author | Claire <claire.github-309c@sitedethib.com> | 2021-03-19 02:45:34 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-19 02:45:34 +0100 |
| commit | b3582298341e32528929c6f3292e36a6fa261ba5 (patch) | |
| tree | c98c19b47de969cbd9f3b3f29bb22ba41eb04cae /app/controllers/admin/domain_blocks_controller.rb | |
| parent | 55ac2b9c6085def9e692fa69b849239c1249d9fd (diff) | |
Further preparation for Rails 6 (#15916)
* Use ActiveRecord::Result#to_ary instead of deprecated to_hash They do the same thing, and to_hash has been removed from Rails 6.1 * Explicitly name polymorphic indexes to workaround a bug in Rails 6.1 cf. https://github.com/rails/rails/issues/41693 * Fix incorrect usage of “foreign_key” in migration script * Use `ActiveModel::Errors#delete` instead of deprecated clear method * Fix link headers tests on Rails 6.1 Rails 6.1 adds values to the Link header by default, thus it is not a LinkHeader object anymore. Fix the test to parse the Link header instead of assuming it is a LinkHeader.
Diffstat (limited to 'app/controllers/admin/domain_blocks_controller.rb')
| -rw-r--r-- | app/controllers/admin/domain_blocks_controller.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/controllers/admin/domain_blocks_controller.rb b/app/controllers/admin/domain_blocks_controller.rb index ba927b04a..b140c454c 100644 --- a/app/controllers/admin/domain_blocks_controller.rb +++ b/app/controllers/admin/domain_blocks_controller.rb @@ -22,7 +22,7 @@ module Admin if existing_domain_block.present? && !@domain_block.stricter_than?(existing_domain_block) @domain_block.save flash.now[:alert] = I18n.t('admin.domain_blocks.existing_domain_block_html', name: existing_domain_block.domain, unblock_url: admin_domain_block_path(existing_domain_block)).html_safe # rubocop:disable Rails/OutputSafety - @domain_block.errors[:domain].clear + @domain_block.errors.delete(:domain) render :new else if existing_domain_block.present? |