Allow default values in parameters

.gitignore

@@ -14,4 +14,6 @@ gradle-app.setting
 # gradle/wrapper/gradle-wrapper.properties
 
 .idea/
-out/
+out/
+src/test/*
+*.log

src/main/kotlin/me/devoxin/flight/api/CommandWrapper.kt

@@ -8,22 +8,26 @@ import kotlin.coroutines.suspendCoroutine
 import kotlin.reflect.KFunction
 import kotlin.reflect.KParameter
 import kotlin.reflect.full.callSuspendBy
+import kotlin.reflect.full.instanceParameter
 
 class CommandWrapper(
-        val name: String,
-        val arguments: List<Argument>,
-        val category: String,
-        val properties: Command,
-        val async: Boolean,
-        val method: KFunction<*>,
-        val cog: Cog,
-        internal val contextParameter: KParameter
+    val name: String,
+    val arguments: List<Argument>,
+    val category: String,
+    val properties: Command,
+    val async: Boolean,
+    val method: KFunction<*>,
+    val cog: Cog,
+    private val contextParameter: KParameter
 ) {
 
     /**
      * Calls the related method with the given args.
      */
     fun execute(ctx: Context, args: HashMap<KParameter, Any?>, complete: (Boolean, Throwable?) -> Unit) {
+        method.instanceParameter?.let {
+            args[it] = cog
+        }
         args[contextParameter] = ctx
 
         try {
@@ -39,6 +43,9 @@ class CommandWrapper(
      * Calls the related method with the given args, except in an async manner.
      */
     suspend fun executeAsync(ctx: Context, args: HashMap<KParameter, Any?>, complete: (Boolean, Throwable?) -> Unit) {
+        method.instanceParameter?.let {
+            args[it] = cog
+        }
         args[contextParameter] = ctx
         //suspendCoroutine<Unit> {
             try {

src/main/kotlin/me/devoxin/flight/arguments/ArgParser.kt

@@ -105,7 +105,7 @@ class ArgParser(
             }
         }
 
-        if (!result.isPresent && !arg.optional) {
+        if (!result.isPresent && !arg.optional && !arg.isNullable) {
             throw BadArgument(arg, argument)
         }
 
@@ -134,7 +134,7 @@ class ArgParser(
             for (arg in cmd.arguments) {
                 val res = parser.parse(arg)
 
-                if (res != null || arg.valueRequired) {
+                if (res != null || (arg.isNullable && !arg.optional)) {
                     //This will only place the argument into the map if the value is null,
                     // or if the parameter requires a value (i.e. marked nullable).
                     //Commands marked optional already have a parameter so they don't need user-provided values

src/main/kotlin/me/devoxin/flight/arguments/Argument.kt

@@ -7,6 +7,6 @@ class Argument(
     val type: Class<*>,
     val greedy: Boolean,
     val optional: Boolean, // Denotes that a parameter has a default value.
-    val valueRequired: Boolean, // Denotes whether it's marked nullable, thus always requires a value.
+    val isNullable: Boolean,
     internal val kparam: KParameter
 )

src/main/kotlin/me/devoxin/flight/utils/Indexer.kt

@@ -92,9 +92,9 @@ class Indexer : Closeable {
             val type = p.type.jvmErasure.javaObjectType
             val greedy = p.hasAnnotation<Greedy>()
             val optional = p.isOptional
-            val valueRequired = p.type.isMarkedNullable
+            val isNullable = p.type.isMarkedNullable
 
-            arguments.add(Argument(pName, type, greedy, optional, valueRequired, p))
+            arguments.add(Argument(pName, type, greedy, optional, isNullable, p))
         }
 
         return CommandWrapper(name, arguments, category, properties, async, meth, cog, ctxParam)