From b71e97a5e910d74d850d772cf4e9297079924fe8 Mon Sep 17 00:00:00 2001 From: spiral Date: Mon, 6 Sep 2021 19:26:47 -0400 Subject: [PATCH] fix: actually pass group actioned/not actioned count properly to GenerateResponse I hope this works. This code is all a mess. Refactor when? --- PluralKit.Bot/Commands/Groups.cs | 2 +- PluralKit.Bot/Commands/MemberGroup.cs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/PluralKit.Bot/Commands/Groups.cs b/PluralKit.Bot/Commands/Groups.cs index b9eb60b6..b93bd55a 100644 --- a/PluralKit.Bot/Commands/Groups.cs +++ b/PluralKit.Bot/Commands/Groups.cs @@ -471,7 +471,7 @@ namespace PluralKit.Bot } else return; // otherwise toAction "may be undefined" - await ctx.Reply(GroupAddRemoveResponseService.GenerateResponse(op, members.Count, 1, members.Count - existingMembersInGroup.Count, existingMembersInGroup.Count)); + await ctx.Reply(GroupAddRemoveResponseService.GenerateResponse(op, members.Count, 1, toAction.Count, members.Count - toAction.Count)); } public async Task ListGroupMembers(Context ctx, PKGroup target) diff --git a/PluralKit.Bot/Commands/MemberGroup.cs b/PluralKit.Bot/Commands/MemberGroup.cs index e324f233..37cdc281 100644 --- a/PluralKit.Bot/Commands/MemberGroup.cs +++ b/PluralKit.Bot/Commands/MemberGroup.cs @@ -55,7 +55,7 @@ namespace PluralKit.Bot } else return; // otherwise toAction "may be unassigned" - await ctx.Reply(GroupAddRemoveResponseService.GenerateResponse(op, 1, groups.Count, groups.Count - existingGroups.Count, existingGroups.Count)); + await ctx.Reply(GroupAddRemoveResponseService.GenerateResponse(op, 1, groups.Count, toAction.Count, groups.Count - toAction.Count)); } public async Task List(Context ctx, PKMember target)