From 6ceca143b41c798c20f59d7810691d16a418367a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Thu, 10 Oct 2019 02:54:20 +0200 Subject: [PATCH] Remove redundant check deleting content blocks The `find` method raises an exception if nothing is found, so there's no need to check if it found something. --- .../admin/site_customization/content_blocks_controller.rb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/app/controllers/admin/site_customization/content_blocks_controller.rb b/app/controllers/admin/site_customization/content_blocks_controller.rb index eb6076dbe..d92dbc88e 100644 --- a/app/controllers/admin/site_customization/content_blocks_controller.rb +++ b/app/controllers/admin/site_customization/content_blocks_controller.rb @@ -67,8 +67,7 @@ class Admin::SiteCustomization::ContentBlocksController < Admin::SiteCustomizati end def delete_heading_content_block - heading_content_block = Budget::ContentBlock.find(params[:id]) - heading_content_block.destroy if heading_content_block + Budget::ContentBlock.find(params[:id]).destroy notice = t("admin.site_customization.content_blocks.destroy.notice") redirect_to admin_site_customization_content_blocks_path, notice: notice end