From 6107e9502e2e707d28b4f4dbdc1561fd6f172e46 Mon Sep 17 00:00:00 2001
From: korikisulda <korikisulda@gmail.com>
Date: Sun, 7 Dec 2014 01:32:35 +0000
Subject: [PATCH] Truncate excessively long online/offline player messages in
 /f show/who. Resolves #154.

---
 .../java/com/massivecraft/factions/cmd/CmdShow.java    | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/main/java/com/massivecraft/factions/cmd/CmdShow.java b/src/main/java/com/massivecraft/factions/cmd/CmdShow.java
index e75d0afa..552bda4f 100644
--- a/src/main/java/com/massivecraft/factions/cmd/CmdShow.java
+++ b/src/main/java/com/massivecraft/factions/cmd/CmdShow.java
@@ -104,7 +104,7 @@ public class CmdShow extends FCommand {
                 }
                 firstAlly = false;
 
-                if (currentAllies.toJSONString().length() >= 32700) { // Client gets kicked at 32767, some leniency
+                if (currentAllies.toJSONString().length() > Short.MAX_VALUE) {
                     allies.add(currentAllies);
                     currentAllies = new FancyMessage();
                 }
@@ -116,7 +116,7 @@ public class CmdShow extends FCommand {
                 }
                 firstEnemy = false;
 
-                if (currentEnemies.toJSONString().length() >= 32700) { // Client gets kicked at 32767, some leniency
+                if (currentEnemies.toJSONString().length() > Short.MAX_VALUE) {
                     enemies.add(currentEnemies);
                     currentEnemies = new FancyMessage();
                 }
@@ -138,6 +138,9 @@ public class CmdShow extends FCommand {
                     online.then(", " + name).tooltip(getToolTips(p));
                 }
                 firstOnline = false;
+                if (online.toJSONString().length() > Short.MAX_VALUE) {
+                    online = new FancyMessage();
+                }
             } else {
                 if (firstOffline) {
                     offline.then(name).tooltip(getToolTips(p));
@@ -145,6 +148,9 @@ public class CmdShow extends FCommand {
                     offline.then(", " + name).tooltip(getToolTips(p));
                 }
                 firstOffline = false;
+                if (offline.toJSONString().length() > Short.MAX_VALUE) {
+                    offline = new FancyMessage();
+                }
             }
         }