From 594bcc5b7b5d56f6a3a715ccbab0ed5acc3703d3 Mon Sep 17 00:00:00 2001 From: Ske Date: Wed, 5 Feb 2020 23:44:03 +0100 Subject: [PATCH] Replace error with warning on legacy proxy command --- PluralKit.Bot/Commands/MemberProxy.cs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/PluralKit.Bot/Commands/MemberProxy.cs b/PluralKit.Bot/Commands/MemberProxy.cs index f2f19eef..1ce8a2e3 100644 --- a/PluralKit.Bot/Commands/MemberProxy.cs +++ b/PluralKit.Bot/Commands/MemberProxy.cs @@ -106,10 +106,14 @@ namespace PluralKit.Bot.Commands var requestedTag = ParseProxyTags(ctx.RemainderOrNull()); if (requestedTag.IsEmpty) throw Errors.EmptyProxyTags(target); - // This is mostly a legacy command, so it's gonna error out if there's + // This is mostly a legacy command, so it's gonna warn if there's // already more than one proxy tag. if (target.ProxyTags.Count > 1) - throw Errors.LegacyAlreadyHasProxyTag(requestedTag, target); + { + var msg = await ctx.Reply($"This member already has more than one proxy tag set: {target.ProxyTagsString().SanitizeMentions()}\nDo you want to replace them?"); + if (!await ctx.PromptYesNo(msg)) + throw Errors.GenericCancelled(); + } if (!await WarnOnConflict(requestedTag)) throw Errors.GenericCancelled();