version bump 1.1.0

refactor of commands.. again

build.gradle

@@ -17,7 +17,7 @@ apply plugin: 'com.github.johnrengelman.shadow'
 //Only edit below this line, the above code adds and enables the necessary things for Forge to be setup.
 
 
-version = "1.0.0"
+version = "1.1.0"
 group = "matterlink" // http://maven.apache.org/guides/mini/guide-naming-conventions.html
 archivesBaseName = "MatterLink"
 

src/main/kotlin/matterlink/MatterLink.kt

@@ -48,7 +48,10 @@ object MatterLink {
         logger.info("Building bridge!")
 
         MatterLinkConfig(event.suggestedConfigurationFile)
+    }
 
+    @Mod.EventHandler
+    fun init(event: FMLInitializationEvent) {
         BridgeCommandRegistry.registerAll(PlayerListCommand, HelpCommand)
     }
 

src/main/kotlin/matterlink/bridge/ServerChatHelper.kt

@@ -1,7 +1,6 @@
 package matterlink.bridge
 
 import matterlink.MatterLink
-import matterlink.bridge.command.BridgeCommand
 import matterlink.bridge.command.BridgeCommandRegistry
 import matterlink.cfg
 import net.minecraft.util.text.TextComponentString

src/main/kotlin/matterlink/bridge/command/BridgeCommandRegistry.kt

@@ -5,28 +5,32 @@ import matterlink.cfg
 
 object BridgeCommandRegistry {
 
-    private val commandMap: HashMap<String, BridgeCommand> = HashMap()
+    private val commandMap: HashMap<String, IBridgeCommand> = HashMap()
 
     fun handleCommand(input: String): Boolean {
         if (!cfg!!.command.enable) return false
         if (input[0] != cfg!!.command.prefix[0] || input.length < 2) return false
 
-        val cmd = input.substring(1).split(delimiters = ' ', ignoreCase = false, limit = 2)
-        val args = if (cmd.size > 1) cmd[1] else ""
+        val cmd = input.substring(1).split(' ', ignoreCase = false, limit = 2)
+        val args = if (cmd.size == 2)
+            cmd[1]
+        else
+            ""
 
-        return if (commandMap.containsKey(cmd[0])) (commandMap[cmd[0]]!!.command(args)) else false
+        return if (commandMap.containsKey(cmd[0])) (commandMap[cmd[0]]!!.call(args)) else false
     }
 
-    fun register(cmd: BridgeCommand): Boolean {
-        if (cmd.name.isBlank() || commandMap.containsKey(cmd.name)) return false
+    fun register(cmd: IBridgeCommand): Boolean {
+        if (cmd.name.isBlank() || commandMap.containsKey(cmd.name)) {
+            MatterLink.logger.error("Failed to register command: '${cmd.name}'")
+            return false
+        }
         commandMap[cmd.name] = cmd
         return true
     }
 
-    fun registerAll(vararg cmds: BridgeCommand) {
-        for (cmd in cmds) {
-            if (!register(cmd)) MatterLink.logger.error("Failed to register command: " + cmd.name)
-        }
+    fun registerAll(vararg commands: IBridgeCommand) {
+        commands.forEach { register(it) }
     }
 
     fun getHelpString(cmd: String): String {
@@ -34,10 +38,9 @@ object BridgeCommandRegistry {
 
         val help = commandMap[cmd]!!.help
 
-        return if (help.isNotBlank()) help else "No help for command " + cfg!!.command.prefix + cmd
+        return if (help.isNotBlank()) help else "No help for '$cmd'"
     }
 
-    fun listCommands(): String {
-        return commandMap.keys.joinToString(" | ")
-    }
+    val commandList: String
+        get() = commandMap.keys.joinToString(separator = ", ")
 }

src/main/kotlin/matterlink/bridge/command/HelpCommand.kt

@@ -4,15 +4,15 @@ import matterlink.bridge.ApiMessage
 import matterlink.bridge.MessageHandler
 import matterlink.cfg
 
-object HelpCommand : BridgeCommand {
+object HelpCommand : IBridgeCommand {
     override val name: String = "help"
     override val help: String = "Returns the help string for the given command. Syntax: help <command>"
-    override fun command(args: String): Boolean {
+    override fun call(args: String): Boolean {
         val msg: String = if (args.isEmpty()) {
-            "Available commands: " + BridgeCommandRegistry.listCommands()
+            "Available commands: ${ BridgeCommandRegistry.commandList}"
         } else {
-            val cmd = args.split(delimiters = *charArrayOf(' '), ignoreCase = false, limit = 2)[0]
-            cmd + ": " + BridgeCommandRegistry.getHelpString(cmd)
+            args.split(" ", ignoreCase = false)
+                    .joinToString(separator = "\n") { "$it: ${ BridgeCommandRegistry.getHelpString(it) }" }
         }
         MessageHandler.transmit(ApiMessage(
                 username = cfg!!.relay.systemUser,

src/main/kotlin/matterlink/bridge/command/IBridgeCommand.kt

@@ -1,7 +1,7 @@
 package matterlink.bridge.command
 
-interface BridgeCommand {
+interface IBridgeCommand {
     val name: String
-    fun command(args: String): Boolean
+    fun call(args: String): Boolean
     val help: String
 }

src/main/kotlin/matterlink/bridge/command/PlayerListCommand.kt

@@ -6,21 +6,15 @@ import matterlink.bridge.MessageHandler
 import matterlink.cfg
 import net.minecraftforge.fml.common.FMLCommonHandler
 
-object PlayerListCommand : BridgeCommand {
+object PlayerListCommand : IBridgeCommand {
     override val name: String = "players"
     override val help: String = "Lists online players."
-    override fun command(args: String): Boolean {
+    override fun call(args: String): Boolean {
         if (args.isNotBlank()) return false
 
-        var output = ""
-
-        for (player: String in FMLCommonHandler.instance().minecraftServerInstance.playerList.onlinePlayerNames) {
-            output = output + player.antiping() + " "
-        }
-
         MessageHandler.transmit(ApiMessage(
                 username = cfg!!.relay.systemUser,
-                text = output
+                text = FMLCommonHandler.instance().minecraftServerInstance.playerList.onlinePlayerNames.joinToString(" ") { it.antiping() }
         ))
 
         return true