github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-01-18 14:42:46 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FIX: Always include the first poster when converting to PM (#16187)

Browse Source 
Regressed in #15626
This commit is contained in:
Jarek Radosz 2022-03-15 10:51:10 +01:00 committed by GitHub
parent de9a031073
commit 383f164672
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/models/topic_converter.rb
View File

@ -63,7 +63,7 @@ class TopicConverter
private private
def posters def posters
@posters ||= @topic.posts.where("post_number > 1").distinct.pluck(:user_id) @posters ||= @topic.posts.distinct.pluck(:user_id)
end end
def increment_users_post_count def increment_users_post_count

5
spec/models/topic_converter_spec.rb
View File

@ -200,6 +200,11 @@ describe TopicConverter do
expect(private_message.topic_allowed_users.count).to eq(1) expect(private_message.topic_allowed_users.count).to eq(1)
end end
it "includes the poster of a single-post topic" do
moderator = Fabricate(:moderator)
private_message = topic.convert_to_private_message(moderator)
expect(private_message.allowed_users).to match_array([topic.user, moderator])
end
end end
context 'topic has replies' do context 'topic has replies' do