Solved [Bukkit 1.7.2] Sending messages to players with something the same

Hello everyone,

My problem is that if i use this code that checks the players channel and if there is someone else on the server with the same current channel i want that the message will only be send to them:

Code:java

package nl.tamajpm.channels.v1.src.events;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.AsyncPlayerChatEvent;
import org.bukkit.plugin.Plugin;
 
public class OnPlayerChatEvent implements Listener {
    private Plugin plugin;
 
    public OnPlayerChatEvent(Plugin instance) {
        plugin = instance;
    }
 
    @EventHandler
    public void onPlayerChat(AsyncPlayerChatEvent e) {
        Player player = e.getPlayer();
 
        for(Player players : Bukkit.getOnlinePlayers()) {
            if(plugin.getConfig().getString(players.getName() + ".CurrentChannel").equals(plugin.getConfig().getString(player.getName() + ".CurrentChannel"))) {
 
            }
        }
    }
}
 

Sorry, but my English is very bad at this time.

 

You have the basis of the idea, just do players.sendMessage(/*the message content*/); . Also, you should probably load the data when you load the plugin, rather than using the config for every check.

 

Yes, but if i send the message it will be send to all players without checking the channel.

 

Modify the event's recipients list.

 

Oké, another problem. I want this:

- The player needs to be in the same channel as the player to receive the message.
- The player needs to have toggled the channel chat, if he is not he gets the normal chat.
- The player only gets the messages from his channel.

How can i make this, i don't know how to make this. This is the last part of the plugin, so.

Is this right:

Code:java

package nl.tamajpm.channels.v1.src.events;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.AsyncPlayerChatEvent;
import org.bukkit.plugin.Plugin;
 
public class OnPlayerChat implements Listener {
    private Plugin plugin;
 
    public OnPlayerChat(Plugin instance) {
        plugin = instance;
    }
 
    @EventHandler
    public void onPlayerChat(AsyncPlayerChatEvent e) {
        Player player = e.getPlayer();
 
        if(!plugin.getConfig().getString(player.getName() + ".CurrentChannel").equals("")) {
            if(plugin.getConfig().getBoolean(player.getName() + ".Toggle")) {
                for(Player players : Bukkit.getOnlinePlayers()) {
                    e.getRecipients().remove(players);
                }
 
                for(Player players : Bukkit.getOnlinePlayers()) {
                    if(plugin.getConfig().getString(players.getName() + ".CurrentChannel").equals(plugin.getConfig().getString(player.getName() + ".CurrentChannel"))) {
                        e.getRecipients().add(players);
                    }
                }
            }
        }
    }
}
 

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

 

Code:

 for(Player players : Bukkit.getOnlinePlayers()) {
e.getRecipients().remove(players);
}

You can use

Code:

e.getRecipients().clear()

instead of using a loop here.
The rest of the code seems to be ok. Just try it.

But one thing: As the name says, this event is (at least most of the time) "asynchronous".
Means, it's run by a separate thread and not the servers main thread.
In such async tasks you may not use any Bukkit API calls, or bad things may happen.
So if you need to do "player.someting()" or "Bukkit.something()" you should embedd this in a synchronous timer task.

Example:

Code:

plugin.getServer().getScheduler().runTask(plugin, new Runnable()
{
    public void run()
    {
        // your code with Bukkit API calls here
    }
});

 

Thanks

 

I have written the code everything is working but when i'm in a channel i can still get the messages from the public chat?

Sorry, my code at the moment:

Code:java

package nl.tamajpm.channels.v1.src.events;
 
import nl.tamajpm.channels.v1.src.plugin.Channels;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.AsyncPlayerChatEvent;
 
public class OnPlayerChat implements Listener {
 
    @EventHandler
    public void onPlayerChat(AsyncPlayerChatEvent e) {
        Player player = e.getPlayer();
 
        if(!Channels.getPlugin().getConfig().getString(player.getName() + ".CurrentChannel").equals("")) {
            if(Channels.getPlugin().getConfig().getBoolean(player.getName() + ".Toggle") == true) {
                e.getRecipients().clear();
 
                for(Player players : Bukkit.getOnlinePlayers()) {
                    if(Channels.getPlugin().getConfig().getString(players.getName() + ".CurrentChannel").equals(Channels.getPlugin().getConfig().getString(player.getName() + ".CurrentChannel"))) {
                        e.getRecipients().add(players);
                    }
                }
            }
        }
    }
}
 

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

 

Bump

 

Ok, so basically you're checking if the player who sent the message is in a channel, but you also need to check if there are players on the server that are in a channel and remove them from recipients (so they don't get a message from someone In public chat or whatever you called it.
Try something like this

Code:

 @EventHandler
    public void onPlayerChat(AsyncPlayerChatEvent e) {
        Player player = e.getPlayer();
       
        if(!Channels.getPlugin().getConfig().getString(player.getName() + ".CurrentChannel").equals("")) {
            if(Channels.getPlugin().getConfig().getBoolean(player.getName() + ".Toggle") == true) {
                e.getRecipients().clear();
               
                for(Player players : Bukkit.getOnlinePlayers()) {
                    if(Channels.getPlugin().getConfig().getString(players.getName() + ".CurrentChannel").equals(Channels.getPlugin().getConfig().getString(player.getName() + ".CurrentChannel"))) {
                        e.getRecipients().add(players);
                    }
                }
            }
        }
//add this
else{
//loop through players
  for(Player players : Bukkit.getOnlinePlayers()) {
//if that player is in a channel, remove him from recipients
                    if(!Channels.getPlugin().getConfig().getString(players.getName() + ".CurrentChannel").equals("")) {
                        e.getRecipients().remove(players);
                    }
                }
}
    }

Also you shouldn't use the config for every check like 1Rogue said

 

Thanks, that part is working now. But one problem more xD. If a player has disabled (toggled) his channel chat he is getting the channel chat and the public chat. But when he is disabled his channel chat he is should not be able to get that messages from the channel. How i fix this if you can understand me?

 

Actually, I don't really understand :/ so you're saying that if they are in public chat they get messages from the channels?

 

Maybe i can get your Skype i can show you.

Thanks everyone for helping!!!

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