From efea296c7a5b535d7e152f694ccab8313f873103 Mon Sep 17 00:00:00 2001
From: Guo Xiang Tan <tgx_world@hotmail.com>
Date: Mon, 24 Oct 2016 12:02:38 +0800
Subject: [PATCH] FIX: Do not cook post if `Post#raw` has not been changed.

---
 app/models/post.rb       |  6 +++++-
 spec/models/post_spec.rb | 15 +++++++++++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/app/models/post.rb b/app/models/post.rb
index fcc5e0c8182..8fc89f682c6 100644
--- a/app/models/post.rb
+++ b/app/models/post.rb
@@ -522,7 +522,11 @@ class Post < ActiveRecord::Base
 
   before_save do
     self.last_editor_id ||= user_id
-    self.cooked = cook(raw, topic_id: topic_id) unless new_record?
+
+    if !new_record? && raw_changed?
+      self.cooked = cook(raw, topic_id: topic_id)
+    end
+
     self.baked_at = Time.new
     self.baked_version = BAKED_VERSION
   end
diff --git a/spec/models/post_spec.rb b/spec/models/post_spec.rb
index a5ceb4a391e..e70f176d4af 100644
--- a/spec/models/post_spec.rb
+++ b/spec/models/post_spec.rb
@@ -628,6 +628,21 @@ describe Post do
 
   end
 
+  describe 'before save' do
+    let(:cooked) { "<p><div class=\"lightbox-wrapper\"><a data-download-href=\"//localhost:3000/uploads/default/34784374092783e2fef84b8bc96d9b54c11ceea0\" href=\"//localhost:3000/uploads/default/original/1X/34784374092783e2fef84b8bc96d9b54c11ceea0.gif\" class=\"lightbox\" title=\"Sword reworks.gif\"><img src=\"//localhost:3000/uploads/default/optimized/1X/34784374092783e2fef84b8bc96d9b54c11ceea0_1_690x276.gif\" width=\"690\" height=\"276\"><div class=\"meta\">\n<span class=\"filename\">Sword reworks.gif</span><span class=\"informations\">1000x400 1000 KB</span><span class=\"expand\"></span>\n</div></a></div></p>" }
+
+    let(:post) do
+      Fabricate(:post,
+        raw: "<img src=\"/uploads/default/original/1X/34784374092783e2fef84b8bc96d9b54c11ceea0.gif\" width=\"690\" height=\"276\">",
+        cooked: cooked
+      )
+    end
+
+    it 'should not cook the post if raw has not been changed' do
+      post.save!
+      expect(post.cooked).to eq(cooked)
+    end
+  end
 
   describe 'after save' do