diff --git a/app/models/reviewable.rb b/app/models/reviewable.rb index 13468019e3e..78d4b98e650 100644 --- a/app/models/reviewable.rb +++ b/app/models/reviewable.rb @@ -133,7 +133,7 @@ class Reviewable < ActiveRecord::Base target: target, topic: topic, created_by: created_by, - reviewable_by_moderator: reviewable_by_moderator && !(topic&.private_message?), + reviewable_by_moderator: reviewable_by_moderator, payload: payload, potential_spam: potential_spam ) diff --git a/spec/models/reviewable_spec.rb b/spec/models/reviewable_spec.rb index 30bd5d68550..9b46eb7c529 100644 --- a/spec/models/reviewable_spec.rb +++ b/spec/models/reviewable_spec.rb @@ -78,17 +78,6 @@ RSpec.describe Reviewable, type: :model do expect(r1.pending?).to eq(true) expect(r0.pending?).to eq(false) end - - it 'sets the reviewable_by_moderator attribute to false when the topic is a PM' do - pm_topic = Fabricate(:private_message_topic) - post = Fabricate(:post, topic: pm_topic) - reviewable = ReviewableFlaggedPost.needs_review!( - target: post, topic: pm_topic, - created_by: Discourse.system_user, reviewable_by_moderator: true - ) - - expect(reviewable.reviewable_by_moderator).to eq(false) - end end context ".list_for" do