]> source.dussan.org Git - redmine.git/commitdiff
Code cleanup.
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Tue, 11 Dec 2012 19:42:12 +0000 (19:42 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Tue, 11 Dec 2012 19:42:12 +0000 (19:42 +0000)
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10984 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/controllers/custom_fields_controller.rb

index 59de860cd45b8d07c6b8a7ec90b4a5297194cef4..8c853df126ce46262e9a7b07b69f9847458ee74c 100644 (file)
@@ -31,7 +31,7 @@ class CustomFieldsController < ApplicationController
   end
 
   def create
-    if request.post? and @custom_field.save
+    if @custom_field.save
       flash[:notice] = l(:notice_successful_create)
       call_hook(:controller_custom_fields_new_after_save, :params => params, :custom_field => @custom_field)
       redirect_to custom_fields_path(:tab => @custom_field.class.name)
@@ -44,7 +44,7 @@ class CustomFieldsController < ApplicationController
   end
 
   def update
-    if request.put? and @custom_field.update_attributes(params[:custom_field])
+    if @custom_field.update_attributes(params[:custom_field])
       flash[:notice] = l(:notice_successful_update)
       call_hook(:controller_custom_fields_edit_after_save, :params => params, :custom_field => @custom_field)
       redirect_to custom_fields_path(:tab => @custom_field.class.name)
@@ -54,10 +54,11 @@ class CustomFieldsController < ApplicationController
   end
 
   def destroy
-    @custom_field.destroy
-    redirect_to custom_fields_path(:tab => @custom_field.class.name)
-  rescue
-    flash[:error] = l(:error_can_not_delete_custom_field)
+    begin
+      @custom_field.destroy
+    rescue
+      flash[:error] = l(:error_can_not_delete_custom_field)
+    end
     redirect_to custom_fields_path(:tab => @custom_field.class.name)
   end