Skip to content

Commit f8bc8f4

Browse files
committed
Change join / leave messages to also show the amount of players online.
1 parent 65126c1 commit f8bc8f4

File tree

1 file changed

+27
-2
lines changed

1 file changed

+27
-2
lines changed

src/main/java/com/cssbham/minecraftcore/MinecraftCore.java

Lines changed: 27 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import com.cssbham.minecraftcore.commands.CommandMakeGreen;
44
import com.cssbham.minecraftcore.discord.DiscordBridge;
5+
import org.bukkit.Server;
56
import org.bukkit.event.EventHandler;
67
import org.bukkit.event.Listener;
78
import org.bukkit.event.player.AsyncPlayerChatEvent;
@@ -50,11 +51,35 @@ public void onPlayerChat(AsyncPlayerChatEvent event) {
5051

5152
@EventHandler
5253
public void onPlayerJoin(PlayerJoinEvent event) {
53-
discordBridge.sendMessageToDiscord(event.getPlayer(), "__*has joined the server*__");
54+
discordBridge.sendMessageToDiscord(event.getPlayer(),
55+
"__*has joined the server, " + getOnlineMessage(event.getPlayer().getServer(), false) + "*__");
5456
}
5557

5658
@EventHandler
5759
public void onPlayerQuit(PlayerQuitEvent event) {
58-
discordBridge.sendMessageToDiscord(event.getPlayer(), "__*has left the server*__");
60+
discordBridge.sendMessageToDiscord(event.getPlayer(),
61+
"__*has left the server, " + getOnlineMessage(event.getPlayer().getServer(), true) + "*__");
62+
}
63+
64+
private String getOnlineMessage(Server server, boolean leaving) {
65+
int amount = server.getOnlinePlayers().size();
66+
if (leaving) {
67+
amount--;
68+
}
69+
// This shouldn't? happen.
70+
if (amount < 0) {
71+
amount = 0;
72+
}
73+
switch (amount) {
74+
case 0: {
75+
return "there are now no players online.";
76+
}
77+
case 1: {
78+
return "there is now 1 player online.";
79+
}
80+
default: {
81+
return "there are now " + amount + " players online.";
82+
}
83+
}
5984
}
6085
}

0 commit comments

Comments
 (0)