From 4d1cdaa7f0afd0d674bb1c9cce9623bea9eaa078 Mon Sep 17 00:00:00 2001 From: ementalo Date: Mon, 8 Oct 2012 22:21:03 +0100 Subject: Adding join / quit messages in the config test #1697 PlayerChatEvent -> AsyncPlayerChatEvent --- EssentialsChat/src/net/ess3/chat/EssentialsChat.java | 5 +++-- .../src/net/ess3/chat/EssentialsChatPlayer.java | 20 ++++++++++---------- .../src/net/ess3/chat/EssentialsLocalChatEvent.java | 8 ++++---- .../EssentialsChatPlayerListenerHighest.java | 6 +++--- .../EssentialsChatPlayerListenerLowest.java | 11 ++++++++--- .../EssentialsChatPlayerListenerNormal.java | 6 +++--- 6 files changed, 31 insertions(+), 25 deletions(-) (limited to 'EssentialsChat/src/net') diff --git a/EssentialsChat/src/net/ess3/chat/EssentialsChat.java b/EssentialsChat/src/net/ess3/chat/EssentialsChat.java index f191e345e..b80d38f6d 100644 --- a/EssentialsChat/src/net/ess3/chat/EssentialsChat.java +++ b/EssentialsChat/src/net/ess3/chat/EssentialsChat.java @@ -1,5 +1,6 @@ package net.ess3.chat; +import java.util.Collections; import java.util.HashMap; import java.util.Map; import java.util.logging.Level; @@ -10,7 +11,7 @@ import net.ess3.api.IPlugin; import net.ess3.chat.listenerlevel.EssentialsChatPlayerListenerHighest; import net.ess3.chat.listenerlevel.EssentialsChatPlayerListenerLowest; import net.ess3.chat.listenerlevel.EssentialsChatPlayerListenerNormal; -import org.bukkit.event.player.PlayerChatEvent; +import org.bukkit.event.player.AsyncPlayerChatEvent; import org.bukkit.plugin.PluginManager; import org.bukkit.plugin.java.JavaPlugin; @@ -35,7 +36,7 @@ public class EssentialsChat extends JavaPlugin return; } - final Map chatStore = new HashMap(); + final Map chatStore = Collections.synchronizedMap(new HashMap()); final EssentialsChatPlayerListenerLowest playerListenerLowest = new EssentialsChatPlayerListenerLowest(getServer(), ess, chatStore); final EssentialsChatPlayerListenerNormal playerListenerNormal = new EssentialsChatPlayerListenerNormal(getServer(), ess, chatStore); diff --git a/EssentialsChat/src/net/ess3/chat/EssentialsChatPlayer.java b/EssentialsChat/src/net/ess3/chat/EssentialsChatPlayer.java index 20019b179..3d1be4521 100644 --- a/EssentialsChat/src/net/ess3/chat/EssentialsChatPlayer.java +++ b/EssentialsChat/src/net/ess3/chat/EssentialsChatPlayer.java @@ -16,7 +16,7 @@ import org.bukkit.Server; import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; import org.bukkit.event.Listener; -import org.bukkit.event.player.PlayerChatEvent; +import org.bukkit.event.player.AsyncPlayerChatEvent; //TODO: Translate the local/spy tags public abstract class EssentialsChatPlayer implements Listener @@ -24,37 +24,37 @@ public abstract class EssentialsChatPlayer implements Listener protected transient IEssentials ess; protected final static Logger LOGGER = Logger.getLogger("Minecraft"); protected final transient Server server; - protected final transient Map chatStorage; + protected final transient Map chatStorage; public EssentialsChatPlayer(final Server server, final IEssentials ess, - final Map chatStorage) + final Map chatStorage) { this.ess = ess; this.server = server; this.chatStorage = chatStorage; } - public void onPlayerChat(final PlayerChatEvent event) + public void onPlayerChat(final AsyncPlayerChatEvent event) { } - public ChatStore getChatStore(final PlayerChatEvent event) + public ChatStore getChatStore(final AsyncPlayerChatEvent event) { return chatStorage.get(event); } - public void setChatStore(final PlayerChatEvent event, final ChatStore chatStore) + public void setChatStore(final AsyncPlayerChatEvent event, final ChatStore chatStore) { chatStorage.put(event, chatStore); } - public ChatStore delChatStore(final PlayerChatEvent event) + public ChatStore delChatStore(final AsyncPlayerChatEvent event) { return chatStorage.remove(event); } - protected void chargeChat(final PlayerChatEvent event, final ChatStore chatStore) + protected void chargeChat(final AsyncPlayerChatEvent event, final ChatStore chatStore) { try { @@ -75,7 +75,7 @@ public abstract class EssentialsChatPlayer implements Listener } } - protected void formatChat(final PlayerChatEvent event, final ChatStore chatStore) + protected void formatChat(final AsyncPlayerChatEvent event, final ChatStore chatStore) { final IUser user = chatStore.getUser(); if (Permissions.CHAT_COLOR.isAuthorized(user)) @@ -109,7 +109,7 @@ public abstract class EssentialsChatPlayer implements Listener } } - protected void handleLocalChat(final PlayerChatEvent event, final ChatStore chatStore) + protected void handleLocalChat(final AsyncPlayerChatEvent event, final ChatStore chatStore) { ISettings settings = ess.getSettings(); long radius = settings.getData().getChat().getLocalRadius(); diff --git a/EssentialsChat/src/net/ess3/chat/EssentialsLocalChatEvent.java b/EssentialsChat/src/net/ess3/chat/EssentialsLocalChatEvent.java index e0ce6602d..d3bafa984 100644 --- a/EssentialsChat/src/net/ess3/chat/EssentialsLocalChatEvent.java +++ b/EssentialsChat/src/net/ess3/chat/EssentialsLocalChatEvent.java @@ -4,7 +4,7 @@ import org.bukkit.entity.Player; import org.bukkit.event.Cancellable; import org.bukkit.event.Event; import org.bukkit.event.HandlerList; -import org.bukkit.event.player.PlayerChatEvent; +import org.bukkit.event.player.AsyncPlayerChatEvent; public class EssentialsLocalChatEvent extends Event implements Cancellable @@ -14,7 +14,7 @@ public class EssentialsLocalChatEvent extends Event implements Cancellable private String format = "<%1$s> %2$s"; private long radius; private boolean cancelled = false; - private PlayerChatEvent parentEvent = null; + private AsyncPlayerChatEvent parentEvent = null; private static final HandlerList handlers = new HandlerList(); public EssentialsLocalChatEvent(final Player player, final String message, final String format, final long radius) @@ -25,7 +25,7 @@ public class EssentialsLocalChatEvent extends Event implements Cancellable this.radius = radius; } - public EssentialsLocalChatEvent(final PlayerChatEvent event, final long radius) + public EssentialsLocalChatEvent(final AsyncPlayerChatEvent event, final long radius) { this(event.getPlayer(), event.getMessage(), event.getFormat(), radius); this.parentEvent = event; @@ -93,7 +93,7 @@ public class EssentialsLocalChatEvent extends Event implements Cancellable this.radius = radius; } - public PlayerChatEvent getParentEvent() + public AsyncPlayerChatEvent getParentEvent() { return parentEvent; } diff --git a/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerHighest.java b/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerHighest.java index dca0c3b79..3ecb1f733 100644 --- a/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerHighest.java +++ b/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerHighest.java @@ -7,21 +7,21 @@ import net.ess3.chat.EssentialsChatPlayer; import org.bukkit.Server; import org.bukkit.event.EventHandler; import org.bukkit.event.EventPriority; -import org.bukkit.event.player.PlayerChatEvent; +import org.bukkit.event.player.AsyncPlayerChatEvent; public class EssentialsChatPlayerListenerHighest extends EssentialsChatPlayer { public EssentialsChatPlayerListenerHighest(final Server server, final IEssentials ess, - final Map chatStorage) + final Map chatStorage) { super(server, ess, chatStorage); } @EventHandler(priority = EventPriority.HIGHEST) @Override - public void onPlayerChat(final PlayerChatEvent event) + public void onPlayerChat(final AsyncPlayerChatEvent event) { final ChatStore chatStore = delChatStore(event); if (event.isCancelled()) diff --git a/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerLowest.java b/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerLowest.java index f5a8c26ab..836d6af86 100644 --- a/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerLowest.java +++ b/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerLowest.java @@ -8,27 +8,32 @@ import net.ess3.chat.EssentialsChatPlayer; import org.bukkit.Server; import org.bukkit.event.EventHandler; import org.bukkit.event.EventPriority; -import org.bukkit.event.player.PlayerChatEvent; +import org.bukkit.event.player.AsyncPlayerChatEvent; public class EssentialsChatPlayerListenerLowest extends EssentialsChatPlayer { public EssentialsChatPlayerListenerLowest(final Server server, final IEssentials ess, - final Map chatStorage) + final Map chatStorage) { super(server, ess, chatStorage); } @EventHandler(priority = EventPriority.LOWEST) @Override - public void onPlayerChat(final PlayerChatEvent event) + public void onPlayerChat(final AsyncPlayerChatEvent event) { if (event.isCancelled()) { return; } final IUser user = ess.getUserMap().getUser(event.getPlayer()); + if(user == null) + { + event.setCancelled(true); + return; + } final ChatStore chatStore = new ChatStore(ess, user, getChatType(event.getMessage())); setChatStore(event, chatStore); formatChat(event, chatStore); diff --git a/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerNormal.java b/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerNormal.java index 201d946c5..bef4322d6 100644 --- a/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerNormal.java +++ b/EssentialsChat/src/net/ess3/chat/listenerlevel/EssentialsChatPlayerListenerNormal.java @@ -7,21 +7,21 @@ import net.ess3.chat.EssentialsChatPlayer; import org.bukkit.Server; import org.bukkit.event.EventHandler; import org.bukkit.event.EventPriority; -import org.bukkit.event.player.PlayerChatEvent; +import org.bukkit.event.player.AsyncPlayerChatEvent; public class EssentialsChatPlayerListenerNormal extends EssentialsChatPlayer { public EssentialsChatPlayerListenerNormal(final Server server, final IEssentials ess, - final Map chatStorage) + final Map chatStorage) { super(server, ess, chatStorage); } @EventHandler(priority = EventPriority.NORMAL) @Override - public void onPlayerChat(final PlayerChatEvent event) + public void onPlayerChat(final AsyncPlayerChatEvent event) { if (event.isCancelled()) { -- cgit v1.2.3