I know this method is called once per hand (so twice) but how can i do so it's called only once, when the player right clicks the block without checking for a specific item in hand?

Edited June 13, 20178 yr by JimiIT92

It's called exactly once.

 

Once for each hand

Then why this code runs twice?

@Override
    public boolean onBlockActivated(World worldIn, BlockPos pos, IBlockState state, EntityPlayer playerIn,
            EnumHand hand, ItemStack heldItem, EnumFacing side, float hitX, float hitY, float hitZ) {
        if(!worldIn.isRemote) {
 
                Faction f = FactionUtils.getFaction(playerIn.getUniqueID());
                if(f != null) {
                    if(Utils.samePos(pos, f.getCore()) && f.getCoreDimension() == playerIn.dimension) {
                        if(f.getOwner().equals(playerIn.getUniqueID())) {
                            int money = f.getMoney();
                            int level = f.getCoreLevel();
                            if(level == CoreFaction.coreMaxLevel) {
                                Utils.sendMessage(playerIn, Utils.getTranslation("faction.upgrade.maxlevel", TextFormatting.RED));
                            }
                            else if(money >= (CoreFaction.coreCost * (level + 1))) {
                                f.setCoreLevel(level + 1);
                                f.setMoney(f.getMoney() - (CoreFaction.coreCost * (level + 1)));
                                Utils.sendMessage(playerIn, Utils.getTranslation("faction.upgrade", TextFormatting.GREEN));
                                FactionUtils.saveFaction(f);
                            } else
                                Utils.sendMessage(playerIn, Utils.getTranslation("faction.upgrade.nomoney", TextFormatting.GREEN));
                        } else
                            Utils.sendMessage(playerIn, Utils.getTranslation("faction.upgrade.nopermission", TextFormatting.RED));
                    } else
                        Utils.sendMessage(playerIn, Utils.getTranslation("faction.upgrade.noowner", TextFormatting.RED));
                } else {
                    Utils.sendMessage(playerIn, Utils.getTranslation("faction.upgrade.noowner", TextFormatting.RED));
                }
            
        }
        
        return super.onBlockActivated(worldIn, pos, state, playerIn, hand, heldItem, side, hitX, hitY, hitZ);
    }

What should i check to make sure it runs only once?

What happens if you change

return super.onBlockActivated(worldIn, pos, state, playerIn, hand, heldItem, side, hitX, hitY, hitZ);

to

return true;

?

Thank you, changing that to true solved the problem

2 hours ago, Draco18s said:

Once for each hand