[WIP] Post に更新者の情報を持てるようにする [closes #66]

app/assets/stylesheets/posts.scss

@@ -79,16 +79,18 @@
   }
 }
 
-.author {
+.user-information {
   border-top: 1px solid $border-gray;
   padding: 1.5rem 0;
   text-align: right;
 }
 
-.author__nickname {
+.author__nickname,
+.modifier__nickname {
   font-size: .9rem
 }
 
-.author__created-at {
+.author__created-at,
+.modifier__created-at {
   font-size: .8rem
 }

app/controllers/posts_controller.rb

@@ -26,7 +26,7 @@ def create
   end
 
   def update
-    if @post.update(post_params)
+    if @post.update(post_params.merge(updated_by_id: current_user.id))
       redirect_to @post, notice: 'Post was successfully updated.'
     else
       render :edit

app/models/post.rb

@@ -5,6 +5,14 @@ class Post < ApplicationRecord
 
   has_paper_trail
 
+  before_validation :copy_created_by_id_to_updated_by_id, on: :create
+
+  private
+
+  def copy_created_by_id_to_updated_by_id
+    self.updated_by_id = created_by_id
+  end
+
   concerning :Notifiable do
     included do
       after_create_commit :notify_to_create if slack_notification_enabled?

app/views/posts/show.html.slim

@@ -5,15 +5,26 @@
         = @post.title
       .post__body
         == sanitize(auto_link(markdown_to_html(emojify(@post.body))))
-      .author
+      .user-information
         .row
-          .col.s3.offset-s9
-            .author__nickname
-              | Created by
-              span.blue-text
-                |  #{@post.created_by.nickname}
-            .author__created-at.grey-text.text-darken-1
-              = @post.created_at
+          .col.s6.offset-s6
+            .author
+              .author__nickname
+                | Created by
+                span.blue-text
+                  |  #{@post.created_by.nickname}
+              .author__created-at.grey-text.text-darken-1
+                = @post.created_at
+        .row
+          .col.s6.offset-s6
+            .updater
+              .modifier__nickname
+                | Updated by
+                span.blue-text
+                  / TODO N+1
+                  |  #{User.find(@post.updated_by_id).nickname}
+              .modifier__created-at.grey-text.text-darken-1
+                = @post.updated_at
 .row
   .col.s12
     = link_to edit_post_path(@post), class: 'waves-effect waves-light btn' do

db/migrate/20160828130911_add_updated_by_id_to_posts.rb

@@ -0,0 +1,6 @@
+class AddUpdatedByIdToPosts < ActiveRecord::Migration[5.0]
+  def change
+    add_reference :posts, :updated_by, references: :users, index: true
+    add_foreign_key :posts, :users, column: :updated_by_id
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160827120218) do
+ActiveRecord::Schema.define(version: 20160828130911) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -21,7 +21,9 @@
     t.integer  "created_by_id"
     t.datetime "created_at",    null: false
     t.datetime "updated_at",    null: false
+    t.integer  "updated_by_id"
     t.index ["created_by_id"], name: "index_posts_on_created_by_id", using: :btree
+    t.index ["updated_by_id"], name: "index_posts_on_updated_by_id", using: :btree
   end
 
   create_table "users", force: :cascade do |t|
@@ -44,4 +46,5 @@
   end
 
   add_foreign_key "posts", "users", column: "created_by_id"
+  add_foreign_key "posts", "users", column: "updated_by_id"
 end

test/models/post_test.rb

@@ -6,4 +6,13 @@ class PostTest < ActiveSupport::TestCase
     post.body = 'Alice was beginning to get ...'
     refute post.valid?
   end
+
+  test 'should set updater same as creator before validation on create' do
+    post = Post.new
+    post.title = 'Alice In Wonderland'
+    post.body = 'Alice was beginning to get ...'
+    post.created_by_id = users(:alice).id
+    post.valid?
+    assert_equal users(:alice).id, post.updated_by_id
+  end
 end