Merge pull request #43 from BluevaDevelopment/dnlblasco-patch-1

Enhance join message broadcasting in PlayerJoinListener

Author: dnlblasco

src/main/java/net/blueva/core/listeners/PlayerJoinListener.java

@@ -32,6 +32,7 @@
 import net.blueva.core.configuration.ConfigManager;
 import net.blueva.core.configuration.DataManager;
 import net.blueva.core.modules.economy.EconomyModule;
+import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 import org.bukkit.event.player.PlayerJoinEvent;
@@ -63,10 +64,11 @@ public void OPJ(PlayerJoinEvent event) throws IOException {
 		}
 
 		if(ConfigManager.Modules.welcome.getBoolean("welcome.enabled")) {
+			event.setJoinMessage("");
 			if(ConfigManager.Modules.welcome.getBoolean("welcome.broadcast.join.enabled")) {
-				event.setJoinMessage(MessagesUtils.formatLegacy(event.getPlayer(), ConfigManager.Modules.welcome.getString("welcome.broadcast.join.message")).replace("%player_name%", event.getPlayer().getDisplayName()));
-			} else {
-				event.setJoinMessage("");
+				for(Player player : Bukkit.getOnlinePlayers()) {
+					main.adventure().player(player).sendMessage(MessagesUtils.format(event.getPlayer(), ConfigManager.Modules.welcome.getString("welcome.broadcast.join.message").replace("%player_name%", event.getPlayer().getDisplayName())));
+				}
 			}
 
 			if(ConfigManager.Modules.welcome.getBoolean("welcome.message.enabled")) {
@@ -99,4 +101,4 @@ public void OPJ(PlayerJoinEvent event) throws IOException {
 		}
 	}
 
-}
+}