From ab3b30dee4c6e18114e9a61e233a641c43632c13 Mon Sep 17 00:00:00 2001
From: Mihail Gribkov <61027276+Misha-133@users.noreply.github.com>
Date: Sat, 18 Nov 2023 23:50:55 +0300
Subject: [PATCH] [Fix] Missing null checks in `RoleConnectionProperties`
(#2759)
* add null check
* more null checks
---
.../RoleConnectionProperties.cs | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/Discord.Net.Core/Entities/ApplicationRoleConnection/RoleConnectionProperties.cs b/src/Discord.Net.Core/Entities/ApplicationRoleConnection/RoleConnectionProperties.cs
index 667e5ec1..cea78829 100644
--- a/src/Discord.Net.Core/Entities/ApplicationRoleConnection/RoleConnectionProperties.cs
+++ b/src/Discord.Net.Core/Entities/ApplicationRoleConnection/RoleConnectionProperties.cs
@@ -120,13 +120,16 @@ public class RoleConnectionProperties
{
PlatformName = platformName;
PlatformUsername = platformUsername;
- Metadata = metadata.ToDictionary();
+ Metadata = metadata?.ToDictionary() ?? new ();
}
///
/// Initializes a new instance of .
///
- public RoleConnectionProperties() { }
+ public RoleConnectionProperties()
+ {
+ Metadata = new();
+ }
///
/// Initializes a new with the data from provided .
@@ -136,6 +139,6 @@ public class RoleConnectionProperties
{
PlatformName = roleConnection.PlatformName,
PlatformUsername = roleConnection.PlatformUsername,
- Metadata = roleConnection.Metadata.ToDictionary()
+ Metadata = roleConnection.Metadata?.ToDictionary()
};
}