Tell me why this wont work?

@Adamki11s
Could someone tell me why these commands wont work like i want?

https://github.com/stelar7/messenger/blob/master/messenger/src/my/plugin/stelar7/Messenger/MSEX.java

It's supposed to be like a PM between players, and if a user types "/mlog player1" he gets all the messages thats sent from player 1

 

I don't understand; what's going wrong?

Are you trying to have 2 users private chat and the whole server is seeing these messages?

 

>.> Because nowhere in your code are you "getting" the messages from the log.

BTW: this plugin is heavily flawed. Even if you do manage to get all the messages in the log... You can't see but the latest messages.

Otherwise... good job on messy code.

also, you need.

Code:

if(!(sender instanceof Player)) return;

 

@Adamki11s

No, the PM part is working...

What i want is that when someone uses I.E "/mlog player1" they spy on whatever he sends in a PM

It's not getting the code from the logg, its getting it in realtime...

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Then just catch the onPlayerChat event and if the player name is equal to who is being spied on send that player the chat.

 

well you aren't listening to the "spying" correctly.

1- You shouldn't save it as "Player, Player." That limits it to one spy.
2- You're looking if "contains" player. That's wrong as well.
3- Look at the source for some of my chat plugins, see if that helps you.

Wrong, he wants to listen to commands. Which would work fine in onCommand, he's just trying to process the data incorrectly.

IE: (this is completely wrong)

Code:

if (players.containsValue(player)) {
	op.sendMessage(ChatColor.BLUE + "[" + msender
		+ "] Til [" + player.getName() + "] : " + temp);
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

How should i save it then?

why?

 

bump

 

I've given my input, I'm not going to just hand you the fixes.

Store it as:
Player, List<Player>

You need to loop through all players to see if they are spying that player. And I tell you, Quantum Connectors, ChannelChat, IgnoreChat, and I think PailPlus all have examples of doing it.