update to 5.1.0

fixed blank slash options
koply · Jun 25, 2023 · 080b17c · 080b17c
1 parent a324a55
commit 080b17c
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 9 deletions.
diff --git a/internal/src/main/java/me/koply/kcommando/internal/boxes/SlashBox.java b/internal/src/main/java/me/koply/kcommando/internal/boxes/SlashBox.java
@@ -16,7 +16,8 @@ public SlashBox(Object instance, Method method, Class<?> clazz, HandleSlash info
     }
 
     public Optional<Perm> getPerm() {
-        return Optional.of(method.getAnnotation(Perm.class));
+        Perm annotation = method.getAnnotation(Perm.class);
+        return annotation == null ? Optional.empty() : Optional.of(annotation);
     }
 
 }
diff --git a/javacord-integration/pom.xml b/javacord-integration/pom.xml
@@ -13,7 +13,7 @@
     <artifactId>javacord-integration</artifactId>
 
     <properties>
-        <javacord.version>3.7.0</javacord.version>
+        <javacord.version>3.8.0</javacord.version>
     </properties>
 
     <dependencies>

diff --git a/jda-integration/pom.xml b/jda-integration/pom.xml
@@ -21,7 +21,7 @@
     </repositories>
 
     <properties>
-        <jda.version>5.0.0-alpha.21</jda.version>
+        <jda.version>5.0.0-beta.11</jda.version>
     </properties>
 
     <dependencies>

diff --git a/jda-integration/src/main/java/me/koply/kcommando/integration/impl/jda/JDAIntegration.java b/jda-integration/src/main/java/me/koply/kcommando/integration/impl/jda/JDAIntegration.java
@@ -31,6 +31,7 @@ public JDAIntegration(JDA api) {
         this.api = api;
     }
 
+    @Override
     public void registerCommandHandler(CommandHandler handler) {
         api.addEventListener(new CommandListener(handler));
     }
@@ -55,17 +56,22 @@ public void registerSlashCommand(SlashBox box) {
 
         Option[] options = info.options();
         OptionData[] optionDatas = new OptionData[options.length];
+        int filledDatas = 0;
         for (int i = 0; i < options.length; i++) {
             if (options[i].type() == me.koply.kcommando.internal.OptionType.UNKNOWN) continue;
             OptionType type = OptionType.fromKey(options[i].type().value);
             optionDatas[i] = new OptionData(type, options[i].name(), options[i].desc(), options[i].required());
+            filledDatas++;
         }
 
+        OptionData[] rolledOptionDatas = new OptionData[filledDatas];
+        System.arraycopy(optionDatas, 0, rolledOptionDatas, 0, filledDatas);
+
         boolean guildOnly = !info.enabledInDms();
 
         CommandData data = new CommandDataImpl(name, desc)
-                .addOptions(optionDatas)
-                .setGuildOnly(guildOnly);
+                .setGuildOnly(guildOnly)
+                .addOptions(rolledOptionDatas);
 
         box.getPerm().ifPresent(perm -> data.setDefaultPermissions(DefaultMemberPermissions.enabledFor(Util.getPermissions(perm.value()))));
 

diff --git a/jda-integration/src/main/test/me/koply/aaa/Test.java b/jda-integration/src/main/test/me/koply/aaa/Test.java
@@ -86,6 +86,11 @@ public static void slash(SlashCommandInteractionEvent e) {
         e.getHook().sendMessage("Bu bir cevap mı bilmiyorum. -> " + val).queue();
     }
 
+    @HandleSlash(name = "amogus", desc = "amogus", guildId = 674334330444709904L)
+    public static void nonoptions(SlashCommandInteractionEvent e) {
+        e.reply("yes man u got that").queue();
+    }
+
     @Override
     public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent event) {
         System.out.println("slash command timez " + event.getName() + " - " + event.getCommandId());

diff --git a/pom.xml b/pom.xml
@@ -10,9 +10,9 @@
     <version>5.1.0</version>
 
     <scm>
-        <connection>scm:git:git://github.com/MusaBrt/KCommando.git</connection>
-        <developerConnection>scm:git:ssh://github.com/MusaBrt/KCommando.git</developerConnection>
-        <url>https://github.com/MusaBrt/KCommando</url>
+        <connection>scm:git:git://github.com/koply/KCommando.git</connection>
+        <developerConnection>scm:git:ssh://github.com/koply/KCommando.git</developerConnection>
+        <url>https://github.com/koply/KCommando</url>
     </scm>
     <modules>
         <module>all</module>
@@ -25,7 +25,7 @@
 
     <name>KCommando</name>
     <description>Annotation based command handler wrapper for JDA &amp; Javacord.</description>
-    <url>https://github.com/MusaBrt/KCommando</url>
+    <url>https://github.com/koply/KCommando</url>
 
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>