Browse Source

Automatically fix ordering of optional command options (#276)

* auto fix optional command option order

* clean up indentation
pull/1923/head
CottageDwellingCat GitHub 3 years ago
parent
commit
37d9bb3ddd
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
  1. +4
    -2
      src/Discord.Net.Core/Entities/Interactions/SlashCommands/SlashCommandBuilder.cs

+ 4
- 2
src/Discord.Net.Core/Entities/Interactions/SlashCommands/SlashCommandBuilder.cs View File

@@ -100,7 +100,7 @@ namespace Discord
{ {
var options = new List<ApplicationCommandOptionProperties>(); var options = new List<ApplicationCommandOptionProperties>();


Options.ForEach(x => options.Add(x.Build()));
Options.OrderByDescending(x => x.Required ?? false).ToList().ForEach(x => options.Add(x.Build()));


props.Options = options; props.Options = options;
} }
@@ -378,7 +378,9 @@ namespace Discord
Default = Default, Default = Default,
Required = Required, Required = Required,
Type = Type, Type = Type,
Options = Options?.Count > 0 ? Options.Select(x => x.Build()).ToList() : new List<ApplicationCommandOptionProperties>(),
Options = Options?.Count > 0
? Options.OrderByDescending(x => x.Required ?? false).Select(x => x.Build()).ToList()
: new List<ApplicationCommandOptionProperties>(),
Choices = Choices, Choices = Choices,
Autocomplete = Autocomplete, Autocomplete = Autocomplete,
ChannelTypes = ChannelTypes, ChannelTypes = ChannelTypes,


Loading…
Cancel
Save