github/Discord.Net
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[Fix] NRE on thread member creation without a guild user in the payload (#3030)

Browse Source 
lets hope for the best
This commit is contained in:
Mihail Gribkov
2024-11-23 20:31:10 +03:00
committed by GitHub
parent a1e50c4bbf
commit 96a832788c
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/Discord.Net.WebSocket/Entities/Channels/SocketThreadChannel.cs
View File

@@ -178,14 +178,14 @@ namespace Discord.WebSocket
member.Update(model);
else
{
member = SocketThreadUser.Create(Guild, this, model, guildMember);
member.GlobalUser.AddRef();
member = SocketThreadUser.Create(Guild, this, model, guildMember ?? Guild.GetUser(model.UserId.GetValueOrDefault(0)));
member.GlobalUser?.AddRef();
_members[member.Id] = member;
}
return member;
}
/// <inheritdoc />
/// <inheritdoc cref="SocketGuildChannel.GetUser"/>
public new SocketThreadUser GetUser(ulong id)
{
var user = Users.FirstOrDefault(x => x.Id == id);

4
src/Discord.Net.WebSocket/Entities/Users/SocketThreadUser.cs
View File

@@ -251,9 +251,9 @@ namespace Discord.WebSocket
public override string GetDisplayAvatarUrl(ImageFormat format = ImageFormat.Auto, ushort size = 128)
=> GuildUser.GetGuildAvatarUrl() ?? base.GetDisplayAvatarUrl(format, size);
internal override SocketGlobalUser GlobalUser { get => GuildUser.GlobalUser; set => GuildUser.GlobalUser = value; }
internal override SocketGlobalUser GlobalUser { get => GuildUser?.GlobalUser; set => GuildUser.GlobalUser = value; }
internal override SocketPresence Presence { get => GuildUser.Presence; set => GuildUser.Presence = value; }
internal override SocketPresence Presence { get => GuildUser?.Presence; set => GuildUser.Presence = value; }
/// <summary>
/// Gets the guild user of this thread user.