Solved Give Item on player join

I want when someone joins my server, to give them the compass to navigate, But when i give the compass, it just doesn't recognize it, but when i make a command for it it works

Code:

Main.java

Code:java

public ItemStack compass = new ItemStack(Material.COMPASS, 1);
    public String world = this.getConfig().getString("spawnworld");
 
    public Main plugin;
 
    private LoginHandler a = new LoginHandler(this);
    private ItemDropHandler b = new ItemDropHandler(this);
    private ItemMoveHandler c = new ItemMoveHandler(this);
 
    public void giveItem(Player p){
        p.getInventory().setItem(4, compass);
    }
 
    @Override
    public void onEnable() {
 
        ItemMeta compassi = compass.getItemMeta();
        compassi.setDisplayName(ChatColor.BLACK + "[" + ChatColor.DARK_AQUA + "Golem" + ChatColor.GREEN + "Navigator" + ChatColor.BLACK + "]");
        List<String> l = Arrays.asList(ChatColor.LIGHT_PURPLE + "Right click to use!");
        compassi.setLore(l);
        compass.setItemMeta(compassi);
 
        PluginManager pm = getServer().getPluginManager();
 
        pm.registerEvents(this.a, this);
        pm.registerEvents(this.b, this);
        pm.registerEvents(this.c, this);
 
        this.getConfig().options().copyDefaults();
        if (this.getConfig().getString("spawnworld") == null) {
            this.getConfig().set("spawnworld", "world");
        }
        this.saveConfig();
 
        // TODO Auto-generated method stub
        super.onEnable();
    }
 
    @Override
    public void onDisable() {
        // TODO Auto-generated method stub
        super.onDisable();
    }
 
    @Override
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        if(command.getName().equalsIgnoreCase("apple")){
            Player p = (Player) sender;
            //WORKS
            giveItem(p);
        }
        return super.onCommand(sender, command, label, args);
    }

LoginHandler.java

Code:java

public class LoginHandler implements Listener {
 
    private Main plugin;
 
    public LoginHandler(Main plugin) {
        this.plugin = plugin;
    }
 
    @EventHandler(priority = EventPriority.HIGHEST)
    public void PlayerJoin(PlayerLoginEvent e) {
        Player p = e.getPlayer();
        System.out.println("DEBUG1");       
        if (p.getWorld() == plugin.getServer().getWorld(plugin.world)) {
            System.out.println("DEBUG2");
            //DOESN'T WORK
            plugin.giveItem(p);
        }
    }
 
}

Does anyone know the answer to this? Thanks already

 

Try using the PlayerJoinEvent.

 

Please read my signature: Player p = (Player) sender;
also does spawnworld world exist?
try instead of == using .equals(obj);
EDIT: as TheMcScavenger said (i didnt notice) use player join event.

 

Thank you very much, that was indeed the problem, Thanks <3