UUID Lookup Warning

Looking up UUIDs can take a while. There's no a more immediate warning that this is happening, so players don't think the command has failed.
FerusGrim · Apr 29, 2014 · fa9d1d6 · fa9d1d6
1 parent 08b9efa
commit fa9d1d6
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 4 deletions.
diff --git a/src/main/java/io/github/ferusgrim/GrimList/Commands/AddPlayer.java b/src/main/java/io/github/ferusgrim/GrimList/Commands/AddPlayer.java
@@ -52,6 +52,11 @@ public boolean run(CommandSender sender, String name){
     }
 
     private void runOperation(CommandSender sender, String name){
+        if (sender instanceof Player) {
+            sender.sendMessage(plugin.mStart + "Looking up UUID. This can take a moment...");
+        } else {
+            plugin.log("INFO", "Looking up UUID. This can take a moment...");
+        }
         new AsyncThenSyncOperation(plugin, true){
             private Map<String, UUID> response = null;
 

diff --git a/src/main/java/io/github/ferusgrim/GrimList/Commands/DeleteRecord.java b/src/main/java/io/github/ferusgrim/GrimList/Commands/DeleteRecord.java
@@ -42,9 +42,9 @@ public boolean run(CommandSender sender, String name) {
                         }
                     }else{
                         if(sender instanceof Player){
-                            sender.sendMessage(plugin.mStart + "No record found for '" + name + "'!");
+                            sender.sendMessage(plugin.mStart + "Player record not found!");
                         }else{
-                            plugin.log("INFO", "No record found for '" + name + "'!");
+                            plugin.log("INFO", "Player record not found!");
                         }
                     }
                 }
@@ -54,6 +54,11 @@ public boolean run(CommandSender sender, String name) {
     }
 
     private void runOperation(CommandSender sender, String name) {
+        if (sender instanceof Player) {
+            sender.sendMessage(plugin.mStart + "Looking up UUID. This can take a moment...");
+        } else {
+            plugin.log("INFO", "Looking up UUID. This can take a moment...");
+        }
         new AsyncThenSyncOperation(plugin, true) {
             private Map<String, UUID> response = null;
 
@@ -91,9 +96,9 @@ protected void execSyncThen() {
                             }
                         }else{
                             if(sender instanceof Player){
-                                sender.sendMessage(plugin.mStart + "No record found for '" + name + "'!");
+                                sender.sendMessage(plugin.mStart + "Player record not found!");
                             }else{
-                                plugin.log("INFO", "No record found for '" + name + "'!");
+                                plugin.log("INFO", "Player record not found!");
                             }
                         }
                         break;

diff --git a/src/main/java/io/github/ferusgrim/GrimList/Commands/RemovePlayer.java b/src/main/java/io/github/ferusgrim/GrimList/Commands/RemovePlayer.java
@@ -52,6 +52,11 @@ public boolean run(CommandSender sender, String name) {
     }
 
     private void runOperation(CommandSender sender, String name){
+        if (sender instanceof Player) {
+            sender.sendMessage(plugin.mStart + "Looking up UUID. This can take a moment...");
+        } else {
+            plugin.log("INFO", "Looking up UUID. This can take a moment...");
+        }
         new AsyncThenSyncOperation(plugin, true){
             private Map<String, UUID> response = null;
 

diff --git a/src/main/java/io/github/ferusgrim/GrimList/Commands/ViewPlayer.java b/src/main/java/io/github/ferusgrim/GrimList/Commands/ViewPlayer.java
@@ -95,6 +95,11 @@ public void outputText(CommandSender sender, String uuid){
     }
 
     public void runOperation(CommandSender sender, String name){
+        if (sender instanceof Player) {
+            sender.sendMessage(plugin.mStart + "Looking up UUID. This can take a moment...");
+        } else {
+            plugin.log("INFO", "Looking up UUID. This can take a moment...");
+        }
         new AsyncThenSyncOperation(plugin, true){
             private Map<String, UUID> response = null;