Teleportation wait, dont move problems!

Okay, basically im trying to have it when you type /warp 1v1 you warp to that location, and if there is a player nearby you have to wait 6seconds but if you move for what ever reason you cant teleport, you get a message saying the teleportation has been cancelled.
sCommands.java

Code:java

 
package me.sensethegod;
 
import me.sensethegod.commands.Commands;
import me.sensethegod.listeners.PlayerListener;
 
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class sCommands extends JavaPlugin {
private Commands executor;
 
@Override
public void onEnable() {
 this.executor = new Commands(this);
 getServer().getPluginManager().registerEvents(new PlayerListener(this.executor), this);
 getLogger().info("SenseTheGod's plugin enabled!");
}
 
@Override
public boolean onCommand(CommandSender sender, Command cmd, String Label, String[] args) {
if (sender instanceof Player) {
return this.executor.execute(sender, cmd, args); //returns true/false
} else {
getLogger().warning("Console isn't supported!");
}
return false; //fall back to return false
}
}
 

Commands.java

Code:java

 
package me.sensethegod.commands;
 
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
 
import me.sensethegod.sCommands;
 
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
 
public class Commands {
private final sCommands plugin;
 
public Commands(sCommands instance) {
this.plugin = instance;
}
 
public int WarpTask;
public List<String> playersWaitingToWarp = new ArrayList<String>();
 
public boolean execute(CommandSender s, Command c, String[] a) {
if (s instanceof Player) {
final Player player = (Player) s;
 
   if (c.getName().equalsIgnoreCase("help"))
   {
     player.sendMessage(ChatColor.DARK_GRAY + "--=======ProphecyKits Help=======--");
     player.sendMessage(ChatColor.BOLD + "IP: Prophecykits.us");
     player.sendMessage(ChatColor.GRAY + "Type /kits to view all of our kits");
     player.sendMessage(ChatColor.GRAY + "Type /bal to view your balance");
     player.sendMessage(ChatColor.GRAY + "Type /staff to view our staff");
     player.sendMessage(ChatColor.GRAY + "Type /warps to view all the warps");
     player.sendMessage(ChatColor.GRAY + "Type /donate to purchase a rank");
     player.sendMessage(ChatColor.GRAY + "Type /lag to view the amount of lag(0% Lag)");
     player.sendMessage(ChatColor.GRAY + "Type /helpop <message> to recieve help from staff");
 
     return true;
   }
 
   else if (c.getName().equalsIgnoreCase("warps"))
   {
     player.sendMessage(ChatColor.DARK_GRAY + "ProphecyWarps: " + ChatColor.GRAY + "1v1, 1v1FPS, Brackets, RaceOfDeath, Shop, HG");
     return true;
   }
 
   else if (c.getName().equalsIgnoreCase("warp")) {
   if (a.length == 1) {
           if (a[0].equalsIgnoreCase("1v1")) {
           List<Entity> nearbyEntities = player.getNearbyEntities(16.0, 16.0, 16.0);
           if (nearbyEntities.isEmpty()) {
       Location location = new Location(player.getWorld(), 10835, 212, 10208);
       player.teleport(location);
       player.sendMessage(ChatColor.AQUA + "You have been teleported to 1v1 arena.");
           } else {
           Iterator<Entity> nearbyEntityIterator = nearbyEntities.iterator();
           while(nearbyEntityIterator.hasNext()) {
           if (nearbyEntityIterator.hasNext() && (nearbyEntityIterator.next() instanceof Player)) {
           player.sendMessage(ChatColor.GRAY + "There is a player nearby, you must wait 6 seconds to teleport, don't move!");
           this.playersWaitingToWarp.add(player.getName());
           WarpTask = this.plugin.getServer().getScheduler().scheduleSyncDelayedTask(this.plugin, new Runnable() {
           @Override
           public void run() {
           Location location = new Location(player.getWorld(), 10835, 212, 10208);
           player.teleport(location);
           player.sendMessage(ChatColor.AQUA + "You have been teleported to 1v1 arena.");
           }
           }, 120L);
           this.playersWaitingToWarp.remove(player.getName());
           return true;
           }
           }
           }
           }
 
           if (a[0].equalsIgnoreCase("1v1fps")) {
           Location location = new Location(player.getWorld(), 10679, 137, 10065);
                player.teleport(location);
                player.sendMessage(ChatColor.AQUA + "You have been teleported to FPS arena.");
           }
 
           else if (a[0].equalsIgnoreCase("shop")) {
           Location location = new Location(player.getWorld(), 10671, 13, 10089);
                player.teleport(location);
                player.sendMessage(ChatColor.AQUA + "You have been teleported to the shop.");
           }
 
           else if (a[0].equalsIgnoreCase("brackets")) {
           Location location = new Location(player.getWorld(), -609, 6, -527);
                player.teleport(location);
                player.sendMessage(ChatColor.AQUA + "You have been teleported to the brackets.");
           }
 
           else if (a[0].equalsIgnoreCase("raceofdeath")) {
           Location location = new Location(player.getWorld(), 10789, 8, 10003);
                player.teleport(location);
                player.sendMessage(ChatColor.AQUA + "You have been teleported to the race of death.");
           }
 
           else if (a[0].equalsIgnoreCase("hg")) {
           Location location = new Location(player.getWorld(), 2946, 11, 5234);
                player.teleport(location);
                player.sendMessage(ChatColor.AQUA + "You have been teleported to hg.");
           }
   } else {
player.sendMessage(ChatColor.DARK_GRAY + "ProphecyWarps: " + ChatColor.GRAY + "1v1, 1v1FPS, Brackets, RaceOfDeath, Shop, HG");
   return true;
   }
 
   if (c.getName().equalsIgnoreCase("plugins"))
   {
   player.sendMessage(ChatColor.WHITE + "Plugins (3): " + ChatColor.GREEN + "MuchProphecy, SuchKits, Wow");
   return true;
   }
 
   if (c.getName().equalsIgnoreCase("pl"))
   {
     player.sendMessage(ChatColor.WHITE + "Plugins (3): " + ChatColor.GREEN + "MuchProphecy, SuchKits, Wow");
     return true;
   }
   if (c.getName().equalsIgnoreCase("faggot"))
   {
     player.sendMessage(ChatColor.GRAY + "Why did you type /faggot? I think your a faggot.");
     return true;
   }
   if (c.getName().equalsIgnoreCase("prophecykits"))
   {
     player.sendMessage(ChatColor.GREEN + "The best server ever kid, btw your eZ.");
     return true;
   }
   if (c.getName().equalsIgnoreCase("staff"))
   {
     player.sendMessage(ChatColor.DARK_GRAY + "-=====-Staff List-=====-");
     player.sendMessage(ChatColor.GRAY + "Owner: " + ChatColor.DARK_RED + "Negations, Zepthic.");
     player.sendMessage(ChatColor.GRAY + "Coder: " + ChatColor.AQUA + "DatSword, MooshViolet.");
     player.sendMessage(ChatColor.GRAY + "Admin: " + ChatColor.RED + "Industrialize.");
     player.sendMessage(ChatColor.GRAY + "Mod: " + ChatColor.DARK_PURPLE + "RedEyedBadger, InsideInside.");
     player.sendMessage(ChatColor.GRAY + "T-Mod: " + ChatColor.LIGHT_PURPLE + "Rabiesking, vaporRaids. ");
 
     return true;
   }
 }
}
return false;
}
}
 

PlayerListener.java

Code:java

 
package me.sensethegod.listeners;
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerMoveEvent;
import me.sensethegod.commands.Commands;
public class PlayerListener implements Listener {
private Commands commands;
 
public PlayerListener(Commands cmdInstance) {
this.commands = cmdInstance;
}
 
 
@EventHandler(priority = EventPriority.NORMAL, ignoreCancelled = false)
public void pME(PlayerMoveEvent event) {
if (this.commands.playersWaitingToWarp.contains(event.getPlayer().getName())) {
this.commands.playersWaitingToWarp.remove(event.getPlayer().getName());
event.getPlayer().sendMessage(ChatColor.DARK_GRAY + "You moved so you were unable to warp!");
Bukkit.getScheduler().cancelTask(this.commands.WarpTask);
}
}
}
 

Please help me fix this, all help is appreciated it! Please give me good answers too!

 

SenseTheGod your current code will cancel teleportation even when you just moved the cursor, you should check if e.getFrom() != e.getTo() before canceling it.

 

Hello! I attempted this myself, and I seem that I succeeded. If you move just one block, it will cancel the teleportation. You can still move your mouse; you can even jump. I used a BukkitRunnable and it checked the conditions while the time was greater than or equal to 0. It was fairly simple, and I'll give you the code, but you'll have to modify it to your style.

Code:java

public final class WarpDelay extends JavaPlugin implements Listener {
 
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
    }
 
    public boolean onCommand(CommandSender s, Command c, String l, String[] args) {
        String cmd = c.getName();
        if (!(s instanceof Player)) return false;
        final Player p = (Player) s;
 
        if (cmd.equalsIgnoreCase("warpme")) {
            Location loc = p.getLocation();
            final int x = loc.getBlockX();
            final int y = loc.getBlockY();
            final int z = loc.getBlockZ();
            final int delay = 6;
            p.sendMessage("You will teleport in " + delay + " seconds");
            new BukkitRunnable() {
                int t = delay;
 
                @Override
                public void run() {
                    if (t <= 0) {
                        cancel();
                        p.sendMessage("Teleporting!");
                        // p.teleport(location);
                        return;
                    }
 
                    Location loc2 = p.getLocation();
                    int xx = loc2.getBlockX();
                    int yy = loc2.getBlockY();
                    int zz = loc2.getBlockZ();
 
                    if (xx != x || yy != y || zz != z) {
                        p.sendMessage("You moved, so teleportation was cancelled!");
                        cancel();
                    }
 
                    t--;
                }
            }.runTaskTimer(this, 20L, 20L);
            return true;
        }
        return false;
    }
}
 

 

CoderMusgrove , I tested it out on my own teleporting plugin and it works except for one problem, if you teleport successfully, it plays the movement message. It might be just me, but if it isn't I would put the movement message in a for loop and add a boolean to check if it was the teleporting itself that triggered the movement message.

 

Okay thanks man! I really appreciate it, Do you mind putting in the fixed code in please?

 

This should work in theory but this is CoderMusgrove's example edited by text so it might not. Let me do a bit more testing before using my code over Musgrove's.

Code:

public final class WarpDelay extends JavaPlugin implements Listener {
 
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
    }
 
    public boolean onCommand(CommandSender s, Command c, String l, String[] args) {
        String cmd = c.getName();
        if (!(s instanceof Player)) return false;
        final Player p = (Player) s;
 
        if (cmd.equalsIgnoreCase("warpme")) {
            Location loc = p.getLocation();
            final int x = loc.getBlockX();
            final int y = loc.getBlockY();
            final int z = loc.getBlockZ();
            final int delay = 6;
            p.sendMessage("You will teleport in " + delay + " seconds");
            new BukkitRunnable() {
                int t = delay;
 
                @Override
                public void run() {
                boolean haveTeleported = false;
                    if (t <= 0) {
                        cancel();
                        p.sendMessage("Teleporting!");
                        // p.teleport(location);
                        haveTeleported = true;
                    }
 
                    Location loc2 = p.getLocation();
                    int xx = loc2.getBlockX();
                    int yy = loc2.getBlockY();
                    int zz = loc2.getBlockZ();
 
                    if(haveTeleported = false) {
                    if (xx != x || yy != y || zz != z) {
                        p.sendMessage("You moved, so teleportation was cancelled!");
                        cancel();
                    }
                    }
 
                    t--;
                }
            }.runTaskTimer(this, 20L, 20L);
            return true;
        }
        return false;
    }
}

 

Do you think you could just make one like mine? Sorry, im new to all of this.

 

That's very strange, I'm not getting that at all.

Edit: I think it might be that I forgot to return; at the end!
Original Post Edited.

 

This is the code from my edited version, make sure to change the variables to your own. Its a bit slow but I think that it was already slow so im not sure if changing it is the cause.

Code:java

 
new BukkitRunnable() {
 int t = delay;
@SuppressWarnings("unused")
boolean haveTeleported = false;
@Override
public void run() {
if (t <= 0) {
cancel();
player.sendMessage("Teleporting!");
//player.teleport(location);
haveTeleported = true;
}
 
Location loc2 = player.getLocation();
int xx = loc2.getBlockX();
int yy = loc2.getBlockY();
int zz = loc2.getBlockZ();
 
if(haveTeleported = false) {
if (xx != currX || yy != currY || zz != currZ) {
player.sendMessage("You moved, so teleportation was cancelled!");
cancel();
}
}
t--;
}
}.runTaskTimer(this, 20L, 20L);

CoderMusgrove , it could be because I'm using my own plugin for this but I do agree with you that the cancel not working for me is strange. Ill try using your code and seeing if it works for me.

The reason I think your not getting it CoderMusgrove , is because your plugin doesn't actually teleport you and that is what's triggering the message. I guessing another solution would to be just to flip the teleporting and the movement check around. SenseTheGod , what you have to to is take CodesMusgrove's code and put it in a BukkitRunnable(), making your event part useless. If you need me to explain in more detail, feel free to ask since I also just started .

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

 

Thanks man i appreciate it, do you have skype? add my new one just start a conversation with me please.