PlayerJoinEvent not firing

I am new to bukkit development and I created a small plugin for our server. The plugin needs to do some tasks when a player joins. however, when I added a playerJoinEvent to my listeners class, it does not seem to trigger. The player never gets the message. I have set up my plugin correctly and registered my listeners class, and I know this because all the other events in my code work. I really hope to be able to solve this problem so I can put up my server. Thanks!
Onenable method:

Code:java

 
 
@Override
    public void onEnable() {
        getLogger().info("War Plugin Enabled!");
        getServer().getPluginManager().registerEvents(new WarListener(this), this);
        groups.Groups.england = (List<String>) this.getConfig().getList("england");
        groups.Groups.france = (List<String>) this.getConfig().getList("france");
    }
 

entire listener class:

Code:java

 
package Main;
 
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.block.BlockBreakEvent;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.player.PlayerChatEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerLoginEvent;
 
public class WarListener implements Listener {
 
    public Main plugin;
 
    public WarListener(Main instance){
        plugin = instance;
    }
 
    @EventHandler
    public void OnPlayerInteract(PlayerInteractEvent event){
        if (event.getClickedBlock().getX() > 499 && event.getClickedBlock().getX() < 1501 &&
                event.getClickedBlock().getZ() > -501 && event.getClickedBlock().getZ() < 501){
 
            if (groups.Groups.england.contains(event.getPlayer().toString())){
                event.setCancelled(true);
                event.getPlayer().sendMessage(ChatColor.RED + "Oops! Only the French can place here!");
            }
        }
        if (event.getClickedBlock().getX() < -499 && event.getClickedBlock().getX() > -1501 &&
                event.getClickedBlock().getZ() > -501 && event.getClickedBlock().getZ() < 501){
 
            if (groups.Groups.france.contains(event.getPlayer().toString())){
                event.setCancelled(true);
                event.getPlayer().sendMessage(ChatColor.RED + "Oops! Only the English can place here!");
            }
        }
 
    }
    @EventHandler
    public void OnBlockBreak(BlockBreakEvent event){
        if (event.getBlock().getX() > 499 && event.getBlock().getX() < 1501 && event.getBlock().getZ() > -501 &&
                event.getBlock().getZ() < 501){
            if (groups.Groups.england.contains(event.getPlayer().toString())){
                event.setCancelled(true);
                event.getPlayer().sendMessage(ChatColor.RED + "Oops! Only the French can break here!");
            }
        }
        if (event.getBlock().getX() < -499 && event.getBlock().getX() > -1501 &&
                event.getBlock().getZ() > -501 && event.getBlock().getZ() < 501){
 
            if (groups.Groups.france.contains(event.getPlayer().toString())){
                event.setCancelled(true);
                event.getPlayer().sendMessage(ChatColor.RED + "Oops! Only the English can break here!");
            }
        }
    }
    @EventHandler
    public void OnPlayerAttack(EntityDamageByEntityEvent event){
        if (event.getEntity() instanceof Player && event.getDamager() instanceof Player){
            if((groups.Groups.england.contains(event.getEntity().toString()) && groups.Groups.england.contains(event.getDamager().toString()))
                    || (groups.Groups.france.contains(event.getEntity().toString()) && groups.Groups.france.contains(event.getDamager().toString()))){
                event.setCancelled(true);
                Player damager = (Player) event.getDamager();
                damager.sendMessage(ChatColor.RED + "You can't hurt your fellow Countrymen!");
            }
 
        }
    }
    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event){
        groups.Groups.chatmode.put(event.getPlayer().toString(), false);
        event.setJoinMessage("welcome to the server");
    }
    @EventHandler
    public void onPlayerChat(PlayerChatEvent event){
        if (groups.Groups.chatmode.get(event.getPlayer().toString())){
            event.setCancelled(true);
            if (groups.Groups.england.contains(event.getPlayer().toString())){
                for(Player player : Bukkit.getServer().getOnlinePlayers()){
                    if(groups.Groups.england.contains(player.toString())){
                        player.sendMessage(ChatColor.GREEN + "[Allied Chat]" + event.getPlayer().getName() + ": " + ChatColor.DARK_GRAY + event.getMessage());
                    }
                }
            }
            if (groups.Groups.france.contains(event.getPlayer().toString())){
                for(Player player : Bukkit.getServer().getOnlinePlayers()){
                    if(groups.Groups.france.contains(player.toString())){
                        player.sendMessage(ChatColor.GREEN + "[Allied Chat]" + event.getPlayer().getName() + ": " + ChatColor.DARK_GRAY + event.getMessage());
                    }
                }
            }
        }
    }
}
 

 

@Nukreeper64

Im pretty sure its event.getPlayer().getName();

 

Have you registered the listener in the main class?
Get your plugin manager and call registerEvents(yourlistener, this); in the main class.

 

Yes, I have registered my listener. As I said in my post, all of my other events work.

 

@Nukreeper64
either use @567legodude's solution or check the console if there is an exception in the line before the message

 

@Nukreeper64 May I ask where you are getting the line groups.Groups.chatmode.put(event.getPlayer().toString, false); from? Also, if you just want to send the player joining a message, use the code below. (Since you're new and all. )

Code:

    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event) {
        Player player = event.getPlayer();
               
                player.sendMessage("This is the message that people see when joining.");

// Also, if you want the message to be colored, use the code below.

                player.sendMessage(ChatColor.RED + "This message is red.");

Hope that I helped a little.

 

what do you mean?
do you mean event.getPlayer().getName().sendMessage()?

 

@Nukreeper64 Please show onEnable method.

 

As I said to legodude, my listener class is already registered, and there are no errors in the code or in the server log before or after I log in.

I edited my post to show onEnable(). hopefully this will provide us with some answers.

 

@Nukreeper64
instead of event.setJoinMessage("welcome to the server");
you should probably do event.getPlayer().sendMessage("Message");

Also, I'm pretty sure, although it has been a while, that using AsyncPlayerChatEvent rather than PlayerChatEvent, is better performance-wise.

 

I changed my code so that it looks exactly like this, but the message never shows up. I have updated my post with more of my code, so maybe that will reveal the problem.

 

@Nukreeper64 Hm... This probably doesn't have to do with anything, but your package looks funny.

Code:

package Main;

// Should be...

package me.Username.PluginName;

// According to Bukkit at least.

 

@Nukreeper64

groups.Groups.chatmode.put(event.getPlayer().getName(), false);

 

that is for another part of the plugin. changing that doesn't affect anything.

 

@Nukreeper64 do this:

Code:

    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event){
    Player player = event.getPlayer();
        player.sendMessage("welcome to the server");
    }

Instead of this:

Code:

 @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event){
        groups.Groups.chatmode.put(event.getPlayer().toString(), false);
        event.setJoinMessage("welcome to the server");
    }