From f17375b9d03ed420c83ed41d58e363a022606c68 Mon Sep 17 00:00:00 2001 From: Shinya Maeda Date: Mon, 4 Dec 2017 22:18:58 +0900 Subject: [PATCH] Optimize Clusters::CreateService#execute --- app/controllers/projects/clusters/user_controller.rb | 2 +- app/services/clusters/create_service.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/controllers/projects/clusters/user_controller.rb b/app/controllers/projects/clusters/user_controller.rb index 7f108c54b19..d7678512073 100644 --- a/app/controllers/projects/clusters/user_controller.rb +++ b/app/controllers/projects/clusters/user_controller.rb @@ -11,7 +11,7 @@ class Projects::Clusters::UserController < Projects::ApplicationController def create @cluster = ::Clusters::CreateService .new(project, current_user, create_params) - .execute(nil) + .execute if @cluster.persisted? redirect_to project_cluster_path(project, @cluster) diff --git a/app/services/clusters/create_service.rb b/app/services/clusters/create_service.rb index 1d407739b21..7b697f6d807 100644 --- a/app/services/clusters/create_service.rb +++ b/app/services/clusters/create_service.rb @@ -2,7 +2,7 @@ module Clusters class CreateService < BaseService attr_reader :access_token - def execute(access_token) + def execute(access_token = nil) @access_token = access_token create_cluster.tap do |cluster|