Loading Locations from ArrayList and setting Type doesn't work

I am coding a spleef plugin but the resetting doesn't work!

This is my code:

Code:java

ArrayList<Location> blocks = new ArrayList<Location>();
 
@EventHandler
public void onPlayerInteract(PlayerInteractEvent e) {
if(e.getAction() == Action.LEFT_CLICK_BLOCK) {
if(e.getClickedBlock().getType() == Material.GRASS) {
e.getClickedBlock().setType(Material.AIR);
blocks.add(e.getClickedBlock().getLocation());
}
}
}
 
// In the main class I set the Exceutor to this class
@Override
public boolean onCommand(CommandSender cs, Command cmd, String label, String[] args) {
 
for(Location l : blocks) {
l.getBlock().setTypeId(5);
}
 
blocks.clear();
 
return true;
}

NOTE: setting the clicked block to air works!

 

I don't believe onCommand needs @override before it. Does it?

 

adam753
No it doesn't.
But it doesn't changes anything

 

Uh... well you aren't even providing a command in the onCommand method...

You need to do something like:

Code:

public boolean onCommand(CommandSender cs, Command cmd, String label, String[] args)
{
    if(cmd.getName().equalsIgnoreCase("reset"))
    {
        for(Location l : blocks)
        {
            l.getBlock().setType(5);
        }
        return true;
    }
    return false;
}

 

I don't see what is wrong with that, I've copy pasted your code and it works fine. Have you registered the command properly? Has it been added to your plugin.yml?

Here's my code for reference: http://hastebin.com/xusaqawuno.avrasm