[Warnings] Add setting for warn channel and DMing warns toggle

changelog.d/warnings/2929.enhance.rst

@@ -0,0 +1,2 @@
+Added features to mod to be able to toggle warns being sent to both DM's and channel.
+Added a command to set the channel the warns get sent to. (defaults to the channel the warn was issued in). 

redbot/cogs/warnings/warnings.py

@@ -1,6 +1,7 @@
 from collections import namedtuple
 from typing import Union, Optional
 
+import contextlib
 import discord
 
 from redbot.cogs.warnings.helpers import (
@@ -24,7 +25,14 @@
 class Warnings(commands.Cog):
     """Warn misbehaving users and take automated actions."""
 
-    default_guild = {"actions": [], "reasons": {}, "allow_custom_reasons": False}
+    default_guild = {
+        "actions": [],
+        "reasons": {},
+        "allow_custom_reasons": False,
+        "toggle_dm": True,
+        "warn_channel": None,
+        "toggle_channel": False,
+    }
 
     default_member = {"total_points": 0, "status": "", "warnings": {}}
 
@@ -76,6 +84,49 @@ async def allowcustomreasons(self, ctx: commands.Context, allowed: bool):
         else:
             await ctx.send(_("Custom reasons have been disabled."))
 
+    @warningset.command()
+    @commands.guild_only()
+    async def toggledm(self, ctx: commands.Context):
+        """Toggle whether warnings should be sent to users in DMs."""
+        guild = ctx.guild
+        toggle = not await self.config.guild(guild).toggle_dm()
+        await self.config.guild(guild).toggle_dm.set(toggle)
+        if toggle:
+            await ctx.send(_("Warnings will now be sent to users in DMs."))
+        else:
+            await ctx.send(_("Warnings will no longer be sent to users in DMs."))
+
+    @warningset.command()
+    @commands.guild_only()
+    async def warnchannel(self, ctx: commands.Context, channel: discord.TextChannel = None):
+        """Set the channel where warnings should be sent to.
+
+        Leave empty to use the channel `[p]warn` command was called in.
+        """
+        guild = ctx.guild
+        if channel:
+            await self.config.guild(guild).warn_channel.set(channel.id)
+            await ctx.send(
+                _("The warn channel has been set to {channel}").format(channel=channel.mention)
+            )
+        else:
+            await self.config.guild(guild).warn_channel.set(channel)
+            await ctx.send(_("Warnings will now be sent in the channel command was used in."))
+
+    @warningset.command()
+    @commands.guild_only()
+    async def togglechannel(self, ctx: commands.Context):
+        """
+        Toggle if warnings should be sent to a channel set with `[p]warningset warnchannel`
+        """
+        guild = ctx.guild
+        toggle = await self.config.guild(guild).toggle_channel()
+        await self.config.guild(guild).toggle_channel.set(not toggle)
+        if not toggle:
+            await ctx.send(_("Warnings will now be sent to set warn channel."))
+        else:
+            await ctx.send(_("Warns will no longer be sent in the channel."))
+
     @commands.group()
     @commands.guild_only()
     @checks.guildowner_or_permissions(administrator=True)
@@ -251,7 +302,7 @@ async def warn(
         user: discord.Member,
         points: Optional[int] = 1,
         *,
-        reason: str
+        reason: str,
     ):
         """Warn the user for the specified reason.
 
@@ -260,6 +311,8 @@ async def warn(
         `<reason>` can be a registered reason if it exists or a custom one
         is created by default.
         """
+        channel = ctx.channel
+        guild = ctx.guild
         if user == ctx.author:
             await ctx.send(_("You cannot warn yourself."))
             return
@@ -300,20 +353,37 @@ async def warn(
         await member_settings.total_points.set(current_point_count)
 
         await warning_points_add_check(self.config, ctx, user, current_point_count)
-        try:
+        dm = await self.config.guild(ctx.guild).toggle_dm()
+        if dm:
             em = discord.Embed(
                 title=_("Warning from {user}").format(user=ctx.author),
                 description=reason_type["description"],
             )
             em.add_field(name=_("Points"), value=str(reason_type["points"]))
-            await user.send(
-                _("You have received a warning in {guild_name}.").format(
-                    guild_name=ctx.guild.name
-                ),
-                embed=em,
+            with contextlib.suppress(discord.HTTPException):
+                await user.send(
+                    _("You have received a warning in {guild_name}.").format(
+                        guild_name=ctx.guild.name
+                    ),
+                    embed=em,
+                )
+
+        toggle_channel = await self.config.guild(guild).toggle_channel()
+        if toggle_channel:
+            em = discord.Embed(
+                title=_("Warning from {user}").format(user=ctx.author),
+                description=reason_type["description"],
             )
-        except discord.HTTPException:
-            pass
+            em.add_field(name=_("Points"), value=str(reason_type["points"]))
+            warn_channel = self.bot.get_channel(await self.config.guild(guild).warn_channel())
+            if warn_channel:
+                channel = warn_channel
+                await ctx.tick()
+            else:
+                channel = ctx.channel
+            await channel.send(_("{user} has been warned.").format(user=user.mention), embed=em)
+        else:
+            await ctx.tick()
         try:
             reason_msg = _(
                 "{reason}\n\nUse `{prefix}unwarn {user} {message}` to remove this warning."
@@ -338,7 +408,6 @@ async def warn(
             )
         except RuntimeError:
             pass
-        await ctx.send(_("User {user} has been warned.").format(user=user))
 
     @commands.command()
     @commands.guild_only()