From 8dc2be80ef56df9be14659201397db99c93e1df4 Mon Sep 17 00:00:00 2001
From: uu59 <k@uu59.org>
Date: Wed, 4 Feb 2015 16:13:48 +0900
Subject: [PATCH] Separate fat action

---
 .../fluentd/settings_controller.rb            | 36 +++++++++++--------
 1 file changed, 21 insertions(+), 15 deletions(-)

diff --git a/app/controllers/fluentd/settings_controller.rb b/app/controllers/fluentd/settings_controller.rb
index 96fe0da..b9ef70f 100644
--- a/app/controllers/fluentd/settings_controller.rb
+++ b/app/controllers/fluentd/settings_controller.rb
@@ -18,22 +18,9 @@ class Fluentd::SettingsController < ApplicationController
 
   def  update
     if params[:dryrun]
-      if dryrun(params[:config])
-        flash.now[:success] = I18n.t('messages.dryrun_is_passed')
-      else
-        flash.now[:danger] = @fluentd.agent.last_error_message
-      end
-      @config = params[:config]
-      render "edit"
+      handle_dryrun
     else
-      begin
-        update_config(params[:config])
-        redirect_to daemon_setting_path(@fluentd)
-      rescue Fluent::ConfigParseError => e
-        @config = params[:config]
-        flash.now[:danger] = e.message
-        render "edit"
-      end
+      handle_update
     end
   end
 
@@ -48,6 +35,25 @@ class Fluentd::SettingsController < ApplicationController
     @config = @fluentd.agent.config
   end
 
+  def handle_dryrun
+    if dryrun(params[:config])
+      flash.now[:success] = I18n.t('messages.dryrun_is_passed')
+    else
+      flash.now[:danger] = @fluentd.agent.last_error_message
+    end
+    @config = params[:config]
+    render "edit"
+  end
+
+  def handle_update
+    update_config(params[:config])
+    redirect_to daemon_setting_path(@fluentd)
+  rescue Fluent::ConfigParseError => e
+    @config = params[:config]
+    flash.now[:danger] = e.message
+    render "edit"
+  end
+
   def dryrun(conf)
     tmpfile = Tempfile.open("fluentd-test-config")
     tmpfile.write params[:config]