Placing mushrooms anywhere?

Hi everyone! Recently I tried a code to place mushrooms anywhere. Basically, I made an interact event for when you right clicked a block with any mushroom, it'll get the relative above it, and set it to mushroom. But it made it so it crashed the server! So I need some help with making this, thanks!

 

What was the stack-trace when the server crashed?

 

What have you tried? You got no proof without code. And since your code crashed the server, it might be good to post it so we can spot your errors.

 

teej107

Code:

    @EventHandler
    public void mushroom(PlayerInteractEvent e){
        Player p = e.getPlayer();
        if(p.isOp()){
            if(e.getAction() == Action.RIGHT_CLICK_BLOCK){
               
                if(p.getItemInHand().getType() == Material.RED_MUSHROOM){
                   
                    e.getClickedBlock().getRelative(BlockFace.UP).setType(Material.RED_MUSHROOM);
                   
                }else{
                    if(p.getItemInHand().getType() == Material.BROWN_MUSHROOM){
                       
                        e.getClickedBlock().getRelative(BlockFace.UP).setType(Material.BROWN_MUSHROOM);
                       
                    }else{
                       
                    }
                }
               
            }
        }
    }

That's it, it's not crashing anymore but it's still not working.

 

Code:java

                }else{
                    if(p.getItemInHand().getType() == Material.BROWN_MUSHROOM){
 
                        e.getClickedBlock().getRelative(BlockFace.UP).setType(Material.BROWN_MUSHROOM);
 
                    }else{
 
                    }
                }

Try using the else if(){ feature.
Also, check if p.getItemInHand() isn't null because if they are holding nothing and clicking a block, you cannot get the type.

 

Does the full event not initiate or what? Try debuggin using System.out.println() to see if the code reaches certain sports.

Also, register your events.

 

Use .equals() to compare objects. Other than that, are you sure you are OP? and as others pointed out, make sure your event is registered.

 

Does not matter if he uses .equals() or == for enums

Code:

public final boolean equals(Object other) {
  return this==other;
}

 

Code0 I'll try that, and I've already registered. teej107 Shouldn't matter.

 

Eh, I am having one those days I guess. I totally forgot that Action is an enum. Put up debug code to make sure the event is firing, and to make sure what if tests are returning false.