[CmdLog] Fix the exception - object has no attribute 'target' (#128)

Co-authored-by: Vexed <[email protected]>

And apply new black style

anotherpingcog/anotherpingcog.py

@@ -154,11 +154,13 @@ async def ping(self, ctx: commands.Context):
  embed.colour = colour
  await message.edit(
  content=(
- "Message Send is worse for slash commands. Try using the text command for "
- "a better result."
- )
- if ctx.interaction
- else None,
+ (
+ "Message Send is worse for slash commands. Try using the text command for "
+ "a better result."
+ )
+ if ctx.interaction
+ else None
+ ),
  embed=embed,
  )
 
@@ -501,9 +503,11 @@ async def settings(self, ctx: commands.Context):
  footer += (
  "Default - the default text will be used in the embed footer."
  if settings.footer == "default"
- else "None - there will not be any footer text in the embed."
- if settings.footer == "none"
- else f"Custom - {settings.footer}"
+ else (
+ "None - there will not be any footer text in the embed."
+ if settings.footer == "none"
+ else f"Custom - {settings.footer}"
+ )
  )
  main_embed.add_field(name="Footer", value=footer, inline=False)
 

betteruptime/commands.py

@@ -178,9 +178,11 @@ async def uptimegraph(self, ctx: commands.Context, num_days: int = 30):
 
  embed = discord.Embed(
  title="Daily uptime data for the last " + str(num_days) + " days",
- description=f"The top lowest uptime days (under `{labelled_pc}%`) will be labelled."
- if labelled_pc
- else None,
+ description=(
+ f"The top lowest uptime days (under `{labelled_pc}%`) will be labelled."
+ if labelled_pc
+ else None
+ ),
  colour=await ctx.embed_colour(),
  )
  embed.set_footer(text="Times are in UTC. This excludes today.")

cmdlog/objects.py

@@ -165,8 +165,8 @@ def __init__(
 
  if isinstance(target, discord.User):
  self.target = BasicDiscordObject(target.id, target.name)
- else:
- target = target
+ else: # partial message, already low mem usage so leave as is
+ self.target = target
 
  self.time = datetime.datetime.now().strftime(TIME_FORMAT)
 
@@ -232,6 +232,6 @@ def __sizeof__(self) -> int:
  size += getsizeof(self.channel)
  size += getsizeof(self.guild)
  size += getsizeof(self.time)
- size += getsizeof(self.target)
+ size += getsizeof(getattr(self, "target", 0))
 
  return size

docs/getting_started.rst

@@ -75,5 +75,5 @@ Cog name Summary
 Support
 -------
 
-If you want some help with my cogs, you can ask in the :ref:`Red - Cog Support<https://discord.gg/GD43Nb9H86>` server,
+If you want some help with my cogs, you can ask in the `Red - Cog Support<https://discord.gg/GD43Nb9H86>`_ server,
 in #support_vex-cogs.

docs/requirements.txt

@@ -1 +1,2 @@
 furo
+sphinx-rtd-theme

fivemstatus/loop.py

@@ -37,9 +37,9 @@ async def fivemstatus_loop(self) -> NoReturn:
  await self.loop_meta.sleep_until_next()
 
  async def update_messages(self) -> None:
- all_guilds: dict[
- int, dict[Literal["message"], MessageData]
- ] = await self.config.all_guilds()
+ all_guilds: dict[int, dict[Literal["message"], MessageData]] = (
+ await self.config.all_guilds()
+ )
  if not all_guilds:
  _log.debug("Nothing registered, nothing to do...")
  return