codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-12-17 19:12:37 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

FIX: setting owner on topic should not bump topic

Browse source
This commit is contained in:
Sam Saffron 2016-03-17 08:32:27 +11:00
parent 20ce7f29e0
commit 8577b7e2cc
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/models/post.rb
View file

@ -439,7 +439,7 @@ class Post < ActiveRecord::Base
new_user: new_user.username_lower new_user: new_user.username_lower
) )
revise(actor, { raw: self.raw, user_id: new_user.id, edit_reason: edit_reason }) revise(actor, {raw: self.raw, user_id: new_user.id, edit_reason: edit_reason}, bypass_bump: true)
if post_number == topic.highest_post_number if post_number == topic.highest_post_number
topic.update_columns(last_post_user_id: new_user.id) topic.update_columns(last_post_user_id: new_user.id)

3
spec/services/post_owner_changer_spec.rb
View file

@ -20,9 +20,12 @@ describe PostOwnerChanger do
end end
it "changes the user" do it "changes the user" do
bumped_at = topic.bumped_at
old_user = p1.user old_user = p1.user
PostOwnerChanger.new(post_ids: [p1.id], topic_id: topic.id, new_owner: user_a, acting_user: editor).change_owner! PostOwnerChanger.new(post_ids: [p1.id], topic_id: topic.id, new_owner: user_a, acting_user: editor).change_owner!
p1.reload p1.reload
expect(p1.topic.bumped_at).to eq(bumped_at)
expect(p1.topic.last_post_user_id).to eq(user_a.id) expect(p1.topic.last_post_user_id).to eq(user_a.id)
expect(old_user).not_to eq(p1.user) expect(old_user).not_to eq(p1.user)
expect(p1.user).to eq(user_a) expect(p1.user).to eq(user_a)