[SOLVED]Duplicated PlayerJoinEvent ? (Duplicated registration of event ;-) )

Hi
In my onPlayerJoin() method, I send a message to the player.
The message is sent 2 times. Why ?
How could I change it ?

 

do you have event.setCancelled ? after de message?

 

Why should I cancel a PlayerJoinEvent ?
It will certainly fix the problem but I'm not sure if it will not had another problem, like if another plugin want to use the PlayerJoinEvent and check if it is cancelled...

 

Can we see code?

 

http://pastebin.com/PkW8XULC
As you can see, I'm using the onPlayerJoin() and not the onPlayerLogin() because of the player.getAddress().getAddress().getHostAddress(); which is Null in onPlayerLogin().
This is done 2 times :

1. player.sendMessage(plugin.HEADER + ChatColor.GREEN
2. + "Bienvenue sur le serveur " + playerName + " !");

 

Urm, you have this twice? :

Code:

                                player.sendMessage(plugin.HEADER + ChatColor.GREEN
                                                + "Bienvenue sur le serveur " + playerName + " !");

 

That was just a wrong copy/paste, I was adding a line. Now look at this :
This is the code
And here is what it does :

 

its not even a cancellable event. And thats a very interesting bug, not quite sure why it's happening. You don't have any other events sending the message correct?

 

I just add the second message, and as you can see it is sent 2 times. The problem is here.

 

yah i just wanted to make sure you didn't forget about a second event you had it in that you forgot about

 

thats a bug :S
Type "return;" after sendding that message... but that is stange because there is no loop...
Mabye you server is having a but on player join... Create another server only with that plugin and see if it dup the message again

 

Prepare to laugh !

PHP:

PluginManager pm = this.getServer().getPluginManager();
        pm.registerEvent(Event.Type.ENTITY_DAMAGE, entityListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.FOOD_LEVEL_CHANGE, entityListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.PLAYER_COMMAND_PREPROCESS, playerListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.PLAYER_CHAT, playerListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.PLAYER_INTERACT, playerListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.SIGN_CHANGE, blockListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.BLOCK_BREAK, blockListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.BLOCK_PLACE, blockListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.PLAYER_JOIN, playerListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.PLAYER_MOVE, playerListener,
                Event.Priority.Normal, this);
        pm.registerEvent(Event.Type.PLAYER_JOIN, playerListener,
                Event.Priority.Normal, this);

Fixed now

 

that'll do it lol, i was actually thinking about that a minute ago but went nah, thats not the problem.

 

I feel stupid now xD
In fact, I add the PLAYER_JOIN event a few days ago, then I deleted it in my playerListener, but not in my main class. Today I had to use it again, and simply add it to the list...