From 7fcd3e10811d2651b0c269518f45ab1b6d789592 Mon Sep 17 00:00:00 2001 From: Oliver Dunk Date: Sat, 12 Mar 2016 17:19:35 +0000 Subject: [PATCH] Remove Java 8 Optionals (woops\!) --- .../oliverdunk/jukeboxapi/commands/JukeboxCommand.java | 5 ++--- .../com/oliverdunk/jukeboxapi/utils/LangUtils.java | 1 - .../com/oliverdunk/jukeboxapi/utils/MessageUtils.java | 10 ++++------ 3 files changed, 6 insertions(+), 10 deletions(-) diff --git a/src/main/java/com/oliverdunk/jukeboxapi/commands/JukeboxCommand.java b/src/main/java/com/oliverdunk/jukeboxapi/commands/JukeboxCommand.java index 92f643f..e5e5e1f 100644 --- a/src/main/java/com/oliverdunk/jukeboxapi/commands/JukeboxCommand.java +++ b/src/main/java/com/oliverdunk/jukeboxapi/commands/JukeboxCommand.java @@ -18,7 +18,6 @@ import org.bukkit.entity.Player; import java.util.HashMap; -import java.util.Optional; @AllArgsConstructor public class JukeboxCommand implements CommandExecutor { @@ -93,7 +92,7 @@ private boolean play(CommandSender sender, String[] args){ if(playFor == null){ HashMap findAndReplace = new HashMap(); findAndReplace.put("user", args[1]); - MessageUtils.sendMessage(sender, "command.notOnline", Optional.of(findAndReplace)); + MessageUtils.sendMessage(sender, "command.notOnline", findAndReplace); return true; } if(args[0].equalsIgnoreCase("music")) JukeboxAPI.play(playFor, args[2], ResourceType.MUSIC); @@ -106,7 +105,7 @@ private boolean stop(CommandSender sender, String[] args){ if(playFor == null){ HashMap findAndReplace = new HashMap(); findAndReplace.put("user", args[1]); - MessageUtils.sendMessage(sender, "command.notOnline", Optional.of(findAndReplace)); + MessageUtils.sendMessage(sender, "command.notOnline", findAndReplace); return true; } JukeboxAPI.stopMusic(playFor); diff --git a/src/main/java/com/oliverdunk/jukeboxapi/utils/LangUtils.java b/src/main/java/com/oliverdunk/jukeboxapi/utils/LangUtils.java index e6062cd..5dbf89a 100644 --- a/src/main/java/com/oliverdunk/jukeboxapi/utils/LangUtils.java +++ b/src/main/java/com/oliverdunk/jukeboxapi/utils/LangUtils.java @@ -5,7 +5,6 @@ import org.json.JSONObject; import java.io.*; -import java.util.Optional; public class LangUtils { diff --git a/src/main/java/com/oliverdunk/jukeboxapi/utils/MessageUtils.java b/src/main/java/com/oliverdunk/jukeboxapi/utils/MessageUtils.java index 78b96f1..6d4e3f7 100644 --- a/src/main/java/com/oliverdunk/jukeboxapi/utils/MessageUtils.java +++ b/src/main/java/com/oliverdunk/jukeboxapi/utils/MessageUtils.java @@ -5,7 +5,6 @@ import org.bukkit.command.CommandSender; import java.util.HashMap; -import java.util.Optional; public class MessageUtils { @@ -18,7 +17,7 @@ public class MessageUtils { * @param key Key which should be used to lookup the message */ public static void sendMessage(CommandSender player, String key){ - sendMessage(player, key, Optional.>empty()); + sendMessage(player, key, null); } /** @@ -28,14 +27,13 @@ public static void sendMessage(CommandSender player, String key){ * @param key Key which should be used to lookup the message * @param findAndReplace Optional list of keys which should be replaced with the corresponding values */ - public static void sendMessage(CommandSender player, String key, Optional> findAndReplace){ + public static void sendMessage(CommandSender player, String key, HashMap findAndReplace){ String message = langUtils.get(key); //Replace any values in the find and replace HashMap, if it is present message = ChatColor.translateAlternateColorCodes('&', message); - if (findAndReplace.isPresent()) { - HashMap findReplace = findAndReplace.get(); - for (String find : findReplace.keySet()) message = message.replace("[" + find + "]", findReplace.get(find)); + if (findAndReplace != null) { + for (String find : findAndReplace.keySet()) message = message.replace("[" + find + "]", findAndReplace.get(find)); } player.sendMessage(message);