From 2e70511748381c2f8f3162a4ba1645565886c5e8 Mon Sep 17 00:00:00 2001
From: mathiascode <mail@mathias.is>
Date: Thu, 16 Apr 2020 20:05:32 +0300
Subject: [PATCH] Optimizations

---
 LICENSE                                      |  0
 README.md                                    |  0
 pom.xml                                      |  0
 src/main/java/pw/kaboom/commandspy/Main.java | 63 +++++++++-----------
 src/main/resources/plugin.yml                |  0
 suppressions.xml                             |  0
 6 files changed, 28 insertions(+), 35 deletions(-)
 mode change 100755 => 100644 LICENSE
 mode change 100755 => 100644 README.md
 mode change 100755 => 100644 pom.xml
 mode change 100755 => 100644 src/main/java/pw/kaboom/commandspy/Main.java
 mode change 100755 => 100644 src/main/resources/plugin.yml
 mode change 100755 => 100644 suppressions.xml

diff --git a/LICENSE b/LICENSE
old mode 100755
new mode 100644
diff --git a/README.md b/README.md
old mode 100755
new mode 100644
diff --git a/pom.xml b/pom.xml
old mode 100755
new mode 100644
diff --git a/src/main/java/pw/kaboom/commandspy/Main.java b/src/main/java/pw/kaboom/commandspy/Main.java
old mode 100755
new mode 100644
index d6cda09..7474aab
--- a/src/main/java/pw/kaboom/commandspy/Main.java
+++ b/src/main/java/pw/kaboom/commandspy/Main.java
@@ -30,21 +30,22 @@ public final class Main extends JavaPlugin implements CommandExecutor, Listener
 	public boolean onCommand(final CommandSender sender, final Command cmd, final String label, final String[] args) {
 		if (sender instanceof ConsoleCommandSender) {
 			sender.sendMessage("Command has to be run by a player");
-		} else {
-			final Player player = (Player) sender;
-			final JavaPlugin plugin = JavaPlugin.getPlugin(Main.class);
-
-			if (plugin.getConfig().contains(player.getUniqueId().toString())) {
-				plugin.getConfig().set(player.getUniqueId().toString(), null);
-				plugin.saveConfig();
-				player.sendMessage("Successfully disabled CommandSpy");
-			} else {
-				plugin.getConfig().set(player.getUniqueId().toString(), true);
-				plugin.saveConfig();
-				player.sendMessage("Successfully enabled CommandSpy");
-			}
-			config = plugin.getConfig();
+			return true;
 		}
+
+		final Player player = (Player) sender;
+		final JavaPlugin plugin = JavaPlugin.getPlugin(Main.class);
+
+		if (plugin.getConfig().contains(player.getUniqueId().toString())) {
+			plugin.getConfig().set(player.getUniqueId().toString(), null);
+			plugin.saveConfig();
+			player.sendMessage("Successfully disabled CommandSpy");
+		} else {
+			plugin.getConfig().set(player.getUniqueId().toString(), true);
+			plugin.saveConfig();
+			player.sendMessage("Successfully enabled CommandSpy");
+		}
+		config = plugin.getConfig();
 		return true;
 	}
 
@@ -54,16 +55,12 @@ public final class Main extends JavaPlugin implements CommandExecutor, Listener
 			UUID uuid = UUID.fromString(uuidString);
 
 			if (Bukkit.getPlayer(uuid) != null) {
-				final Player commandRunner = event.getPlayer();
-				final ChatColor color;
-
-				if (config.contains(commandRunner.getUniqueId().toString())) {
-					color = ChatColor.GREEN;
-				} else {
-					color = ChatColor.RED;
-				}
-
-				Bukkit.getPlayer(uuid).sendMessage(color + "" + commandRunner.getName() + "" + color + ": " + event.getMessage());
+				Bukkit.getPlayer(uuid).sendMessage(
+						ChatColor.YELLOW + ""
+								+ event.getPlayer().getName() + ""
+								+ ChatColor.YELLOW + ": "
+								+ event.getMessage()
+								);
 			}
 		}
 	}
@@ -74,18 +71,14 @@ public final class Main extends JavaPlugin implements CommandExecutor, Listener
 			UUID uuid = UUID.fromString(uuidString);
 
 			if (Bukkit.getPlayer(uuid) != null) {
-				final Player signPlacer = event.getPlayer();
-				final ChatColor color;
-
-				if (config.contains(signPlacer.getUniqueId().toString())) {
-					color = ChatColor.GREEN;
-				} else {
-					color = ChatColor.RED;
-				}
-
-				Bukkit.getPlayer(uuid).sendMessage(color + "" + signPlacer.getName() + "" + color + " created a sign with contents:");
+				Bukkit.getPlayer(uuid).sendMessage(
+						ChatColor.YELLOW + ""
+								+ event.getPlayer().getName() + ""
+								+ ChatColor.YELLOW
+								+ " created a sign with contents:"
+								);
 				for (String line: event.getLines()) {
-					Bukkit.getPlayer(uuid).sendMessage(color + "  " + line);
+					Bukkit.getPlayer(uuid).sendMessage(ChatColor.YELLOW + "  " + line);
 				}
 			}
 		}
diff --git a/src/main/resources/plugin.yml b/src/main/resources/plugin.yml
old mode 100755
new mode 100644
diff --git a/suppressions.xml b/suppressions.xml
old mode 100755
new mode 100644