From 43bc220a3daacb64865637304051ed5da37e5b44 Mon Sep 17 00:00:00 2001 From: rladenson Date: Mon, 10 Feb 2025 19:46:52 -0700 Subject: [PATCH] fix: check group banner existence when querying group banner --- PluralKit.Bot/Commands/Groups.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PluralKit.Bot/Commands/Groups.cs b/PluralKit.Bot/Commands/Groups.cs index fdf0eee1..c9acc464 100644 --- a/PluralKit.Bot/Commands/Groups.cs +++ b/PluralKit.Bot/Commands/Groups.cs @@ -354,8 +354,8 @@ public class Groups { async Task ClearBannerImage() { - await ctx.ConfirmClear("this group's banner image"); ctx.CheckOwnGroup(target); + await ctx.ConfirmClear("this group's banner image"); await ctx.Repository.UpdateGroup(target.Id, new GroupPatch { BannerImage = null }); await ctx.Reply($"{Emojis.Success} Group banner image cleared."); @@ -391,7 +391,7 @@ public class Groups { ctx.CheckSystemPrivacy(target.System, target.BannerPrivacy); - if ((target.Icon?.Trim() ?? "").Length > 0) + if ((target.BannerImage?.Trim() ?? "").Length > 0) switch (ctx.MatchFormat()) { case ReplyFormat.Raw: