Small improvements.

* NEW: the tooltip messages changed if the taret is not the sender.
* BUG: fixed Pong catchphrase not displayed if the user enters the /ping:ping command.
* NEW: the chat separators are only displayed for players.

Author: AmauryCarrade

src/main/java/fr/zcraft/Ping/Ping.java

@@ -169,14 +169,15 @@ else if (label.equalsIgnoreCase("ping"))
         final double[] tps = getServerTPS();
 
 
-        sender.sendMessage(ChatColor.GRAY + "⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅");
+        if (sender instanceof Player)
+            sender.sendMessage(ChatColor.GRAY + "⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅");
 
         if (!isSelf)
         {
-            sender.sendMessage(I.t("{green}{bold}Ping, {0}!", target.getName()));
+            sender.sendMessage(I.t("{green}{bold}{0}'s ping", target.getName()));
             sender.sendMessage("");
         }
-        else if (label.equalsIgnoreCase("ping"))
+        else if (label.toLowerCase().endsWith("ping"))
         {
             sender.sendMessage(I.t("{green}{bold}Pong!"));
             sender.sendMessage("");
@@ -186,10 +187,13 @@ else if (label.equalsIgnoreCase("ping"))
                         .then(I.t("Latency: "))
                         .color(ChatColor.GOLD)
                         .hover(
-                                new ItemStackBuilder(Material.DIAMOND)
+                                new ItemStackBuilder(Material.POTATO_ITEM)
                                         .title(ChatColor.BOLD + I.t("Latency"))
-                                        .longLore(ChatColor.RESET, I.t("The time needed to transfer data from you to the server."), 38)
+                                        .longLore(ChatColor.RESET, isSelf
+                                                ? I.t("The time needed to transfer data from you to the server.")
+                                                : I.t("The time needed to transfer data from {0} to the server.", target.getName()), 38)
                                         .loreLine(ChatColor.GREEN, I.t("The lower the better."))
+                                        .hideAttributes()
                                 .item()
                         )
 
@@ -202,20 +206,22 @@ else if (label.equalsIgnoreCase("ping"))
                         .then(I.t("Server load: "))
                         .color(ChatColor.GOLD)
                         .hover(
-                                new ItemStackBuilder(Material.DIAMOND)
+                                new ItemStackBuilder(Material.POTATO_ITEM)
                                         .title(ChatColor.BOLD + I.t("Ticks per second"))
                                         .longLore(ChatColor.RESET, I.t("The number of cycles the server executes per second. The best is 20; under 15, the server is experiencing difficulties."), 38)
                                         .longLore(ChatColor.GRAY, I.t("The three values are the average number of TPS during the last 1, 5 and 15 minutes."), 38)
                                         .loreLine(ChatColor.GREEN, I.t("The closest to 20 the better."))
-                                        .item()
+                                        .hideAttributes()
+                                .item()
                         )
 
                         .then(tps != null ? formatTPS(tps) : I.t("{gray}(unable to retrieve server load)"))
 
                         .build()
         );
 
-        sender.sendMessage(ChatColor.GRAY + "⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅");
+        if (sender instanceof Player)
+            sender.sendMessage(ChatColor.GRAY + "⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅");
 
         return true;
     }

src/main/resources/i18n/fr_FR.po

@@ -31,8 +31,8 @@ msgid "{ce}You cannot execute this command from the console."
 msgstr "{ce}Vous ne pouvez exécuter cette commande depuis la console."
 
 #: src/main/java/fr/zcraft/Ping/Ping.java:172
-msgid "{green}{bold}Ping, {0}!"
-msgstr "{green}{bold}Ping, {0} !"
+msgid "{green}{bold}{0}'s ping"
+msgstr "{green}{bold}Ping pour {0}"
 
 #: src/main/java/fr/zcraft/Ping/Ping.java:176
 msgid "{green}{bold}Pong!"
@@ -51,6 +51,11 @@ msgid "The time needed to transfer data from you to the server."
 msgstr ""
 "Le temps nécessaire pour transférer des données entre vous et le serveur."
 
+#: src/main/java/fr/zcraft/Ping/Ping.java:186
+msgid "The time needed to transfer data from {0} to the server."
+msgstr ""
+"Le temps nécessaire pour transférer des données entre {0} et le serveur."
+
 #: src/main/java/fr/zcraft/Ping/Ping.java:187
 msgid "The lower the better."
 msgstr "Le plus bas est le mieux."