summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
authorMadeline <46743919+MaddyUnderStars@users.noreply.github.com>2023-07-18 21:04:52 +1000
committerGitHub <noreply@github.com>2023-07-18 21:04:52 +1000
commit5f22faa57b7104ba29a01889a1bafc35b3be4f02 (patch)
treeada83c96958cf0202c9f05bc6ddcc909b528d517 /src
parentMerge pull request #1066 from Zert3x/master (diff)
parentfix get channel messages around parameter (diff)
downloadserver-5f22faa57b7104ba29a01889a1bafc35b3be4f02.tar.xz
Merge pull request #1075 from SpecificProtagonist/get_messages_around
Fix get channel messages `around` parameter
Diffstat (limited to 'src')
-rw-r--r--src/api/routes/channels/#channel_id/messages/index.ts21
1 files changed, 11 insertions, 10 deletions
diff --git a/src/api/routes/channels/#channel_id/messages/index.ts b/src/api/routes/channels/#channel_id/messages/index.ts

index f031fa75..a15b462d 100644 --- a/src/api/routes/channels/#channel_id/messages/index.ts +++ b/src/api/routes/channels/#channel_id/messages/index.ts
@@ -20,7 +20,6 @@ import { handleMessage, postHandleMessage, route } from "@spacebar/api"; import { Attachment, Channel, - ChannelType, Config, DmChannelDTO, FieldErrors, @@ -93,8 +92,6 @@ router.get( if (limit < 1 || limit > 100) throw new HTTPError("limit must be between 1 and 100", 422); - const halfLimit = Math.floor(limit / 2); - const permissions = await getPermission( req.user_id, channel.guild_id, @@ -121,24 +118,28 @@ router.get( ], }; + let messages: Message[]; if (after) { if (BigInt(after) > BigInt(Snowflake.generate())) return res.status(422); query.where.id = MoreThan(after); + messages = await Message.find(query); } else if (before) { if (BigInt(before) < BigInt(req.params.channel_id)) return res.status(422); query.where.id = LessThan(before); + messages = await Message.find(query); } else if (around) { - query.where.id = [ - MoreThan((BigInt(around) - BigInt(halfLimit)).toString()), - LessThan((BigInt(around) + BigInt(halfLimit)).toString()), - ]; - - return res.json([]); // TODO: fix around + query.take = Math.floor(limit / 2); + query.where.id = LessThan(around); + const messages_before = await Message.find(query); + query.where.id = MoreThan(around); + const messages_after = await Message.find(query); + messages = messages_before.concat(messages_after); + } else { + throw new HTTPError("after, around or before must be present", 422); } - const messages = await Message.find(query); const endpoint = Config.get().cdn.endpointPublic; return res.json(