From 87af2711572590668b59600cbe175c5231142719 Mon Sep 17 00:00:00 2001
From: aikaterna <20862007+aikaterna@users.noreply.github.com>
Date: Fri, 17 Feb 2023 09:12:43 -0800
Subject: [PATCH] Horoscope - Fix date text

On horo and chinese horo commands, the date regex was returning a lot of html prepended on the date, causing the response to be over 4000 chars. I added a separate date regex and simplified the original regex for finding the horoscope. Granted I'm not a regex pro by any means so feel free to adjust this PR if you accept it. Seemed to work fine in all my testing of various astral signs and chinese signs.
---
 horoscope/horoscope.py | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/horoscope/horoscope.py b/horoscope/horoscope.py
index 454d216..9710d28 100644
--- a/horoscope/horoscope.py
+++ b/horoscope/horoscope.py
@@ -83,7 +83,8 @@ async def _horoscope(self, ctx, *, sign: str):
             "chinese": "http://www.horoscope.com/us/horoscopes/chinese/horoscope-chinese-daily-today.aspx?sign=",
         }
         regex = [
-            r"<strong( class=\"date\"|)>([^`]*?)<\/strong> - ([^`]*?)\n",
+            r"<\/strong> - ([^`]*?)\n",
+            r"\w+\s\d+,\s\d+",
         ]
         try:
             horos = sign.split(", ")
@@ -106,8 +107,9 @@ async def _horoscope(self, ctx, *, sign: str):
                 async with self.session.get(uir, headers=option) as resp:
                     test = str(await resp.text('ISO-8859-1'))
                     msg = re.findall(regex[0], test)[0]
-                    msg_content = msg[2].replace("</p>", "")
-                    msg = msg_content + " - " + msg[1]
+                    date = re.findall(regex[1], test)[0]
+                    msg_content = msg.replace("</p>", "")
+                    msg = msg_content + " - " + date
                     await ctx.send(
                         "Today's chinese horoscope for the one"
                         " born in the year of the {} is:\n".format(sign) + box(msg)
@@ -128,8 +130,9 @@ async def _horoscope(self, ctx, *, sign: str):
                 async with self.session.get(uir, headers=option) as resp:
                     test = str(await resp.text('ISO-8859-1'))
                     msg = re.findall(regex[0], test)[0]
-                    msg_content = msg[2].replace("</p>", "")
-                    msg = msg_content + " - " + msg[1]
+                    date = re.findall(regex[1], test)[0]
+                    msg_content = msg.replace("</p>", "")
+                    msg = msg_content + " - " + date
                     if style == "love":
                         await ctx.send(
                             "Today's love horoscope for **{}** is:\n".format(sign) + box(msg)