From 03cc3379f13f468c171f312d8db228f0ab279e6a Mon Sep 17 00:00:00 2001
From: caneleex <git@mlnr.dev>
Date: Wed, 20 Apr 2022 22:50:51 +0200
Subject: [PATCH] prevent creating OptionData with OptionType#UNKNOWN (#2101)

---
 .../jda/api/interactions/commands/build/OptionData.java       | 4 ++++
 .../jda/api/interactions/commands/build/SlashCommandData.java | 3 +++
 2 files changed, 7 insertions(+)

diff --git a/src/main/java/net/dv8tion/jda/api/interactions/commands/build/OptionData.java b/src/main/java/net/dv8tion/jda/api/interactions/commands/build/OptionData.java
index 48118da03f..345248f780 100644
--- a/src/main/java/net/dv8tion/jda/api/interactions/commands/build/OptionData.java
+++ b/src/main/java/net/dv8tion/jda/api/interactions/commands/build/OptionData.java
@@ -95,6 +95,7 @@ public class OptionData implements SerializableData
      *         If any of the following checks fail
      *         <ul>
      *             <li>{@code type} is not null</li>
+     *             <li>{@code type} is not {@link OptionType#UNKNOWN UNKNOWN}</li>
      *             <li>{@code name} is alphanumeric (with dash), lowercase and between 1 and {@value #MAX_NAME_LENGTH} characters long</li>
      *             <li>{@code description} is between 1 and {@value #MAX_DESCRIPTION_LENGTH} characters long</li>
      *         </ul>
@@ -121,6 +122,7 @@ public OptionData(@Nonnull OptionType type, @Nonnull String name, @Nonnull Strin
      *         If any of the following checks fail
      *         <ul>
      *             <li>{@code type} is not null</li>
+     *             <li>{@code type} is not {@link OptionType#UNKNOWN UNKNOWN}</li>
      *             <li>{@code name} is alphanumeric (with dash), lowercase and between 1 and {@value #MAX_NAME_LENGTH} characters long</li>
      *             <li>{@code description} is between 1 and {@value #MAX_DESCRIPTION_LENGTH} characters long</li>
      *         </ul>
@@ -149,6 +151,7 @@ public OptionData(@Nonnull OptionType type, @Nonnull String name, @Nonnull Strin
      *         If any of the following checks fail
      *         <ul>
      *             <li>{@code type} is not null</li>
+     *             <li>{@code type} is not {@link OptionType#UNKNOWN UNKNOWN}</li>
      *             <li>{@code name} is alphanumeric (with dash), lowercase and between 1 and {@value #MAX_NAME_LENGTH} characters long</li>
      *             <li>{@code description} is between 1 and {@value #MAX_DESCRIPTION_LENGTH} characters long</li>
      *             <li>{@link OptionType#canSupportChoices()} is false then {@code isAutoComplete} is also false</li>
@@ -157,6 +160,7 @@ public OptionData(@Nonnull OptionType type, @Nonnull String name, @Nonnull Strin
     public OptionData(@Nonnull OptionType type, @Nonnull String name, @Nonnull String description, boolean isRequired, boolean isAutoComplete)
     {
         Checks.notNull(type, "Type");
+        Checks.check(type != OptionType.UNKNOWN, "Cannot make option of unknown type!");
         this.type = type;
 
         setName(name);
diff --git a/src/main/java/net/dv8tion/jda/api/interactions/commands/build/SlashCommandData.java b/src/main/java/net/dv8tion/jda/api/interactions/commands/build/SlashCommandData.java
index df17515324..0974d84461 100644
--- a/src/main/java/net/dv8tion/jda/api/interactions/commands/build/SlashCommandData.java
+++ b/src/main/java/net/dv8tion/jda/api/interactions/commands/build/SlashCommandData.java
@@ -161,6 +161,7 @@ default SlashCommandData addOptions(@Nonnull Collection<? extends OptionData> op
      * @throws IllegalArgumentException
      *         <ul>
      *             <li>If you try to mix subcommands/options/groups in one command.</li>
+     *             <li>If the option type is {@link OptionType#UNKNOWN UNKNOWN}.</li>
      *             <li>If the option type is {@link OptionType#SUB_COMMAND} or {@link OptionType#SUB_COMMAND_GROUP}.</li>
      *             <li>If the provided option type does not support auto-complete</li>
      *             <li>If this option is required and you already added a non-required option.</li>
@@ -196,6 +197,7 @@ default SlashCommandData addOption(@Nonnull OptionType type, @Nonnull String nam
      * @throws IllegalArgumentException
      *         <ul>
      *             <li>If you try to mix subcommands/options/groups in one command.</li>
+     *             <li>If the option type is {@link OptionType#UNKNOWN UNKNOWN}.</li>
      *             <li>If the option type is {@link OptionType#SUB_COMMAND} or {@link OptionType#SUB_COMMAND_GROUP}.</li>
      *             <li>If this option is required and you already added a non-required option.</li>
      *             <li>If more than 25 options are provided.</li>
@@ -227,6 +229,7 @@ default SlashCommandData addOption(@Nonnull OptionType type, @Nonnull String nam
      * @throws IllegalArgumentException
      *         <ul>
      *             <li>If you try to mix subcommands/options/groups in one command.</li>
+     *             <li>If the option type is {@link OptionType#UNKNOWN UNKNOWN}.</li>
      *             <li>If the option type is {@link OptionType#SUB_COMMAND} or {@link OptionType#SUB_COMMAND_GROUP}.</li>
      *             <li>If this option is required and you already added a non-required option.</li>
      *             <li>If more than 25 options are provided.</li>