Solved Set ban reason in PlayerLoginEvent

Hey,
How can I get a ban reason from a config and set it if a player logs on he sees the reason why he/she is banned? I wrote a code but it didn't work because Essentials overwrite it each time.

My code:

Code:java

@EventHandler
              public void onFirstJoin(PlayerLoginEvent event){
                  Player player = event.getPlayer();
                  String bm = ChatColor.YELLOW + getConfig().getString("Players." + player.getName() + ".Reason");
                  event.setKickMessage(ChatColor.RED + "Banned: " + bm);
              }

 

Check that event.getResult() == KICK_BANNED, and that bm isn't null before setting the kick message to it
And you may want to set the priority to normal/high to override essentials:
@EventHander (priority = EventPriority.HIGH)

GusGold
Actually PlayerLoginEvent, which does it all simply with event.getReason(), .setKickMessage() and .disallow()

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

 

Chinwe
So I only need to do:

Code:java

@EventHandler (priority = EventPriority.HIGH)
public void onFirstJoin(PlayerLoginEvent event){
Player player = event.getPlayer();
String bm = ChatColor.YELLOW + getConfig().getString("Players." + player.getName() + ".Reason");
event.setKickMessage(ChatColor.RED + "Banned: " + bm);
}

If I've read the last message correct?

 

Josh014 That will override Essentials, though you also need to:

Code:

if (event.getReason() == Result.KICK_BANNED)
{
  String bm = ChatColor.YELLOW + getConfig().getString("Players." + player.getName() + ".Reason");
  if (bm != null)
      event.setKickMessage(bm);
}

 

Chinwe
The only thing what I need to add is the:

Code:java

if (event.getReason() == Result.KICK_BANNED){

 

I use this: (Also, has 2 Lines in the Ban Message ^^)

Code:

    //Ban Join
    @EventHandler(priority = EventPriority.HIGH)
    public void onPlayerBanned (PlayerJoinEvent e) {
    Player player = e.getPlayer();
    String msg = ChatColor.RED + "Your account has been permantly banned!" + ChatColor.RESET + "\n" + ChatColor.GRAY + "If you feel like this is an error talk to us on the forum.";
    if(this.getConfig().contains("ban." + player.getName()))
        if(!player.isOp())
        player.kickPlayer(msg);
      }
 

 

Chinwe
Alright I use this but I get a red error line under "event.getReason()" it says "Add cast to 'event'"

Code:java

              public void onFirstJoin(PlayerLoginEvent event){
                  if (event.getReason() == Result.KICK_BANNED){
                  Player player = event.getPlayer();
                  String bm = ChatColor.YELLOW + getConfig().getString("Players." + player.getName() + ".Reason");
                  if(bm != null){
                  event.setKickMessage(ChatColor.RED + "Banned: " + bm);
                  }
                  }
              }

Chinwe
I think I have it almost but if I join when I'm banned I still see the Essentials message...
Code:

Code:java

@EventHandler (priority = EventPriority.HIGH)
              public void onPlayerBanned(PlayerLoginEvent event){
                  Player player = event.getPlayer();
                  String bm = ChatColor.YELLOW + getConfig().getString("Players." + player.getName() + ".Reason");
                  if (event.getResult() == Result.KICK_BANNED){
                  if(bm != null){
                  event.setKickMessage(ChatColor.RED + "Banned: " + bm);
                  }
                  }
              }

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

 

just asking, did you registered your class as an event listener?
if yes, try priority HIGHEST and print out a message to the console from the event handler
- if you cant see your message, your class doesn't know it has t handle the event
if no, lookup the wiki.bukkit.org how to register a listener class

 

ferrybig
I did registered the event listener & I did HIGHEST and nothing happends.

 

The eventpriority system is kind of upside down, but not really. I'll explain why...

EventPriority.HIGHEST is not the one being run first, it is in fact the exact opposite, so that it can make use of the servers resources (where the word Highest comes from) after all the small plugins have ran their events.

The name at the end is therefore the amount of server resources the plugin is allowed to use, and is named this way to make the server run more smoothly.

Note: If any of this has changed in the last year, please prove me wrong, as I might not be up to date

 

whitehooder
Do you know why my event isn't working then?

 

Josh014 I suggest you try to set the priority to LOWEST, as it will be ran first (or hopefully first)

 

whitehooder
I'm gonna test it out.

whitehooder
I tested it out but without succes >.>

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

 

I'll have a shot at it. Will post here later

 

whitehooder
Thank you . do you need my code?

Code:

Code:java

@EventHandler (priority = EventPriority.LOWEST)
              public void onPlayerBanned(PlayerLoginEvent event){
                  Player player = event.getPlayer();
                  String bm = ChatColor.YELLOW + getConfig().getString("Players." + player.getName() + ".Reason");
                  if (event.getResult() == Result.KICK_BANNED){
                  if(bm != null){
                  event.setKickMessage(ChatColor.RED + "Banned: " + bm);
                  }
                  }
              }

 

Well, this worked out great on the first try:

Code:java

@EventHandler (priority = EventPriority.LOWEST)
public void onPlayerBanned(PlayerLoginEvent event) {
String bm = ChatColor.YELLOW + "You burned his house down... on valentines day...";
if (event.getResult() == Result.KICK_BANNED) {
if(bm != null){
event.setKickMessage(ChatColor.RED + "Banned: " + bm);
}
}
}

Edit: Ahh, no sorry. Installed essentials to test it out and it displayed the essentials ban msg. I'll fool around with it a bit more

Okay, just thought about it for a few seconds, and it's all just basic logics really

You see... The lowest priority is run first, which means you will have to use a higher priority to override the message already set by essentials. I basically just switched it out with EventPriority.HIGHEST and it worked like a charm.

Just for the sake of copy&paste

Code:java

@EventHandler (priority = EventPriority.HIGHEST)
public void onPlayerBanned(PlayerLoginEvent event) {
String bm = ChatColor.YELLOW + "You burned his house down... on valentines day...";
if (event.getResult() == Result.KICK_BANNED) {
if(bm != null){
event.setKickMessage(ChatColor.RED + "Banned: " + bm);
}
}
}

Seems like you have already tried this (with HIGHEST) and I think the reason it didn't work out for you was that bm was null since the configuration failed, so if it doesn't work out of the box for you with config loading, I suggest you add in some debugging messages. Just a basic syso of bm after loading the value from the config would help you out.

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

 

whitehooder
Can you explain debugging messages? I never used them before lol.

 

Basically just output to the console so you can see what is happening. If you think there is a problem when getting the config string you would simply just print out the string to see it for yourself. Example:

Code:java

@EventHandler (priority = EventPriority.HIGHEST)
public void onPlayerBanned(PlayerLoginEvent event) {
    String bm = ChatColor.YELLOW + getConfig().getString("Players." + player.getName() + ".Reason");
    System.out.println("The value of bm is: " + bm);
    if (event.getResult() == Result.KICK_BANNED) {
        if(bm != null){
            event.setKickMessage(ChatColor.RED + "Banned: " + bm);
        }
    }
}
 

If a message saying "The value of bm is: ......" the error is not there, if it prints noting more or you receive a null pointer exception in the console the error lies in that piece of code. Just basic debugging.

 

whitehooder
I don't know what happened but I added the debugging thing and now it works lol... Thank you :3

 

Just remove the debugging line then.

 

I get an error under "KICK_BANNED" help?

 

iMangoDev Full code & error please

 

whitehooder I got an error while doing "Result.KICK_BANNED " so can you help me? And here is the code:

Code:java

@EventHandler (priority = EventPriority.HIGHEST)
    public void Login(PlayerLoginEvent e){
        String bm = "§cYou are banned";
        if(e.getResult() == Result.KICK_BANNED){
            if(bm != null){
                e.setKickMessage(bm);
            }
        }
    }

 

You would have to tell us what the error is Is it an error in the server log when the code is executed or does Eclipse (or whatever IDE you use) show a red line beneath it? Your code looks ok and should be working. (Haven't checked if there is any Result.KICK_BANNED)

 

whitehooder nvm i fixed that i impored the wronk KICK_BANNED