Right Click event not working but Left Click is

Hey guys,
I need some help again .
I'm trying to create a plugin but A left click event will work but not a right. I don't understand why, could you try to give me examples and tell me how to fix this problem.
BTW: This is not my whole code but I know it's working because the Left Click works!
CODE:

Code:java

    @SuppressWarnings("deprecation")
    @EventHandler(priority = EventPriority.LOW)
    public void onPlayerInteract(PlayerInteractEvent event) {
        Player player = event.getPlayer();
        ItemStack held = player.getItemInHand();
        Block target = player.getTargetBlock(null, 100);
        if (event.getAction() == Action.LEFT_CLICK_AIR || event.getAction() == Action.LEFT_CLICK_BLOCK)
        if(held.getType() == Material.STICK){
        ItemMeta meta = held.getItemMeta();
        if(meta != null){
        if(meta.getDisplayName().equalsIgnoreCase("Exploder")){
        player.getWorld().strikeLightning(target.getLocation());
        }else{
        if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK)
        if(held.getType() == Material.STICK){
        if(meta != null){
        if(meta.getDisplayName().equalsIgnoreCase("Exploder")){
        player.getWorld().createExplosion(target.getLocation(), 5);
        }else{
            return;
        }
        }
        }
        }
        }
        }
    }
}

 

josh12341
Try removing both of the else statements as they are not needed

 

It still doesn't work please help!

 

Looks like you have some statements nested where they shouldn't be.

http://www.cs.umd.edu/~clin/MoreJava/ControlFlow/nested-if.html

 

If I'm, wrong please give me an example.
I don't think it matter, but any way here is the new code, Testing now:

Code:java

 @SuppressWarnings("deprecation")
    @EventHandler(priority = EventPriority.LOW)
    public void onPlayerInteract(PlayerInteractEvent event) {
        Player player = event.getPlayer();
        ItemStack held = player.getItemInHand();
        Block target = player.getTargetBlock(null, 100);
        if (event.getAction() == Action.LEFT_CLICK_AIR || event.getAction() == Action.LEFT_CLICK_BLOCK)
          if(held.getType() == Material.STICK){
            ItemMeta meta = held.getItemMeta();
              if(meta != null){
                if(meta.getDisplayName().equalsIgnoreCase("Exploder")){
                  player.getWorld().strikeLightning(target.getLocation());
                    if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK)
                      if(held.getType() == Material.STICK){
                        if(meta != null){
                          if(meta.getDisplayName().equalsIgnoreCase("Exploder")){
                            player.getWorld().createExplosion(target.getLocation(), 5);
        }
        }
        }
        }
        }
        }
    }
}

Nope, still doesn't work.
I'm telling you, the lining doesn't matter.
My left click event is working but just not the right.
If lining does matter can you please fix it and give me an example.
Thanks

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

josh12341 Your "lining" was off.

Code:java

 @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        Player player = event.getPlayer();
        ItemStack held = player.getItemInHand();
        Block target = player.getTargetBlock(null, 100);
        ItemMeta meta = held.getItemMeta();
        if (event.getAction() == Action.LEFT_CLICK_AIR || event.getAction() == Action.LEFT_CLICK_BLOCK) {
            if (held.getType() == Material.STICK) {
                if (meta != null) {
                    if (meta.getDisplayName().equalsIgnoreCase("Exploder")) {
                        player.getWorld().strikeLightning(target.getLocation());
                    }
                }
            }
        }
        if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
            if (held.getType() == Material.STICK) {
                if (meta != null) {
                    if (meta.getDisplayName().equalsIgnoreCase("Exploder")) {
                        player.getWorld().createExplosion(target.getLocation(), 5);
                    }
                }
            }
        }
    }

 

Note: PlayerInteractEvent does not get called if the player is rightclicking a block while holding a item/block in his hand (like, for example, right clicking a sign while having cobblestone in your hand).
It wasn't like this before (I think the change came somewhere around 1.3).

 

OMG thanks, it Works now!