Search now uses interactive embed

cogs/search.py

@@ -1,40 +1,47 @@
+from inspect import currentframe
 import discord
 from discord.ext import commands
-from utils import jisho
+from utils import jisho, interactive
 
-class JishoObject():
+class SearchEmbed(interactive.InteractiveEmbed):
-    def __init__(self, query, owner):
+    REACTIONS = {
-        self.response = jisho.JishoResponse(query)
+        "prev_page": "⬅️",
-        self.total_pages = self.response.entries
+        "next_page": "➡️"
-        self.page = 0
+    }
-        self.owner = owner
 
-    def prev(self):
+    def __init__(self, parent, ctx, response):
-        self.page -= 1
+        super(SearchEmbed, self).__init__(parent.bot, ctx, 60.0)
-        if self.page < 0:
+        self.parent = parent
-            self.page = self.total_pages - 1
+        self.owner = self.ctx.author
+        self.response = response
 
-    def next(self):
+        self.current_page = 0
-        self.page += 1
-        if self.page >= self.total_pages:
-            self.page = 0
 
-class Search(commands.Cog):
+    def prev_page(self):
-    def __init__(self, bot: commands.Bot):
+        self.current_page -= 1
-        self.bot = bot
+        if self.current_page < 0:
-        self.activeObject = None
+            self.current_page = self.response.size - 1
-        self.latestMessage = 0
 
-    async def createEmbed(self):
+    def next_page(self):
-        if self.activeObject.total_pages == 0:
+        self.current_page += 1
-            embed = discord.Embed(
+        if self.current_page >= self.response.size:
-                title = "No search results",
+            self.current_page = 0
-                description = "The search returned nothing. Did you make a typo?",
+
-                colour = 0x56d926
+    async def on_reaction(self, reaction, user):
-            )
+        if reaction.emoji == SearchEmbed.REACTIONS["prev_page"]:
-        else:
+            self.prev_page()
-            response = self.activeObject.response
+            await reaction.remove(user)
-            node = response.nodes[self.activeObject.page]
+
+        if reaction.emoji == SearchEmbed.REACTIONS["next_page"]:
+            self.next_page()
+            await reaction.remove(user)
+
+    async def add_navigation(self, message):
+        await message.add_reaction(SearchEmbed.REACTIONS["prev_page"])
+        await message.add_reaction(SearchEmbed.REACTIONS["next_page"])
+
+    def make_embed(self):
+        node = self.response.nodes[self.current_page]
         embed = discord.Embed(
             title = node.japanese[0][0],
             url = f"https://jisho.org/word/{node.slug}",
@@ -58,49 +65,43 @@ class Search(commands.Cog):
 
 
         embed.set_footer(
-                text = f"{node.ftags} \t\t {self.activeObject.page + 1}/{self.activeObject.total_pages}"
+            text = f"{node.ftags} \t\t {self.current_page + 1}/{self.response.size}"
         )
 
         return embed
 
-    @commands.Cog.listener()
+    async def on_close(self):
-    async def on_reaction_add(self, reaction, user):
+        self.parent.activeObjects.pop(self.ctx.channel.id)
-        message = reaction.message
-        if message.id != self.latestMessage:
-            return
-
-        if user == self.bot.user:
-            return
-
-        if user.id != self.activeObject.owner:
-            return
-
-        if reaction.me:
-            if reaction.emoji == "⬅️":
-                self.activeObject.prev()
-                await reaction.remove(user)
-
-            if reaction.emoji == "➡️":
-                self.activeObject.next()
-                await reaction.remove(user)
-
-        embed = await self.createEmbed()
-        await message.edit(embed=embed)
 
+class Search(commands.Cog):
+    def __init__(self, bot: commands.Bot):
+        self.bot = bot
+        self.activeObjects = {}
 
     @commands.command(name="search", description="Searches Jisho", usage="<query>", aliases=["s"])
     @commands.cooldown(1, 5)
     async def search(self, ctx: commands.Context, *, query: str = None):
         if query is None:
+            embed = discord.Embed(
+                title = "No search results",
+                description = "The search returned nothing. Did you make a typo?",
+                colour = 0x56d926
+            )
+            await ctx.send(embed=embed)
             return
         
-        self.activeObject = JishoObject(query, ctx.author.id)
+        result = jisho.JishoResponse(query)
-        embed = await self.createEmbed()
+        if result.size == 0:
-        message = await ctx.send(embed=embed)
+            embed = discord.Embed(
-        self.latestMessage = message.id
+                title = "No search results",
-        if self.activeObject.total_pages > 0:
+                description = "The search returned nothing. Did you make a typo?",
-            await message.add_reaction("⬅️")
+                colour = 0x56d926
-            await message.add_reaction("➡️")
+            )
+            await ctx.send(embed=embed)
+            return
+
+        self.activeObjects[ctx.channel.id] = SearchEmbed(self, ctx, result)
+        await self.activeObjects[ctx.channel.id].show_embed()
 
     @search.error
     async def search_error(self, ctx, error):

utils/interactive.py

@@ -0,0 +1,93 @@
+import discord
+from discord.ext import commands
+from abc import ABC, abstractmethod
+import asyncio
+
+class InteractiveEmbed(ABC):
+    """
+    This abstract base class can be used to create interactive embeds
+    """
+    def __init__(self, bot, ctx, timeout):
+        """
+        Sets up the embed with needed parameters
+        bot: The bot hosting this embed
+        ctx: The context that caused this embed
+        timeuout: The time until the embed times out (in seconds)
+        """
+        self.bot = bot
+        self.ctx = ctx
+        self.message = None
+        self.timeout = timeout
+
+    @abstractmethod
+    async def on_reaction(self, reaction, user):
+        """
+        Gets called when the user interacted with the embed
+        """
+        pass
+
+    @abstractmethod
+    def make_embed(self):
+        """
+        Creates and returns a new embed
+        """
+        pass
+
+    @abstractmethod
+    async def add_navigation(self, message):
+        """
+        Adds the navigational emotes to the embed
+        """
+        pass
+
+    async def on_close(self):
+        """
+        Can be overridden. Gets called when the embed is closed
+        """
+        pass
+
+    def additional_checks(self, reaction, user):
+        """
+        Can be overridden. Additional checks to do before calling on_reaction()
+        """
+        return True
+
+    async def show_embed(self):
+        """
+        Displays an embed / Creates an embed
+        """
+        if self.message is None:
+            self.message = await self.ctx.send(embed=self.make_embed())
+            await self.add_navigation(self.message)
+
+        else:
+            await self.message.edit(embed=self.make_embed())
+
+        def check(reaction, user):
+            if self.message is None:
+                return False
+
+            if user.id != self.ctx.author.id:
+                return False
+
+            if reaction.message.id != self.message.id:
+                return False
+
+            if not reaction.me:
+                return False
+
+            return self.additional_checks(reaction, user)
+
+        try:
+            reaction, user = await self.bot.wait_for("reaction_add", check=check, timeout=self.timeout)
+            await self.on_reaction(reaction, user)
+            await self.show_embed()
+        except asyncio.TimeoutError:
+            await self.close_embed()
+
+    async def close_embed(self):
+        """
+        Close this embed
+        """
+        await self.message.clear_reactions()
+        await self.on_close()

utils/jisho.py

@@ -64,6 +64,8 @@ class JishoResponse():
         self.nodes = []
         self.disassemble()
 
+        self.size = len(self.nodes)
+
     def query(self):
         url = TEMPLATE_URL.format(urllib.parse.quote_plus(self.query_string))
         r = requests.get(url, headers=HEADER)