Triphion

I SOLVED IT! Thank you so much diesieben! ❤️ This is how i did it:

EDIT: I realized that there is actually the setLightningIndex function that gets the server to crash. And this is the error i get everytime: And this is how my setLightningFunction looks right now:

I do in-fact know how static works, this was actually me being frustrated and just trying everything(which is never a good thing). Static is a declaration that has the same value for the class, including all individual objects that stem from the class has the same value. I have now removed those. I realized now aswell, but i don't know how to get that specific itemstack and then use the method since i cannot cast directly to an itemstack. This is what i have for now, and it is not valid/or working method, just starting the method. if (player.getHeldItem(EnumHand.MAIN_HAND).getItem() == ModItems.ZEUS_LIGHTNING_CHARGED) { System.out.println("Succesfully retrieved Item."); ItemStack weapon = player.getHeldItem(EnumHand.MAIN_HAND); ((ItemZeusLightningCharged)weapon).setLightningIndex(weapon, + 1); } How do i cast directly to the itemstack and not the item class as a whole? Because i can get the itemstack, but not cast it directly to the itemstack and change that specific itemstack. [16:42:22] [Netty Server IO #1/INFO] [STDOUT]: [com.etauricstaff.etauricmod.network.MessageGetPlayer:handleServerSide:36]: Succesfully retrieved Item. [16:42:22] [Netty Server IO #1/INFO] [STDOUT]: [com.etauricstaff.etauricmod.network.MessageGetPlayer:handleServerSide:39]: Itemstack: 1xitem.zeus_lightning_charged@0

Thank you for the advice. I have tried to make something like that. However, the problem is that the player doesn't register the active itemstack, and instead has it as simple air. And i don't really know how to fix that. ((ItemZeusLightningCharged)player.getHeldItemMainhand().getItem()).setLightningIndex(player.getActiveItemStack(), +1); This is the function that i'm calling when the player presses a button to change the firing mode. After that, the lightning index is going up, and then the item property is supposed to override and give the item a new texture. public static int getLightningIndex(ItemStack item) { NBTTagCompound nbtTagCompound = item.getTagCompound(); if(nbtTagCompound != null) return nbtTagCompound.getInteger("lIndex"); return 0; } public static void setLightningIndex(ItemStack item, int lightningValue) { NBTTagCompound nbtTagCompound = item.getTagCompound(); if (lightningValue < 2) { } else if (lightningValue == 2) { lightningValue = 0; } nbtTagCompound.setInteger("lIndex", lightningValue); ItemZeusLightningCharged.lightningIndex = lightningValue; Utils.getLogger().info(lightningValue); } This is the methods i'm using to get the index number. public static int lightningIndex; This is the int/float i'm using to check if the item should override.

So the problem is very simple, i want the item i'm holding to change, and not the other ones of the same item in my inventory. Here is the class with the method. Any ideas? EntityLightningBolt lightning; private int lightning_index; public ItemZeusLightningCharged(String unlocalizedName) { super(unlocalizedName); this.setCreativeTab(CreativeTabs.COMBAT); this.lightning_index = 0; this.addPropertyOverride(new ResourceLocation(Reference.MODID, "zeus_mode"), new IItemPropertyGetter() { @Override public float apply(ItemStack stack, World worldIn, EntityLivingBase entityIn) { Utils.getLogger().info("Itemstack: " + stack); if (entityIn != null && entityIn.getHeldItem(EnumHand.MAIN_HAND).getItem() == ModItems.ZEUS_LIGHTNING_CHARGED) { Utils.getLogger().info("Player has Item."); return (float)((ItemZeusLightningCharged)entityIn.getHeldItem(EnumHand.MAIN_HAND).getItem()).lightning_index; } else { return 0.0F; } } }); } To clarify, the lightning index is an int that resembles the different modes. Edit: This doesn't actually work serverside either.

To start off. The item in question is working, the problem lies in my eventhandler that is responsible for switching the firing modes when a button is pressed. private static boolean first_keypress = true; private static EntityLivingBase player; private static ItemZeusLightningCharged z_lightning; These are the variables that i'm using. The first_keypress is used to keep the key triggered once, and then you can trigger it once more after you release it. The player is to check if the entity is a player. The lightning is used as the weapon reference. @SubscribeEvent(priority = EventPriority.NORMAL, receiveCanceled = true) public static void onItemCheck(LivingUpdateEvent event) { if (event.getEntityLiving() instanceof EntityPlayerMP) { player = (EntityPlayerMP) event.getEntityLiving(); if (event.getEntity() == player && player.getHeldItemMainhand().getItem() == ModItems.ZEUS_LIGHTNING_CHARGED) { z_lightning = (ItemZeusLightningCharged) player.getHeldItemMainhand().getItem(); } else { z_lightning = null; } } } This is the method that checks if the entity is an instance of entityplayer, and if it is, it changes the player variable to the entity. And under that we have a check to see if the player is holding the lightning weapon. And if it is, it changes the weapon variable to the weapon the player is holding. And if it isn't holding the weapon, the weapon reference is null. @SubscribeEvent(priority=EventPriority.NORMAL, receiveCanceled=true) public static void onEvent(KeyInputEvent event) { KeyBinding[] keyBindings = ClientProxy.keyBindings; if (keyBindings[0].isPressed()) { if (first_keypress) { first_keypress = false; Utils.getLogger().info("Trying to check Item."); checkItem(); } } else if (!keyBindings[0].isPressed()) { first_keypress = true; } } private static void checkItem() { if (z_lightning != null) { Utils.getLogger().info("Checked Item."); z_lightning.lightningIndexController(); } } Next we have the keycheck. If the player presses the designated keybinding and it's the first keypress, then it calls checkItem();. It then changes the first_keypress to false and when you release the button, it changes to true. I am probably going to change the boolean to something as a timer instead if i have more of these events. Anyway, the checkItem(); checks if the weapon reference isn't null, and if it isn't null, it then gets the weapon reference and calls indexController(); on the weapon reference. public void lightningIndexController() { if (this.lightning_index < 2) { this.lightning_index += 1; } if (this.lightning_index == 2) { this.lightning_index = 0; } Utils.getLogger().info("Lightning Index: " + this.lightning_index); } The index controller adds a number to the index if it is lower than the maximum number, and resets to 0 if it goes to the maximum number. Now, the problem is that this doesn't work on servers, it does however work on singleplayer worlds. I have done several checks, and the problem seems to be that the weapon reference doesn't get stored, it only changes to the current weapon inside the weaponcheck. After the weaponcheck inside the updateevent, it just goes back to null. I don't know why this is the case on servers, but not on singleplayer worlds. Would appreciate some pointers here. ^^ EDIT: SOLVED IT! I created and used packets to fix it.

for (Entity entity : this.friendlyList) { if (entity.isDead) { this.friendlyList.remove(entity); } } Noticed aswell that if an undead dies, it crashes, i tried making this method to fix it, but to no avail. Edit: fixed it. Removed the method in its entirety and it somehow solved the problem ¯\_(ツ)_/¯ But i still need help regarding selecting target that is closest to the entity. @Override public void setAttackTarget(EntityLivingBase entitylivingbaseIn) { if (this.canTarget) { this.enemyList.removeAll(this.friendlyList); Entity enemy = (Entity) this.enemyList.get(0); double d0 = this.getDistanceSqToEntity((Entity) enemy); EntityLivingBase entityToAttack = (EntityLivingBase) enemy; if (d0 < 30.0D && d0 > 0) { entitylivingbaseIn = (EntityLivingBase) enemy; super.setAttackTarget(entitylivingbaseIn); } else { super.setAttackTarget(null); } } } I've started with this, but it's nowhere near done. So i need a few pointers to get the target i want. Edit: I fixed it! Thanks for all your help guys! ^^ @Override public void setAttackTarget(EntityLivingBase entitylivingbaseIn) { if (this.canTarget) { for (Entity entity : this.enemyList) { double d0 = this.getDistanceSqToEntity(entity); if (d0 < 40.0D) { super.setAttackTarget((EntityLivingBase) entity); } } } else { super.setAttackTarget(null); } } @Override protected void updateAITasks() { this.targetList = this.world.getEntitiesWithinAABBExcludingEntity(this, new AxisAlignedBB((double)this.posX - 17.5D, (double)this.posY - 17.5D, (double)this.posZ - 17.5D, (double)this.posX + 17.5D, (double)this.posY + 17.5D, (double)this.posZ + 17.5D)); if (this.getAttackTarget() == null && this.isAlly()) { for (Entity entity : this.targetList) { if (entity instanceof EntityUndead && ((EntityUndead) entity).isAlly() && !this.friendlyList.contains(entity)) { this.friendlyList.add(entity); } if (entity instanceof EntityPlayerMP && !this.friendlyList.contains(entity)) { this.friendlyList.add(entity); } } } else if (!(this.getAttackTarget() == null)) { this.canTarget = false; if (this.friendlyList.contains(this.getAttackTarget())) { this.setAttackTarget(null); } } this.targetList.removeAll(this.friendlyList); this.enemyList.clear(); this.enemyList.addAll(this.targetList); if (!this.enemyList.isEmpty()) { this.canTarget = true; } else if (this.enemyList.isEmpty()) { this.canTarget = false; } }

Tried a much higher limit aswell as no limit. But it still crashes. Any other suggestion on why it crashes? It gets called too often in updateAITasks and i should use some other update method? Undead class: https://github.com/triphion/Ancient-Mod/blob/master/src/main/java/ancient/entity/EntityUndead

if (this.getAttackTarget() == null && this.isAlly()) { for (Entity entity : this.targetList) { if (entity instanceof EntityUndead && ((EntityUndead) entity).isAlly() && !friendlyList.contains(entity)) { friendlyList.add(entity); } if (entity instanceof EntityPlayerMP && !friendlyList.contains(entity)) { friendlyList.add(entity); } } } Did it and it works, now however i need to check for all entities in the enemy list for the one that is closest to the entity. Any suggestions? Edit: I realized that the game crashes when there's too many loyal undeads. Not sure where in the code it gets overexerted.

What i'm trying to achieve is a filtering system where my loyal undeads will select its next target based on a list that will get updated in updateAITasks. I think i have almost got it, but i'm still running into a few problems, such as the entity selecting a few targets that is not supposed to be targeted. To make sure this does what it's supposed to, i made a targetList which creates an aabb around the entity and then checks if there are any allies in that aabb, if there are, it adds the allies in the aabb. Or atleast is supposed to, but my problem is that i don't know how to reference all the friendly ones and then add them all at the same time. My entity class = https://github.com/triphion/Ancient-Mod/blob/master/src/main/java/ancient/entity/EntityUndead Worth noting is the way i'm adding and filtering. So first of i start with checking if the current target is null and if the targetList has any loyal undeads or players in it. If it has, then it's supposed to add them all. Then i remove them from the targetList and then add all of the remaining entities in the enemyList. Then every time this method gets called, so as to not overexert anything, i remove all the entities in the enemyList and then the process continue.

Worked for me. A massive thanks to everyone who helped me! :D

I do have a logger function, but only one that gets called preinit. If i want the logger function to get called, this way of doing it does work when im on singleplayer, however in server window, the log function never gets mentioned? @Override public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand handIn) { if (!worldIn.isRemote) { EntityLivingBase player = ((EntityLivingBase)playerIn); RayTraceResult pos = player.rayTrace(22.0D, 20.0F); double x = pos.getBlockPos().getX(); double y = pos.getBlockPos().getY() + 1.0D; double z = pos.getBlockPos().getZ(); Utils.getLogger().info("Raytrace: " + pos.toString().substring(5)); if (player.attemptTeleport(x, y, z)) { player.getHeldItem(handIn).damageItem(1, playerIn); } } else{ return new ActionResult(EnumActionResult.PASS, playerIn.getHeldItem(handIn)); } return new ActionResult(EnumActionResult.SUCCESS, playerIn.getHeldItem(handIn)); }

Found it. java.util.concurrent.ExecutionException: java.lang.NoSuchMethodError: net.minecraft.entity.EntityLivingBase.rayTrace(DF)Lnet/minecraft/util/math/RayTraceResult; at java.util.concurrent.FutureTask.report(FutureTask.java:122) ~[?:1.8.0_151] at java.util.concurrent.FutureTask.get(FutureTask.java:192) ~[?:1.8.0_151] at net.minecraft.util.Util.runTask(Util.java:30) [Util.class:?] at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:754) [MinecraftServer.class:?] at net.minecraft.server.dedicated.DedicatedServer.updateTimeLightAndEntities(DedicatedServer.java:402) [DedicatedServer.class:?] at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:699) [MinecraftServer.class:?] at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:548) [MinecraftServer.class:?] at java.lang.Thread.run(Thread.java:748) [?:1.8.0_151] Caused by: java.lang.NoSuchMethodError: net.minecraft.entity.EntityLivingBase.rayTrace(DF)Lnet/minecraft/util/math/RayTraceResult; at com.triphion.ancient.items.ItemTheCosmician.onItemRightClick(ItemTheCosmician.java:45) ~[ItemTheCosmician.class:?] at net.minecraft.item.ItemStack.useItemRightClick(ItemStack.java:213) ~[ItemStack.class:?] at net.minecraft.server.management.PlayerInteractionManager.processRightClick(PlayerInteractionManager.java:387) ~[PlayerInteractionManager.class:?] at net.minecraft.network.NetHandlerPlayServer.processTryUseItem(NetHandlerPlayServer.java:739) ~[NetHandlerPlayServer.class:?] at net.minecraft.network.play.client.CPacketPlayerTryUseItem.processPacket(CPacketPlayerTryUseItem.java:43) ~[CPacketPlayerTryUseItem.class:?] at net.minecraft.network.play.client.CPacketPlayerTryUseItem.processPacket(CPacketPlayerTryUseItem.java:9) ~[CPacketPlayerTryUseItem.class:?] at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:21) ~[PacketThreadUtil$1.class:?] at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) ~[?:1.8.0_151] at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_151] at net.minecraft.util.Util.runTask(Util.java:29) ~[Util.class:?] ... 5 more [14:11:24] [Server thread/FATAL]: Error executing task This is the error and i have no idea how to fix it. The line that is the problem is the raytracing line that i talked about but i don't know more than this. This is line 45: RayTraceResult pos = player.rayTrace(22.0D, 20.0F); Any ideas?

Yes, that is the one. But now i realize the if(!worldIn.isRemote) but the raytracing isn't working serverside. Anything i'm missing? Also, i added added a cast to the player saying it's an entitylivingbase, although the player class does extend entitylivingbase but i thought it could help just being even more specific.

So i tried with console-printing and when im on the server it doesn't get called after the if(!worldIn.isRemote) check for some reason. @Override public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand handIn) { scala.Console.print("Gets called. "); if (!worldIn.isRemote) { scala.Console.print("worki. "); RayTraceResult pos = playerIn.rayTrace(22.0D, 0.0F); double x = pos.getBlockPos().getX(); double y = pos.getBlockPos().getY() + 1.0D; double z = pos.getBlockPos().getZ(); if (playerIn.attemptTeleport(x, y, z)) { scala.Console.print("Teleported. "); playerIn.getHeldItem(handIn).damageItem(1, playerIn); } } else{ return new ActionResult(EnumActionResult.FAIL, playerIn.getHeldItem(handIn)); } return new ActionResult(EnumActionResult.SUCCESS, playerIn.getHeldItem(handIn)); } Anything that seems to doesn't work on server here? EDIT: I got a message from the server window that there's a problem with my raytracing apparently? Doesn't that get called correctly on server side, am i missing some important raytracing check for serverside?