ProjectsController#create refactored to reduce it's complexity

Rafael Manzo
1 parent 252766fc
Showing 1 changed file with 14 additions and 9 deletions Show diff stats
app/controllers/projects_controller.rb
@@ -21,15 +21,7 @@ class ProjectsController &lt; ApplicationController
   def create
     @project = Project.new(project_params)
     respond_to do |format|
-      if @project.save
-        current_user.project_ownerships.create project_id: @project.id
-
-        format.html { redirect_to project_path(@project.id), notice: 'Project was successfully created.' }
-        format.json { render action: 'show', status: :created, location: @project }
-      else
-        format.html { render action: 'new' }
-        format.json { render json: @project.errors, status: :unprocessable_entity }
-      end
+      create_project_and_redir(format)
     end
   end
@@ -77,4 +69,17 @@ class ProjectsController &lt; ApplicationController
     def project_params
       params[:project]
     end
+
+    # Extracted code from create action
+    def create_project_and_redir(format)
+      if @project.save
+        current_user.project_ownerships.create project_id: @project.id
+
+        format.html { redirect_to project_path(@project.id), notice: 'Project was successfully created.' }
+        format.json { render action: 'show', status: :created, location: @project }
+      else
+        format.html { render action: 'new' }
+        format.json { render json: @project.errors, status: :unprocessable_entity }
+      end
+    end
 end