Solved Event not working!

Hey, I am working on a new plugin and what it does is: When a player right clicks with a diamond, it's supposed to teleport them to their /home, a command from essentials.

Code:

Code:java

 
package noah.plugins.pluginx;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class PluginX extends JavaPlugin implements Listener {
 
// To Register
public void onEnable() {
getServer().getPluginManager().registerEvents(this, this);
}
 
public void onDisable() {
 
}
 
// Lets start
@EventHandler
public void onPlayerInteract(PlayerInteractEvent e) {
if (e.getPlayer().hasPermission("hometp.tphome")) {
if (e.getPlayer().getItemInHand().getType() == Material.DIAMOND) {
if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK) {
Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), "home");
e.getPlayer().sendMessage("Hi");
}
}
} else {
e.getPlayer().sendMessage(ChatColor.RED + "You don't have permission to teleport home!");
}
}
 
 
}
 

 

This code will execute "home" from the CONSOLE perspective, and because the CONSOLE has no home it doesnt work. Try to set the player as the commandsender. So replace Bukkit.getConsoleSender() with it.

 

Try this:

Code:java

// Lets start
 
@EventHandler
 
public void onPlayerInteract(PlayerInteractEvent e) {
 
Player p = e.getPlayer();
 
if (e.getPlayer().hasPermission("hometp.tphome")) {
 
if (e.getPlayer().getItemInHand().getType() == Material.DIAMOND) {
 
if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK) {
 
Bukkit.getServer().dispatchCommand(p, "home");
 
e.getPlayer().sendMessage("Hi");
 
}
 
}
 
} else {
 
e.getPlayer().sendMessage(ChatColor.RED + "You don't have permission to teleport home!");
 
}
 
}
 
 
 
 
 
}

 

FabeGabeMC
Why feed him code, when you can simply tell him what's wrong, especially if it only requires a minor change?

Techno
In your Bukkit.dispatchCommand() simply change Bukkit.getConsoleSender() to e.getPlayer().

 

Techno
Player object also has it's built in method of performing a command from the player called performCommand

 

The Gaming Grunts trying to help lol

 

FabeGabeMC
I'm not doubting that. I'm just saying that you don't need to provide a giant block of code when you can simply tell him exactly what the problem is.

 

The Gaming Grunts FabeGabeMC Or just give Techno one of the best tricks for finding issues...
Put broadcast messages throughout the code to see what fires and what doesn't then you can see where the code stops working and fix from there

 

Or simply do p.performCommand because you know that's what it was made for.

 

Thanks everyone! You all really helped!

 

The Gaming Grunts i was sending him his code with the fix.

 

FabeGabeMC
You're still missing the point...