Solved Listeners..........

I have this code:

Code:java

package me.sapply.Hellower;
 
import org.bukkit.Location;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerMoveEvent;
 
public class AFKListener implements Listener {
    @EventHandler
    public void onPlayerMove(PlayerMoveEvent event, Location from, Location to) {
        Player player = event.getPlayer();
        if(AFKCommandExecutor.quest.contains(player.getName())){
            Location loc = player.getLocation();
            player.teleport(loc);
            player.sendMessage("You are afk! You cant move!");
        }
    }
 
}
 

but it doesn't prevent a player who is afk (which i have defined correctely in another class) from moving, what do I do wrong here?

 

Hello Datdenkikniet,
because:
A. an Listener Delegator (the Method with the @EventHandler()) only takes one argument which is the Event itself.
B. don't use teleporation use event.setCancelled(true);
so the code should look like that:

Code:java

public class AFKListener implements Listener {
 
    @EventHandler(ignoreCancelled = true)
    public void onPlayerMove(PlayerMoveEvent event) {
        Player player = event.getPlayer();
        if(AFKCommandExecutor.quest.contains(player.getName())){
            event.setCancelled(true);
            player.sendMessage("You are afk! You cant move!");
        }
 
    }
}

 

Datdenkikniet MP5K Cancelling the event will make it look super glitchy... try:

Code:java

event.setTo(event.getFrom());

Instead of cancelling the event.

 

it still doesn't work?!

 

are you sure you added the player to your "quest" List and that you registered your listener?

 

yes, because in my main class, if you use /back when you aren't registered in the hashset it says you are back, and if you are not in the hashmap it says that you aren't afk. (like it should do)
and if you might need it, here is the afk class:

Code:java

public static HashSet<String> quest = new HashSet<String>();
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
        if (cmd.getName().equalsIgnoreCase("afk")){
            if (!(sender instanceof Player)){
                sender.sendMessage("The console cant be afk!");
                return true;
            }
            if (!(quest.contains(sender.getName()))) {
                quest.add(sender.getName());
                sender.sendMessage("You are now afk!");
                Bukkit.broadcastMessage("");
                return true;
        }
            if (quest.contains(sender.getName())){
                sender.sendMessage("You are afk already! Use /back to get back!");
            }
        }
        if (cmd.getName().equalsIgnoreCase("back")){
            if (!(sender instanceof Player)){
                sender.sendMessage("The console cant be afk!");
                return true;
            }
            if (!(quest.contains(sender.getName()))){
                sender.sendMessage("You are not afk!");
            }
            else {
            quest.remove(sender.getName());
            sender.sendMessage("You are back!");
        }
        }
        return false;
    }

 

aha, how do I register my listener if I may ask?

 

call that in your onEnable method.

Code:java

this.getServer().getPluginManager().registerEvents(YOUR LISTENER, this);
 

 

thanks