help needed with AsyncPlayerChatEvent

I have a problem I want to make prefixes with its own plugin and permissions but if I give permission that I'm right all other prefixes to allow me to name

here is the code:

Code:java

package me.beno65dev.chat;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.AsyncPlayerChatEvent;
 
public class Chat implements Listener{
 
    @EventHandler
    public void onChat(AsyncPlayerChatEvent e){
        Player p = e.getPlayer();
        String name = p.getName();
        String message = e.getMessage();
        e.setCancelled(true);
        StringBuilder sb = new StringBuilder();
        if (p.hasPermission("chat.member") && !p.isOp()){
            sb.append(ChatColor.GRAY + "Member " + ChatColor.WHITE + name + ": " + message);
        if (p.hasPermission("chat.elite") && !p.isOp()){
            sb.append(ChatColor.DARK_GREEN + "Elite" + name + ChatColor.WHITE +": " + message);
            }
        if (p.hasPermission("chat.vip") && !p.isOp()){
            sb.append(ChatColor.GOLD + "VIP" + name + ChatColor.WHITE + ": " + message);
            }
        if (p.hasPermission("chat.team") && p.isOp()){
            sb.append(ChatColor.AQUA + "Team" + name + ChatColor.WHITE + ": " + message);
            }
        }
        Bukkit.broadcastMessage(sb.toString());
    }
}

i hope you can help me

 

What is being asked is unclear, but I'm going to take a guess by looking at your code. It looks like if a player has "chat.vip" for example, he will also have all the prefixes before it. This can easily be solved by using "else if" statements, instead of using just "if" statements. The changed code should look like this.

Code:java

if (p.hasPermission("chat.member") && !p.isOp()){
                sb.append(ChatColor.GRAY + "Member " + ChatColor.WHITE + name + ": " + message);
            }else if (p.hasPermission("chat.elite") && !p.isOp()){
                sb.append(ChatColor.DARK_GREEN + "Elite" + name + ChatColor.WHITE +": " + message);
            }else if (p.hasPermission("chat.vip") && !p.isOp()){
                sb.append(ChatColor.GOLD + "VIP" + name + ChatColor.WHITE + ": " + message);
            }else if (p.hasPermission("chat.team") && p.isOp()){
                sb.append(ChatColor.AQUA + "Team" + name + ChatColor.WHITE + ": " + message);
                }

 

What you wana do?

 

mine-care
its for a server that a friend of my have made

 

AndrewAnderVille That would only be the case if they have more than one of the permissions listed.

Beno65Dev You don't really explain what's wrong but you don't really seem to be using StringBuilder correctly. Generally it's used as opposed to concatenation, but you concatenate Strings inside the append statement. It's pretty pointless the way you're doing it.

 

You might want event.setMessage(messageString), or event.setFormat(formatString). I wouldn't know, as the question is very unclear!

 

ever_x that will override the existing ones like group manager ranks won't show prefixes or suffixes... Mabe getdisplayname plus the custom name thingy if you want both

 

mine-care It wont override stuff if you use the monitor event priority ( so your listener gets called last, I think - you may want to check on JavaDocs), then use e.setFormat/Message(e.getFormat / message then your new stuff). That way you can add formats and keep the existing stuff from other plugins. I haven't done this myself, so I cant give you code, but hey, why would I?

 

It will still override stuff, but it is not ment to do that in MONITOR (pick from LOWEST-HIGHEST instead)
Beno65Dev If you are op you won't get any prefix. Instead, use plugin.yml to set op perms as default false and remove the !player.isOp()

Code:

permissions:
    chat.member:
        default: false
    chat.vip:
        default: false
    chat.elite:
        default: false
    chat.team:
        default: false

(Something to read)