Solved PlayerCommandPreprocessEvent Executing the Command

So I'm trying to make a HorseTeleport plugin, where if you are riding a horse, and you issue a command that teleports you (based on my plugins: /tp, /warp, /spawn, /home), it will:

1) Save the Horse you are riding on.
2) Remove you from the Horse
3) Execute the command
4) Teleport the saved Horse entity to the player.

Now, I can easily do steps 1, 2, and 4... But I cannot seem to find a way in the event to execute the command the player did. It's probably rather simple and I'm just completely glossing over it. So any help would be appreciated!


Also, the reason I'm using PlayerCommandPreprocessEvent instead of PlayerTeleportEvent is because if you are riding a vehicle (pig, horse, minecart), you won't actually be teleported, so the event never fires.

 

don't cancel the event and it should still work fine

 

soulofw0lf

But doesn't PlayerCommandPreprocessEvent do things before the command takes place? If it does, teleporting the horse to me before the teleporting command happens is pointless...


Just tested it, and indeed it teleports the horse before the command happens, like I suspected. Is there some sort of code that says to execute the command the user entered, and then do more?

 

Schedule a task with one tick delay?

 

Okay, it looks like player.performCommand(String) works and does what I need it to do...

Now is there a way to cancel the actual command the player typed in, so they don't get teleported twice? =P

 

Code:text

event.setCancelled(true);

 

autoit4you

Thanks, that's just what I needed it to do. Was under the impression setting the event to be cancelled would stop anything in it from happening. Guess not.

And for anyone looking for a similar plugin:

Code:java

import org.bukkit.entity.EntityType;
import org.bukkit.entity.Horse;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerCommandPreprocessEvent;
 
public class HorseTeleport implements Listener
{
    @EventHandler
    public void onHorseTP(PlayerCommandPreprocessEvent event)
    {
        String input = event.getMessage();
        if(input.startsWith("/tp") || input.startsWith("/warp") || input.startsWith("/home") || input.startsWith("/spawn"))
        {
            Player p = event.getPlayer();
            if(p.isInsideVehicle() && p.getVehicle().getType() == EntityType.HORSE)
            {
                Horse h = (Horse) p.getVehicle();
                p.leaveVehicle();
                String message = event.getMessage().substring(1);
                p.performCommand(message);
                h.teleport(p);
                event.setCancelled(true);
                return;
            }
        }
        return;
    }
 
}

Feel free to use my code (and change it however you like).