From 041481f2b64a01fa6ccbbb9da097693bf51fbb06 Mon Sep 17 00:00:00 2001 From: float-trip Date: Fri, 7 Jul 2023 18:53:14 +0000 Subject: [PATCH] Fix /comment JSON response (#167) Since `replies` is no longer added to the response, returning the JSON for the top level comment makes it impossible to get the JSON for a reply. Btw - you might want to remove the `author` field [here](https://fsdfsd.net/rDrama/rDrama/src/commit/60b5b33a7932e8f4d29ca47364352540cf43566e/files/classes/comment.py#L308). It attaches the full user bio and badge list to every comment, which turns something like `GET https://rdrama.net/post/18459` into a 21MB response. `author_name` is pretty much all anyone needs anyway Co-authored-by: float-trip Reviewed-on: https://fsdfsd.net/rDrama/rDrama/pulls/167 Co-authored-by: float-trip Co-committed-by: float-trip --- files/routes/comments.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/files/routes/comments.py b/files/routes/comments.py index 21d060e70..6bd05edb0 100644 --- a/files/routes/comments.py +++ b/files/routes/comments.py @@ -75,7 +75,7 @@ def post_pid_comment_cid(cid, pid=None, anything=None, v=None, sub=None): execute_shadowban_viewers_and_voters(v, post) execute_shadowban_viewers_and_voters(v, comment) - if v and v.client: return top_comment.json + if v and v.client: return comment.json else: if post.is_banned and not (v and (v.admin_level >= PERMS['POST_COMMENT_MODERATION'] or post.author_id == v.id)): template = "post_banned.html" else: template = "post.html"