Fix range of issues presented by 3rd party analyzer

experiment/Discord.Net.BuildOverrides/BuildOverrides.cs

@@ -251,7 +251,7 @@ internal static void HandleLog(Override ovr, string msg)
         private static Assembly _overrideDomain_Resolving(AssemblyLoadContext arg1, AssemblyName arg2)
         {
             // resolve the override id
-            var v = _loadedOverrides.FirstOrDefault(x => x.Value.Any(x => x.Assembly.FullName == arg1.Assemblies.FirstOrDefault().FullName));
+            var v = _loadedOverrides.FirstOrDefault(x => x.Value.Any(x => x.Assembly.FullName == arg1.Assemblies.First().FullName));
 
             return GetDependencyAsync(v.Key.Id, $"{arg2}").GetAwaiter().GetResult();
         }

src/Discord.Net.Core/Entities/Guilds/GuildFeature.cs

@@ -12,174 +12,174 @@ public enum GuildFeature : long
         /// <summary>
         ///     The guild has no features.
         /// </summary>
-        None = 0,
+        None = 0L,
         /// <summary>
         ///     The guild has access to animated banners.
         /// </summary>
-        AnimatedBanner = 1 << 0,
+        AnimatedBanner = 1L << 0,
         /// <summary>
         ///     The guild has access to set an animated guild icon.
         /// </summary>
-        AnimatedIcon = 1 << 1,
+        AnimatedIcon = 1L << 1,
         /// <summary>
         ///     The guild has access to set a guild banner image.
         /// </summary>
-        Banner = 1 << 2,
+        Banner = 1L << 2,
         /// <summary>
         ///     The guild has access to channel banners.
         /// </summary>
-        ChannelBanner = 1 << 3,
+        ChannelBanner = 1L << 3,
         /// <summary>
         ///     The guild has access to use commerce features (i.e. create store channels).
         /// </summary>
-        Commerce = 1 << 4,
+        Commerce = 1L << 4,
         /// <summary>
         ///     The guild can enable welcome screen, Membership Screening, stage channels and discovery, and receives community updates.
         /// </summary>
-        Community = 1 << 5,
+        Community = 1L << 5,
         /// <summary>
         ///     The guild is able to be discovered in the directory.
         /// </summary>
-        Discoverable = 1 << 6,
+        Discoverable = 1L << 6,
         /// <summary>
         ///     The guild has discoverable disabled.
         /// </summary>
-        DiscoverableDisabled = 1 << 7,
+        DiscoverableDisabled = 1L << 7,
         /// <summary>
         ///     The guild has enabled discoverable before.
         /// </summary>
-        EnabledDiscoverableBefore = 1 << 8,
+        EnabledDiscoverableBefore = 1L << 8,
         /// <summary>
         ///     The guild is able to be featured in the directory.
         /// </summary>
-        Featureable = 1 << 9,
+        Featureable = 1L << 9,
         /// <summary>
         ///     The guild has a force relay.
         /// </summary>
-        ForceRelay = 1 << 10,
+        ForceRelay = 1L << 10,
         /// <summary>
         ///     The guild has a directory entry.
         /// </summary>
-        HasDirectoryEntry = 1 << 11,
+        HasDirectoryEntry = 1L << 11,
         /// <summary>
         ///     The guild is a hub.
         /// </summary>
-        Hub = 1 << 12,
+        Hub = 1L << 12,
         /// <summary>
         ///     You shouldn't be here...
         /// </summary>
-        InternalEmployeeOnly = 1 << 13,
+        InternalEmployeeOnly = 1L << 13,
         /// <summary>
         ///     The guild has access to set an invite splash background.
         /// </summary>
-        InviteSplash = 1 << 14,
+        InviteSplash = 1L << 14,
         /// <summary>
         ///     The guild is linked to a hub.
         /// </summary>
-        LinkedToHub = 1 << 15,
+        LinkedToHub = 1L << 15,
         /// <summary>
         ///     The guild has member profiles.
         /// </summary>
-        MemberProfiles = 1 << 16,
+        MemberProfiles = 1L << 16,
         /// <summary>
         ///     The guild has enabled <seealso href="https://discord.com/developers/docs/resources/guild#membership-screening-object">Membership Screening</seealso>.
         /// </summary>
-        MemberVerificationGateEnabled = 1 << 17,
+        MemberVerificationGateEnabled = 1L << 17,
         /// <summary>
         ///     The guild has enabled monetization.
         /// </summary>
-        MonetizationEnabled = 1 << 18,
+        MonetizationEnabled = 1L << 18,
         /// <summary>
         ///     The guild has more emojis.
         /// </summary>
-        MoreEmoji = 1 << 19,
+        MoreEmoji = 1L << 19,
         /// <summary>
         ///     The guild has increased custom sticker slots.
         /// </summary>
-        MoreStickers = 1 << 20,
+        MoreStickers = 1L << 20,
         /// <summary>
         ///     The guild has access to create news channels.
         /// </summary>
-        News = 1 << 21,
+        News = 1L << 21,
         /// <summary>
         ///     The guild has new thread permissions.
         /// </summary>
-        NewThreadPermissions = 1 << 22,
+        NewThreadPermissions = 1L << 22,
         /// <summary>
         ///     The guild is partnered.
         /// </summary>
-        Partnered = 1 << 23,
+        Partnered = 1L << 23,
         /// <summary>
         ///     The guild has a premium tier three override; guilds made by Discord usually have this.
         /// </summary>
-        PremiumTier3Override = 1 << 24,
+        PremiumTier3Override = 1L << 24,
         /// <summary>
         ///     The guild can be previewed before joining via Membership Screening or the directory.
         /// </summary>
-        PreviewEnabled = 1 << 25,
+        PreviewEnabled = 1L << 25,
         /// <summary>
         ///     The guild has access to create private threads.
         /// </summary>
-        PrivateThreads = 1 << 26,
+        PrivateThreads = 1L << 26,
         /// <summary>
         ///     The guild has relay enabled.
         /// </summary>
-        RelayEnabled = 1 << 27,
+        RelayEnabled = 1L << 27,
         /// <summary>
         ///     The guild is able to set role icons.
         /// </summary>
-        RoleIcons = 1 << 28,
+        RoleIcons = 1L << 28,
         /// <summary>
         ///     The guild has role subscriptions available for purchase.
         /// </summary>
-        RoleSubscriptionsAvailableForPurchase = 1 << 29,
+        RoleSubscriptionsAvailableForPurchase = 1L << 29,
         /// <summary>
         ///     The guild has role subscriptions enabled.
         /// </summary>
-        RoleSubscriptionsEnabled = 1 << 30,
+        RoleSubscriptionsEnabled = 1L << 30,
         /// <summary>
         ///     The guild has access to the seven day archive time for threads.
         /// </summary>
-        SevenDayThreadArchive = 1 << 31,
+        SevenDayThreadArchive = 1L << 31,
         /// <summary>
         ///     The guild has text in voice enabled.
         /// </summary>
-        TextInVoiceEnabled = 1 << 32,
+        TextInVoiceEnabled = 1L << 32,
         /// <summary>
         ///     The guild has threads enabled.
         /// </summary>
-        ThreadsEnabled = 1 << 33,
+        ThreadsEnabled = 1L << 33,
         /// <summary>
         ///     The guild has testing threads enabled.
         /// </summary>
-        ThreadsEnabledTesting = 1 << 34,
+        ThreadsEnabledTesting = 1L << 34,
         /// <summary>
         ///     The guild has the default thread auto archive.
         /// </summary>
-        ThreadsDefaultAutoArchiveDuration = 1 << 35,
+        ThreadsDefaultAutoArchiveDuration = 1L << 35,
         /// <summary>
         ///     The guild has access to the three day archive time for threads.
         /// </summary>
-        ThreeDayThreadArchive = 1 << 36,
+        ThreeDayThreadArchive = 1L << 36,
         /// <summary>
         ///     The guild has enabled ticketed events.
         /// </summary>
-        TicketedEventsEnabled = 1 << 37,
+        TicketedEventsEnabled = 1L << 37,
         /// <summary>
         ///     The guild has access to set a vanity URL.
         /// </summary>
-        VanityUrl = 1 << 38,
+        VanityUrl = 1L << 38,
         /// <summary>
         ///     The guild is verified.
         /// </summary>
-        Verified = 1 << 39,
+        Verified = 1L << 39,
         /// <summary>
         ///     The guild has access to set 384kbps bitrate in voice (previously VIP voice servers).
         /// </summary>
-        VIPRegions = 1 << 40,
+        VIPRegions = 1L << 40,
         /// <summary>
         ///     The guild has enabled the welcome screen.
         /// </summary>
-        WelcomeScreenEnabled = 1 << 41,
+        WelcomeScreenEnabled = 1L << 41,
     }
 }

src/Discord.Net.Interactions/TypeConverters/SlashCommands/NullableConverter.cs

@@ -9,10 +9,11 @@ internal class NullableConverter<T> : TypeConverter<T>
 
         public NullableConverter(InteractionService interactionService, IServiceProvider services)
         {
-            var type = Nullable.GetUnderlyingType(typeof(T));
+            var nullableType = typeof(T);
+            var type = Nullable.GetUnderlyingType(nullableType);
 
             if (type is null)
-                throw new ArgumentException($"No type {nameof(TypeConverter)} is defined for this {type.FullName}", "type");
+                throw new ArgumentException($"No type {nameof(TypeConverter)} is defined for this {nullableType.FullName}", nameof(type));
 
             _typeConverter = interactionService.GetTypeConverter(type, services);
         }

src/Discord.Net.Rest/DiscordRestApiClient.cs

@@ -1756,7 +1756,7 @@ public async Task<InviteMetadata> CreateChannelInviteAsync(ulong channelId, Crea
                 if (args.TargetType.Value == TargetUserType.Stream)
                     Preconditions.GreaterThan(args.TargetUserId, 0, nameof(args.TargetUserId));
                 if (args.TargetType.Value == TargetUserType.EmbeddedApplication)
-                    Preconditions.GreaterThan(args.TargetApplicationId, 0, nameof(args.TargetUserId));
+                    Preconditions.GreaterThan(args.TargetApplicationId, 0, nameof(args.TargetApplicationId));
             }
             options = RequestOptions.CreateOrClone(options);
 

src/Discord.Net.Rest/Entities/AuditLogs/DataTypes/ThreadUpdateAuditLogData.cs

@@ -15,7 +15,7 @@ private ThreadUpdateAuditLogData(IThreadChannel thread, ThreadType type, ThreadI
             Thread = thread;
             ThreadType = type;
             Before = before;
-            After = After;
+            After = after;
         }
 
         internal static ThreadUpdateAuditLogData Create(BaseDiscordClient discord, Model log, EntryModel entry)

src/Discord.Net.Rest/Entities/Guilds/GuildHelper.cs

@@ -428,7 +428,7 @@ public static async Task<RestGuildUser> AddGuildUserAsync(IGuild guild, BaseDisc
                 var ids = args.Roles.Value.Select(r => r.Id);
 
                 if (args.RoleIds.IsSpecified)
-                    args.RoleIds.Value.Concat(ids);
+                    args.RoleIds = Optional.Create(args.RoleIds.Value.Concat(ids));
                 else
                     args.RoleIds = Optional.Create(ids);
             }

src/Discord.Net.Rest/Entities/Interactions/RestInteraction.cs

@@ -426,7 +426,7 @@ async Task<IUserMessage> IDiscordInteraction.FollowupWithFileAsync(Stream fileSt
             AllowedMentions allowedMentions, MessageComponent components, Embed embed, RequestOptions options)
             => await FollowupWithFileAsync(fileStream, fileName, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options).ConfigureAwait(false);
         /// <inheritdoc/>
-        async Task<IUserMessage> IDiscordInteraction.FollowupWithFileAsync(string filePath, string text, string fileName, Embed[] embeds, bool isTTS, bool ephemeral,
+        async Task<IUserMessage> IDiscordInteraction.FollowupWithFileAsync(string filePath, string fileName, string text, Embed[] embeds, bool isTTS, bool ephemeral,
             AllowedMentions allowedMentions, MessageComponent components, Embed embed, RequestOptions options)
             => await FollowupWithFileAsync(filePath, text, fileName, embeds, isTTS, ephemeral, allowedMentions, components, embed, options).ConfigureAwait(false);
         /// <inheritdoc/>

src/Discord.Net.WebSocket/Entities/Guilds/SocketGuild.cs

@@ -532,13 +532,10 @@ internal void Update(ClientState state, Model model)
             Features = model.Features;
 
             var roles = new ConcurrentDictionary<ulong, SocketRole>(ConcurrentHashSet.DefaultConcurrencyLevel, (int)(model.Roles.Length * 1.05));
-            if (model.Roles != null)
+            for (int i = 0; i < model.Roles.Length; i++)
             {
-                for (int i = 0; i < model.Roles.Length; i++)
-                {
-                    var role = SocketRole.Create(this, state, model.Roles[i]);
-                    roles.TryAdd(role.Id, role);
-                }
+                var role = SocketRole.Create(this, state, model.Roles[i]);
+                roles.TryAdd(role.Id, role);
             }
             _roles = roles;
 

src/Discord.Net.WebSocket/Entities/Interaction/SocketBaseCommand/SocketResolvableData.cs

@@ -39,7 +39,7 @@ internal SocketResolvableData(DiscordSocketClient discord, ulong? guildId, T mod
             {
                 foreach (var channel in resolved.Channels.Value)
                 {
-                    SocketChannel socketChannel = guild != null
+                    var socketChannel = guild != null
                         ? guild.GetChannel(channel.Value.Id)
                         : discord.GetChannel(channel.Value.Id);
 
@@ -69,7 +69,7 @@ internal SocketResolvableData(DiscordSocketClient discord, ulong? guildId, T mod
                 }
             }
 
-            if (resolved.Roles.IsSpecified)
+            if (resolved.Roles.IsSpecified && guild != null)
             {
                 foreach (var role in resolved.Roles.Value)
                 {