Don’t expose the complete reply object from the notice event.
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Thu, 16 Oct 2014 20:28:49 +0000 (22:28 +0200)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Thu, 16 Oct 2014 20:28:49 +0000 (22:28 +0200)
src/main/java/net/pterodactylus/irc/Connection.java
src/main/java/net/pterodactylus/irc/event/PrivateNoticeReceived.java
src/main/java/net/pterodactylus/xdcc/core/Core.java

index 15bb319..f6b7913 100644 (file)
@@ -416,7 +416,7 @@ public class Connection extends AbstractExecutionThreadService implements Servic
                                                /* CTCP! */
                                                handleCtcp(reply.source().get(), message);
                                        } else if (!prefixHandler.isChannel(recipient)) {
-                                               eventBus.post(new PrivateNoticeReceived(this, reply));
+                                               eventBus.post(new PrivateNoticeReceived(this, reply.source().get(), message));
                                        } else {
                                                eventBus.post(new ChannelNoticeReceived(this, reply.source().get(), recipient, message));
                                        }
index afb6acc..4089fcb 100644 (file)
 package net.pterodactylus.irc.event;
 
 import net.pterodactylus.irc.Connection;
-import net.pterodactylus.irc.Reply;
+import net.pterodactylus.irc.Source;
 
 /**
  * Notifies a listener that a notice was received.
  *
  * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
-public class PrivateNoticeReceived extends AbstractReplyEvent {
+public class PrivateNoticeReceived extends AbstractConnectionEvent {
 
-       /** The target of the notice. */
-       private final String target;
-
-       /** The text of the notice. */
+       private final Source source;
        private final String text;
 
-       /**
-        * Creates a new notice received event.
-        *
-        * @param connection
-        *              The connection the event occured on
-        * @param reply
-        *              The reply that caused the event
-        */
-       public PrivateNoticeReceived(Connection connection, Reply reply) {
-               super(connection, reply);
-               this.target = reply.parameters().get(0);
-               this.text = reply.parameters().get(1);
+       public PrivateNoticeReceived(Connection connection, Source source, String text) {
+               super(connection);
+               this.source = source;
+               this.text = text;
        }
 
-       //
-       // ACCESSORS
-       //
-
-       /**
-        * Returns the target of the notice.
-        *
-        * @return The target of the notice
-        */
-       public String target() {
-               return target;
+       public Source source() {
+               return source;
        }
 
-       /**
-        * Returns the text of the notice.
-        *
-        * @return The text of the notice
-        */
        public String text() {
                return text;
        }
index 8d983d5..509d30e 100644 (file)
@@ -730,7 +730,7 @@ public class Core extends AbstractExecutionThreadService {
                        return;
                }
 
-               eventBus.post(new GenericMessage(String.format("Notice from %s (%s): %s", privateNoticeReceived.reply().source().get(), network.get(), privateNoticeReceived.text())));
+               eventBus.post(new GenericMessage(String.format("Notice from %s (%s): %s", privateNoticeReceived.source(), network.get(), privateNoticeReceived.text())));
        }
 
        /**