From 4ef91c8b1564f3220f873743c012274a48de8587 Mon Sep 17 00:00:00 2001 From: Andrei Date: Fri, 12 May 2017 06:50:29 -0700 Subject: [PATCH] Replace Message.get_reactors w/ Message.get_reactors_iter Ideally this is the only implementation that makes sense. Users wishing to only load some reactors should use the actual HTTP client routes. --- disco/types/message.py | 24 +++++------------------- 1 file changed, 5 insertions(+), 19 deletions(-) diff --git a/disco/types/message.py b/disco/types/message.py index 7b5bcb0..07b44af 100644 --- a/disco/types/message.py +++ b/disco/types/message.py @@ -301,34 +301,20 @@ class Message(SlottedModel): """ return self.client.api.channels_messages_delete(self.channel_id, self.id) - def get_reactors_iter(self, emoji, *args, **kwargs): - """ - Returns an iterator which paginates the reactors for the given emoji. - """ - if isinstance(emoji, Emoji): - emoji = emoji.to_string() - - return Paginator( - self.client.api.channels_messages_reactions_get, - self.channel_id, - self.id, - emoji, - *args, - **kwargs) - def get_reactors(self, emoji, *args, **kwargs): """ - Returns an list of users who reacted to this message with the given emoji. + Returns an iterator which paginates the reactors for the given emoji. Returns ------- - list(:class:`User`) - The users who reacted. + Paginator(`User`) + An iterator which handles pagination of reactors. """ if isinstance(emoji, Emoji): emoji = emoji.to_string() - return self.client.api.channels_messages_reactions_get( + return Paginator( + self.client.api.channels_messages_reactions_get, self.channel_id, self.id, emoji,