From 1db268271d3e2e3c69eb06a2a371ad931592f65c Mon Sep 17 00:00:00 2001
From: StillLutto <still.lutto@gmail.com>
Date: Sat, 30 Nov 2024 23:41:10 +0100
Subject: [PATCH] refactor: add/improve some constructors in
 AbstractStellarCommand & StellarCommand

---
 .../main/kotlin/com/undefined/stellar/StellarCommand.kt    | 7 ++++++-
 .../kotlin/com/undefined/stellar/AbstractStellarCommand.kt | 4 +---
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/api/src/main/kotlin/com/undefined/stellar/StellarCommand.kt b/api/src/main/kotlin/com/undefined/stellar/StellarCommand.kt
index 2df0a35..478517b 100644
--- a/api/src/main/kotlin/com/undefined/stellar/StellarCommand.kt
+++ b/api/src/main/kotlin/com/undefined/stellar/StellarCommand.kt
@@ -12,7 +12,11 @@ val registrars = mapOf(
     "1.20.6" to com.undefined.stellar.v1_20_6.BrigadierCommandRegistrar
 )
 
-class StellarCommand(name: String, vararg aliases: String) : AbstractStellarCommand<StellarCommand>(name) {
+class StellarCommand(name: String, description: String = "", vararg aliases: String = arrayOf()) : AbstractStellarCommand<StellarCommand>(name, description) {
+
+    constructor(name: String, vararg aliases: String) : this(name, "", aliases = aliases)
+    constructor(name: String, aliases: List<String>) : this(name, "", aliases = aliases.toTypedArray())
+
     init {
         this.aliases.addAll(aliases)
     }
@@ -30,4 +34,5 @@ class StellarCommand(name: String, vararg aliases: String) : AbstractStellarComm
             return registrar.parseAndReturnCancelled(sender, input)
         }
     }
+
 }
\ No newline at end of file
diff --git a/common/src/main/kotlin/com/undefined/stellar/AbstractStellarCommand.kt b/common/src/main/kotlin/com/undefined/stellar/AbstractStellarCommand.kt
index 32ae9f7..d7fd6e7 100644
--- a/common/src/main/kotlin/com/undefined/stellar/AbstractStellarCommand.kt
+++ b/common/src/main/kotlin/com/undefined/stellar/AbstractStellarCommand.kt
@@ -15,13 +15,11 @@ import org.bukkit.entity.Player
 import org.bukkit.plugin.java.JavaPlugin
 
 @Suppress("UNCHECKED_CAST")
-abstract class AbstractStellarCommand<T>(val name: String) : SubCommandHandler() {
+abstract class AbstractStellarCommand<T>(val name: String, var description: String = "", var usage: String = "") : SubCommandHandler() {
 
     override fun getBase(): AbstractStellarCommand<*> = this
 
     val aliases: MutableList<String> = mutableListOf()
-    var description: String = ""
-    var usage: String = ""
     val failureMessages: MutableList<Component> = mutableListOf()
     val globalFailureMessages: MutableList<Component> = mutableListOf()
     val failureExecutions: MutableList<CustomStellarExecution<*, Any>> = mutableListOf()