Posted June 13, 20178 yr 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 Don't blame me if i always ask for your help. I just want to learn to be better
June 13, 20178 yr It's called exactly once. Once for each hand Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable. If you think this is the case, JUST REPORT ME. Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice. Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked. DO NOT PM ME WITH PROBLEMS. No help will be given.
June 13, 20178 yr Author 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? Don't blame me if i always ask for your help. I just want to learn to be better
June 13, 20178 yr What happens if you change return super.onBlockActivated(worldIn, pos, state, playerIn, hand, heldItem, side, hitX, hitY, hitZ); to return true; ?
June 13, 20178 yr Author Thank you, changing that to true solved the problem Don't blame me if i always ask for your help. I just want to learn to be better
June 14, 20178 yr 2 hours ago, Draco18s said: Once for each hand Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable. If you think this is the case, JUST REPORT ME. Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice. Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked. DO NOT PM ME WITH PROBLEMS. No help will be given.
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.