Solved EventHandler can't be resolved to a type?

Hi, this is my first plugin that I'm making, so please bear with me if I learn a bit too slow.
Anyways, I'm trying to make a small plugin for my server which only allows you to drop diamonds, but nothing else.

Here is the code:

Code:

package me.Raydond123;

import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.entity.EventHandler;
import org.bukkit.event.player.PlayerDropItemEvent;
import org.bukkit.plugin.java.JavaPlugin;

public class NoDrop extends JavaPlugin{
  
    @Override
    public void onEnable() {
        getLogger().info("Trinket Drop (Version 1.0) has been enabled!");
    }
  
    @Override
    public void onDisable() {
        getLogger().info("Trinket Drop (Version 1.0) has been disabled!");
    }
  
  
    @EventHandler
  
    public void nodrop(PlayerDropItemEvent e)
    {
      
        Player p = e.getPlayer();
      
        if(e.getItemDrop().getItemStack().getType() == Material.DIAMOND)
        {
          
            e.setCancelled(false);
          
        } else
        {
          
            e.setCancelled(true);
          
            p.sendMessage(ChatColor.RED + "You can only drop trinkets!");
          
        }
      
    }
  
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
      
        if (cmd.getName().equalsIgnoreCase("tdrop") && sender instanceof Player ) {
          
            Player player = (Player) sender;
            player.sendMessage(ChatColor.RED + "This is Raydond's first plugin that works! - TrinketDrop v1.0");
          
            return true;
          
        }
      
        return false;
    }
  
}

Could someone please help?

Thanks,
Raymond.

 

@Raydond123 No need to use loggers, Bukkit does it for you.

 

@Raydond123 Well you can change the

Code:

 if(e.getItemDrop().getItemStack().getType() == Material.DIAMOND) {
            e.setCancelled(false);
        } else {
            e.setCancelled(true);
            p.sendMessage(ChatColor.RED + "You can only drop trinkets!");
        }

to

Code:

if (!e.getItemDrop().getItemStack.getType() == Material.DIAMOND) {
e.getCancelled(true);
p.sendMessage("You can only drop trinkets!");
}

 

@Raydond123 Not implementing Listener and I have no idea if you need to register Listeners if they are in the same class. If so you need to do that.

 

@bwfcwalshy Yes, you have to register the Listener if it's in the same class with PluginManager#registerEvents(this, this)

 

Hm.... Okay, I'll take off the loggers.
@nverdier Okay, I've tried what you said on your first post but, it gives me getItemStack cannot be resolved or is not a field.
And, what do you guys mean "implementing Listener"? Sorry, I'm very new to this.

 

There is a duplication glitch in 1.8.1 where players can drop items, open up a 2nd Minecraft client, log-in, and they will still have the dropped items in their inventory plus the items on the ground.

My point is that be careful about allowing players to simply drop diamonds un-checked because they will be able to duplicate them with the glitch.

 

@Raydond123 You know how you extended Java plugin, add on to that and implement Listener.

 

Okay, @bwfcwalshy , I added it but it gave me a bunch of errors. So I changed it to "implements Listener" with an "s".
Now, it tells, Listener cannot be resolved to a type.

 

Code:java

public class NoDrop extends JavaPlugin implements Listener {
    @Override
    public void onEnable() {
        // getLogger().info("Trinket Drop (Version 1.0) has been enabled!");
        Listener listener = this;
        Plugin plugin = this;
        getServer().getPluginManager().registerEvents(listener, plugin);
    }

 

@Europia79 No need for Listener listener = this; and Plugin plugin = this; Just use

Code:

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

 

Here's my code so far:

Code:

package me.Raydond123;

import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.entity.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerDropItemEvent;
import org.bukkit.plugin.Plugin;
import org.bukkit.plugin.java.JavaPlugin;

public class NoDrop extends JavaPlugin implements Listener{
 
    @Override
    public void onEnable() {
        Plugin plugin = this;
        getServer().getPluginManager().registerEvents(this, this);
    }
 
    @Override
    public void onDisable() {
    }
 
 
    @EventHandler
 
    public void nodrop(PlayerDropItemEvent e)
    {
     
        Player p = e.getPlayer();
     
        if (!e.getItemDrop().getItemStack.getType() == Material.DIAMOND) {
            e.setCancelled(true);
            p.sendMessage("You can only drop trinkets!");
            }
     
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
     
        if (cmd.getName().equalsIgnoreCase("tdrop") && sender instanceof Player ) {
         
            Player player = (Player) sender;
            player.sendMessage(ChatColor.RED + "This is Raydond's first plugin that works! - TrinketDrop v1.0");
         
            return true;
         
        }
     
        return false;
    }
 
}

The following errors occur with this code:
EventHandler cannot be resolved to a type line 23
getItemStack cannot be resolved or is not a field line 30
The import org.bukkit.entity.EventHandler cannot be resolved line 8
The method getCancelled(boolean) is undefined for the type PlayerDropItemEvent line 31

 

@Raydond123 Don't try to copy and paste exactly what I put. I made little errors in that just so you wouldn't do that.

 

Sorry, my previous post was messed up because I put an extra line. It's fixed now.

@nverdier Ohhhh okay. I'll learn from that.

New Code:

Code:

package me.Raydond123;

import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.entity.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerDropItemEvent;
import org.bukkit.plugin.Plugin;
import org.bukkit.plugin.java.JavaPlugin;

public class NoDrop extends JavaPlugin implements Listener{
   
    @Override
    public void onEnable() {
        Plugin plugin = this;
        getServer().getPluginManager().registerEvents(this, this);
    }
   
    @Override
    public void onDisable() {
    }
   
   
    @EventHandler
   
    public void nodrop(PlayerDropItemEvent e)
    {
       
        Player p = e.getPlayer();
       
        if (!(e.getItemDrop().getItemStack().getType() == Material.DIAMOND)) {
            e.setCancelled(true);
            p.sendMessage("You can only drop trinkets!");
            }
       
    }
   
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
       
        if (cmd.getName().equalsIgnoreCase("tdrop") && sender instanceof Player ) {
           
            Player player = (Player) sender;
            player.sendMessage(ChatColor.RED + "This is Raydond's first plugin that works! - TrinketDrop v1.0");
           
            return true;
           
        }
       
        return false;
    }
   
}

Okay now, we're back to the same two errors:

EventHandler cannot be resolved to a type line 27
The import org.bukkit.entity.EventHandler cannot be resolved line 8

<The Edit button is your friend, posts merged by Eyamaz.>

 

Do you even understand what each part of your code does?

Java pls.

 

@nverdier

Yes, there's no need for that... You are right. I did that on purpose because it's more readable than (this, this) for newbies...

So... like, if he gets into a situation where he wants to move the listener to another class, then it's more apparent which this needs to be changed to new ExampleListener()

 

@Europia79 I see. But don't spoonfeed!

 

I still can't seem to figure out why the EventHandler isn't working.
Anyone have any ideas?

 

u have to add bukkit as a library

 

I've resorted to using Spigot as a reference library... I can't seem a place to download Bukkit as their download was taken down by a DMCA request...

 

I noticed that your error with the import says "org.bukkit.entity.EventHandler", but you should be importing "
org.bukkit.event.EventHandler". That is likely your issue.

 

Yep, that fixed it! Thanks for helping me out!