IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-25 18:55:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: some posters were not getting added to topic_allowed_users when moving posts to a new PM

Browse Source 
If a user posted twice in a topic then subsequent posters were not getting added as topic_allowed_users.
This commit is contained in:
Arpit Jalan
2019-02-11 17:04:21 +05:30
parent 2c12336c6b
commit dcd7b92532
2 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
spec/models/post_mover_spec.rb
View File

@@ -580,13 +580,15 @@ describe PostMover do
context 'move to new message' do
it "adds post users as topic allowed users" do
personal_message.move_posts(admin, [p2.id, p5.id], title: "new testing message name", tags: ["tag1", "tag2"], archetype: "private_message")
personal_message.move_posts(admin, [p2.id, p3.id, p4.id, p5.id], title: "new testing message name", tags: ["tag1", "tag2"], archetype: "private_message")
p2.reload
expect(p2.topic.archetype).to eq(Archetype.private_message)
expect(p2.topic.topic_allowed_users.where(user_id: another_user.id).count).to eq(1)
expect(p2.topic.topic_allowed_users.where(user_id: evil_trout.id).count).to eq(1)
expect(p2.topic.tags.pluck(:name)).to eq([])
destination_topic = p2.topic
expect(destination_topic.archetype).to eq(Archetype.private_message)
expect(destination_topic.topic_allowed_users.where(user_id: user.id).count).to eq(1)
expect(destination_topic.topic_allowed_users.where(user_id: another_user.id).count).to eq(1)
expect(destination_topic.topic_allowed_users.where(user_id: evil_trout.id).count).to eq(1)
expect(destination_topic.tags.pluck(:name)).to eq([])
end
it "can add tags to new message when allow_staff_to_tag_pms is enabled" do