standardize style and language

- standardize code style across PR
- change some language/fix typos in documentation (code + external)

5 files changed, 16 insertions, 24 deletions

diff --git a/src/main/java/com/MylesAndMore/Tumble/commands/ForceStart.java b/src/main/java/com/MylesAndMore/Tumble/commands/ForceStart.java
index 3953bea..29583a4 100644
--- a/src/main/java/com/MylesAndMore/Tumble/commands/ForceStart.java
+++ b/src/main/java/com/MylesAndMore/Tumble/commands/ForceStart.java
@@ -29,17 +29,16 @@ public class ForceStart implements SubCommand, CommandExecutor, TabCompleter {
 
     @Override
     public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, String[] args) {
-
         Game game;
+
         if (args.length < 1 || args[0] == null) {
-            // no arena passed in, try to infer from game player is in
+            // No arena passed in, try to infer from game player is in
             game = ArenaManager.findGamePlayerIsIn((Player)sender);
             if (game == null) {
                 sender.sendMessage(LanguageManager.fromKey("missing-arena-parameter"));
                 return false;
             }
-        }
-        else {
+        } else {
             String arenaName = args[0];
             if (!ArenaManager.arenas.containsKey(arenaName)) {
                 sender.sendMessage(LanguageManager.fromKey("invalid-arena").replace("%arena%",arenaName));
diff --git a/src/main/java/com/MylesAndMore/Tumble/commands/ForceStop.java b/src/main/java/com/MylesAndMore/Tumble/commands/ForceStop.java
index 37d3465..f308979 100644
--- a/src/main/java/com/MylesAndMore/Tumble/commands/ForceStop.java
+++ b/src/main/java/com/MylesAndMore/Tumble/commands/ForceStop.java
@@ -31,14 +31,13 @@ public class ForceStop implements SubCommand, CommandExecutor, TabCompleter {
 
         Game game;
         if (args.length < 1 || args[0] == null) {
-            // no arena passed in, try to infer from game player is in
+            // No arena passed in, try to infer from game player is in
             game = ArenaManager.findGamePlayerIsIn((Player)sender);
             if (game == null) {
                 sender.sendMessage(LanguageManager.fromKey("missing-arena-parameter"));
                 return false;
             }
-        }
-        else {
+        } else {
             String arenaName = args[0];
             if (!ArenaManager.arenas.containsKey(arenaName)) {
                 sender.sendMessage(LanguageManager.fromKey("invalid-arena").replace("%arena%",arenaName));
diff --git a/src/main/java/com/MylesAndMore/Tumble/commands/Join.java b/src/main/java/com/MylesAndMore/Tumble/commands/Join.java
index 6ce33c3..3f1a8fb 100644
--- a/src/main/java/com/MylesAndMore/Tumble/commands/Join.java
+++ b/src/main/java/com/MylesAndMore/Tumble/commands/Join.java
@@ -50,8 +50,7 @@ public class Join implements SubCommand, CommandExecutor, TabCompleter {
             return false;
         }
         String arenaName = args[0];
-        if (!ArenaManager.arenas.containsKey(arenaName))
-        {
+        if (!ArenaManager.arenas.containsKey(arenaName)) {
             sender.sendMessage(LanguageManager.fromKey("invalid-arena").replace("%arena%", arenaName));
             return false;
         }
@@ -59,17 +58,16 @@ public class Join implements SubCommand, CommandExecutor, TabCompleter {
 
         Game game;
         if (args.length < 2 || args[1] == null) {
-            // no type specified: try to infer game type from game taking place in the arena
+            // No type specified: try to infer game type from game taking place in the arena
             if (arena.game == null) {
-                // cant infer if no game is taking place
+                // Can't infer if no game is taking place
                 sender.sendMessage(LanguageManager.fromKey("specify-game-type"));
                 return false;
             }
 
             game = arena.game;
-        }
-        else {
-            // type specified
+        } else {
+            // Game type specified
             GameType type;
             switch (args[1]) {
                 case "shovels", "shovel"     -> type = GameType.SHOVELS;
@@ -84,9 +82,7 @@ public class Join implements SubCommand, CommandExecutor, TabCompleter {
             if (arena.game == null) {
                 // no game is taking place in this arena, start one
                 game = arena.game = new Game(arena, type);
-            }
-            else
-            {
+            } else {
                 // a game is taking place in this arena, check that it is the right type
                 if (arena.game.type == type) {
                     game = arena.game;
@@ -100,7 +96,7 @@ public class Join implements SubCommand, CommandExecutor, TabCompleter {
             }
         }
 
-        // check to make sure the arena has a game spawn
+        // Check to make sure the arena has a game spawn
         if (game.arena.gameSpawn == null) {
             if (p.isOp()) {
                 sender.sendMessage(LanguageManager.fromKey("arena-not-ready-op"));
diff --git a/src/main/java/com/MylesAndMore/Tumble/commands/Reload.java b/src/main/java/com/MylesAndMore/Tumble/commands/Reload.java
index 79faef9..588755f 100644
--- a/src/main/java/com/MylesAndMore/Tumble/commands/Reload.java
+++ b/src/main/java/com/MylesAndMore/Tumble/commands/Reload.java
@@ -29,7 +29,6 @@ public class Reload implements SubCommand, CommandExecutor, TabCompleter {
 
     @Override
     public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, String[] args) {
-
         for (Arena a : ArenaManager.arenas.values()) {
             if (a.game != null) {
                 a.game.stopGame();
diff --git a/src/main/java/com/MylesAndMore/Tumble/commands/Tumble.java b/src/main/java/com/MylesAndMore/Tumble/commands/Tumble.java
index 8b4f925..e7d1cf4 100644
--- a/src/main/java/com/MylesAndMore/Tumble/commands/Tumble.java
+++ b/src/main/java/com/MylesAndMore/Tumble/commands/Tumble.java
@@ -44,7 +44,7 @@ public class Tumble implements CommandExecutor, TabCompleter {
             return false;
         }
 
-        // pass command action through to subCommand
+        // Pass command action through to subCommand
         subCmd.onCommand(sender, command, args[0], removeFirst(args));
         return true;
     }
@@ -52,7 +52,7 @@ public class Tumble implements CommandExecutor, TabCompleter {
     @Override
     public List<String> onTabComplete(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, String[] args) {
         if (args.length == 1) {
-            // show only subCommands the user has permission for
+            // Show only subCommands the user has permission for
             ArrayList<String> PermittedSubCmds = new ArrayList<>();
             for (SubCommand subCmd: subCommands.values()) {
                 if (sender.hasPermission(subCmd.getPermission())) {
@@ -67,11 +67,10 @@ public class Tumble implements CommandExecutor, TabCompleter {
                 return Collections.emptyList();
             }
 
-            // pass tab complete through to subCommand
+            // Pass tab complete through to subCommand
             if (subCommands.get(args[0]) instanceof TabCompleter tcmp) {
                 return tcmp.onTabComplete(sender, command, args[0], removeFirst(args));
-            }
-            else {
+            } else {
                 return null;
             }
         }