refactoring

Author: cjilbert504

app/controllers/posts_controller.rb

@@ -22,33 +22,13 @@ def edit
   # POST /posts or /posts.json
   def create
     @post = Post.new(post_params)
-
-    if params[:publish]
-      if @post.publish
-        redirect_to @post, notice: "Post published"
-      else
-        render :new, status: :unprocessable_entity
-      end
-    else
-      @post.save
-      redirect_to @post, notice: "Draft saved"
-    end
+    save_post(:new)
   end
 
   # PATCH/PUT /posts/1 or /posts/1.json
   def update
     @post.assign_attributes(post_params)
-
-    if params[:publish]
-      if @post.publish
-        redirect_to @post, notice: "Post published"
-      else
-        render :edit, status: :unprocessable_entity
-      end
-    else
-      @post.save
-      redirect_to @post, notice: "Draft updated"
-    end
+    save_post(:edit)
   end
 
   # DELETE /posts/1 or /posts/1.json
@@ -68,4 +48,25 @@ def set_post
   def post_params
     params.expect(post: [ :title, :body ])
   end
+
+  def save_post(template)
+    if params[:publish]
+      publish_post(template)
+    else
+      save_draft
+    end
+  end
+
+  def publish_post(template)
+    if @post.publish
+      redirect_to @post, notice: "Post published"
+    else
+      render template, status: :unprocessable_entity
+    end
+  end
+
+  def save_draft
+    @post.save
+    redirect_to @post, notice: "Draft saved"
+  end
 end

app/models/post.rb

@@ -2,6 +2,6 @@ class Post < ApplicationRecord
   validates :title, :body, presence: true, on: :publish
 
   def publish
-    save if valid?(:publish)
+    save(context: :publish)
   end
 end