consul/grecia
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove redundant check deleting content blocks

Browse Source 
The `find` method raises an exception if nothing is found, so there's no
need to check if it found something.
This commit is contained in:
Javi Martín
2019-10-10 02:54:20 +02:00
parent aace7aea02
commit 6ceca143b4
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/controllers/admin/site_customization/content_blocks_controller.rb
View File

@@ -67,8 +67,7 @@ class Admin::SiteCustomization::ContentBlocksController < Admin::SiteCustomizati
end end
def delete_heading_content_block def delete_heading_content_block
heading_content_block = Budget::ContentBlock.find(params[:id]) Budget::ContentBlock.find(params[:id]).destroy
heading_content_block.destroy if heading_content_block
notice = t("admin.site_customization.content_blocks.destroy.notice") notice = t("admin.site_customization.content_blocks.destroy.notice")
redirect_to admin_site_customization_content_blocks_path, notice: notice redirect_to admin_site_customization_content_blocks_path, notice: notice
end end