eightball/PluralKit
1
0
Fork 0
mirror of https://github.com/PluralKit/PluralKit.git synced 2026-02-04 04:56:49 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix: do not reveal if displayname is set if name is private

Browse source
This commit is contained in:
rladenson 2025-01-04 04:11:33 -07:00
parent bba86fe98b
commit dbe7560059
1 changed files with 7 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

14
PluralKit.Bot/Commands/MemberEdit.cs
View file

@ -388,7 +388,7 @@ public class MemberEdit
eb.Field(new Embed.Field("Display name", (target.DisplayName != null && showDisplayName)
? boldIf(target.DisplayName, memberGuildConfig?.DisplayName == null)
: "*(none)*"
: "*(no displayname set or name is private)*"
));
if (ctx.Guild != null)
@ -417,18 +417,18 @@ public class MemberEdit
await ctx.Reply(successStr);
}
var noDisplayNameSetMessage = "This member does not have a display name set.";
if (ctx.System?.Id == target.System)
noDisplayNameSetMessage +=
$" To set one, type `{ctx.DefaultPrefix}member {target.Reference(ctx)} displayname <display name>`.";
var isOwner = ctx.System?.Id == target.System;
var noDisplayNameSetMessage = $"This member does not have a display name set{(isOwner ? "" : " or name is private")}."
+ (isOwner ? $" To set one, type `{ctx.DefaultPrefix}member {target.Reference(ctx)} displayname <display name>`." : "");
// No perms check, display name isn't covered by member privacy
// Whether displayname is shown or not should depend on if member name privacy is set.
// If name privacy is on then displayname should look like name.
var format = ctx.MatchFormat();
// if what's next is "raw"/"plaintext" we need to check for null
if (format != ReplyFormat.Standard)
if (target.DisplayName == null)
if (target.DisplayName == null || !target.NamePrivacy.CanAccess(ctx.DirectLookupContextFor(target.System)))
{
await ctx.Reply(noDisplayNameSetMessage);
return;