mirror of
https://github.com/discourse/discourse.git
synced 2025-02-25 18:55:32 -06:00
FIX: Attribute pending post to author in PendingPostSerialier
(#23369)
This fixes a regression introduced by an earlier change which changed `ReviewableQueuedPost` record creation to use the more appropriate `target_created_by_id` for the author of the post being queued instead of setting it to the creator(system user) of the `ReviewableQueuedPost` record.
This commit is contained in:
parent
4f8d52bbcb
commit
9f42a235ab
@ -15,7 +15,11 @@ class PendingPostSerializer < ApplicationSerializer
|
|||||||
|
|
||||||
delegate :created_by, :payload, :topic, to: :object, private: true
|
delegate :created_by, :payload, :topic, to: :object, private: true
|
||||||
delegate :url, to: :topic, prefix: true, allow_nil: true
|
delegate :url, to: :topic, prefix: true, allow_nil: true
|
||||||
delegate :avatar_template, :name, :username, to: :created_by, allow_nil: true
|
delegate :avatar_template, :name, :username, to: :target_created_by, allow_nil: true
|
||||||
|
|
||||||
|
def created_by_id
|
||||||
|
object.target_created_by_id
|
||||||
|
end
|
||||||
|
|
||||||
def raw_text
|
def raw_text
|
||||||
payload["raw"]
|
payload["raw"]
|
||||||
@ -24,4 +28,10 @@ class PendingPostSerializer < ApplicationSerializer
|
|||||||
def title
|
def title
|
||||||
payload["title"] || topic&.title
|
payload["title"] || topic&.title
|
||||||
end
|
end
|
||||||
|
|
||||||
|
private
|
||||||
|
|
||||||
|
def target_created_by
|
||||||
|
object.target_created_by
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
@ -4,7 +4,7 @@ RSpec.describe PendingPostSerializer do
|
|||||||
subject(:serializer) { described_class.new(post, scope: guardian, root: false) }
|
subject(:serializer) { described_class.new(post, scope: guardian, root: false) }
|
||||||
|
|
||||||
let(:guardian) { Guardian.new(author) }
|
let(:guardian) { Guardian.new(author) }
|
||||||
let(:author) { post.created_by }
|
let(:author) { post.target_created_by }
|
||||||
|
|
||||||
before { freeze_time }
|
before { freeze_time }
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user