[plugin] add warning for invalid requirement variables

plugin/src/main/kotlin/me/gabytm/minecraft/arcanevouchers/voucher/requirements/implementations/common/variable/BooleanVariable.kt

@@ -1,9 +1,17 @@
 package me.gabytm.minecraft.arcanevouchers.voucher.requirements.implementations.common.variable
 
 import me.gabytm.minecraft.arcanevouchers.functions.papi
+import me.gabytm.minecraft.arcanevouchers.functions.warning
+import org.bukkit.entity.Player
 
 data class BooleanVariable(
-    private val stringValue: String
+    private val string: String
 ) : Variable<Boolean>(
-    { player -> stringValue.toBooleanStrictOrNull() ?: stringValue.papi(player).toBooleanStrictOrNull() }
-)
+    { player -> string.toBooleanStrictOrNull() ?: string.papi(player).toBooleanStrictOrNull() }
+) {
+
+    override fun warn(player: Player?, requirement: String) {
+        warning("[$requirement] '$string' (${string.papi(player)}) is not a valid boolean (accepted values: true, false)")
+    }
+
+}

plugin/src/main/kotlin/me/gabytm/minecraft/arcanevouchers/voucher/requirements/implementations/common/variable/DoubleVariable.kt

@@ -1,9 +1,17 @@
 package me.gabytm.minecraft.arcanevouchers.voucher.requirements.implementations.common.variable
 
 import me.gabytm.minecraft.arcanevouchers.functions.papi
+import me.gabytm.minecraft.arcanevouchers.functions.warning
+import org.bukkit.entity.Player
 
 data class DoubleVariable(
     private val string: String
 ) : Variable<Double>(
     { player -> string.toDoubleOrNull() ?: string.papi(player).toDoubleOrNull() }
-)
+) {
+
+    override fun warn(player: Player?, requirement: String) {
+        warning("[$requirement] '$string' (${string.papi(player)}) is not a valid Double")
+    }
+
+}

plugin/src/main/kotlin/me/gabytm/minecraft/arcanevouchers/voucher/requirements/implementations/common/variable/LocationVariable.kt

@@ -2,11 +2,13 @@ package me.gabytm.minecraft.arcanevouchers.voucher.requirements.implementations.
 
 import me.gabytm.minecraft.arcanevouchers.Constant
 import me.gabytm.minecraft.arcanevouchers.functions.papi
+import me.gabytm.minecraft.arcanevouchers.functions.warning
 import org.bukkit.Bukkit
 import org.bukkit.Location
+import org.bukkit.entity.Player
 
 data class LocationVariable(
-    private val string: String
+    val string: String
 ) : Variable<Location>(
     transformer@{ player ->
         // world;x;y;z
@@ -22,4 +24,10 @@ data class LocationVariable(
         val z = parts[3].toDoubleOrNull() ?: return@transformer null
         return@transformer Location(world, x, y, z)
     }
-)
+) {
+
+    override fun warn(player: Player?, requirement: String) {
+        warning("[$requirement] '$string' (${string.papi(player)}) is not a valid location (expected format: world;x;y;z)")
+    }
+
+}

plugin/src/main/kotlin/me/gabytm/minecraft/arcanevouchers/voucher/requirements/implementations/common/variable/Variable.kt

@@ -12,4 +12,6 @@ abstract class Variable<T>(
 
     fun get(player: Player?): T? = parsed ?: transformer(player)
 
+    abstract fun warn(player: Player?, requirement: String)
+
 }

plugin/src/main/kotlin/me/gabytm/minecraft/arcanevouchers/voucher/requirements/implementations/location/DistanceRequirement.kt

@@ -26,8 +26,14 @@ class DistanceRequirement(
             return false
         }
 
-        val distance = distanceVariable.get(player) ?: return false
-        val location = locationVariable.get(player) ?: return false
+        val distance = distanceVariable.get(player) ?: kotlin.run {
+            distanceVariable.warn(player, getName())
+            return false
+        }
+        val location = locationVariable.get(player) ?: kotlin.run {
+            locationVariable.warn(player, getName())
+            return false
+        }
 
         if (player.world != location.world) {
             return false

plugin/src/main/kotlin/me/gabytm/minecraft/arcanevouchers/voucher/requirements/implementations/number/NumberRequirement.kt

@@ -21,11 +21,13 @@ class NumberRequirement(
     name, optional, negated, failActions, actionManager
 ) {
 
-    override fun check(t: Player?, arguments: Arguments): Boolean {
-        val leftNumber = left.get(t)
-        val rightNumber = right.get(t)
-
-        if (leftNumber == null || rightNumber == null) {
+    override fun check(player: Player?, arguments: Arguments): Boolean {
+        val leftNumber = left.get(player) ?: kotlin.run {
+            left.warn(player, getName())
+            return false
+        }
+        val rightNumber = right.get(player) ?: kotlin.run {
+            right.warn(player, getName())
             return false
         }