Please read the rules and FAQ before starting a new topic

Currently supported: All versions

[1.9] you can't replace item from the inside code methods

Posted March 23, 20169 yr

good nights

this time this post is not an accident

actually i think i found a bug

the trouble is i can't replace the selected itemStack for another using the methods from the item class like

onItemUseFinish() or onPlayerStoppedUsing() as i do in 1.8

this is the post with original question i made in the forums

http://www.minecraftforge.net/forum/index.php/topic,37521.msg197729.html#msg197729

lets say i have this ItemStackIn is in the slot 1 from the hotbar if in the item code i do something like

@Override
public void onPlayerStoppedUsing(ItemStack stack, World worldIn, EntityLivingBase entityLiving, int timeLeft) {
if (!worldIn.isRemote){
if (entityLiving instanceof EntityPlayer) {
EntityPlayer playerIn = (EntityPlayer) entityLiving;
playerIn.inventory.setInventorySlotContents( 1, replaceItemStack);
}
}

the change is not happening and i remain with ItemStackIn in the slot 1

but if i set the item in any other slot

playerIn.inventory.setInventorySlotContents( 2, replaceItemStack);

i end having ItemStackIn in the slot 1 and replaceItemStack in slot 2

the only slot i couldn't replace is the one containing the item on use

could you fix this

Thank for reading

[/code]

Quote

March 24, 20169 yr

Are you in creative mode?

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Quote

March 27, 20169 yr

Author

Nup

but i test the creative surviver and mode 2 still fail in all

im using

	
// ############################################################################################3
@Override
public ItemStack onItemUseFinish(ItemStack stack, World worldIn, EntityLivingBase entityLiving) {

	if (!worldIn.isRemote) {

		if (entityLiving instanceof EntityPlayer) {

			ItemStack replaceItemStack = new ItemStack(MMMateriales.materialesSuspensionderedstoneActivada, 1, 0);
			Item comparationItem = MMMateriales.materialesSuspensionderedstone;

			EntityPlayer playerIn = (EntityPlayer) entityLiving;
			EnumHand ManoActiva = playerIn.getActiveHand();

			ItemStack leftHand = playerIn.getHeldItem(EnumHand.OFF_HAND);
			ItemStack rightHand = playerIn.getHeldItem(EnumHand.MAIN_HAND);

			int hand = playerIn.inventory.currentItem;
			ItemStack itemstack = null;

			itemstack = playerIn.inventory.getStackInSlot(hand);
			if (ManoActiva != null && ManoActiva == EnumHand.MAIN_HAND && itemstack.getItem() == comparationItem) {

				chat.chatm(playerIn, "replacing the Item in MAIN_HAND");
				playerIn.inventory.setInventorySlotContents(hand, replaceItemStack);
				//playerIn.inventory.markDirty();
			}

			itemstack = playerIn.inventory.getStackInSlot(40);
			if (ManoActiva != null && ManoActiva == EnumHand.OFF_HAND && itemstack.getItem() == comparationItem) {
				chat.chatm(playerIn, "replacing the Item in OFF_HAND");


				playerIn.inventory.setInventorySlotContents(40, replaceItemStack);
				//playerIn.inventory.decrStackSize(40, 1);
				//playerIn.inventory.markDirty();
				//playerIn.inventory.inventoryChanged = true;
			}

			int playerInvZise = playerIn.inventory.getSizeInventory();

			for (int i = 0; i < playerInvZise; ++i) {
				itemstack = playerIn.inventory.getStackInSlot(i);

				if (itemstack != null) {
					System.out.println("SLOT[" + i + "]=" + itemstack.getUnlocalizedName());
				}
			}

		}
	}

	return stack;
}

the funny part is that i only cant'not change the selected item but i can change hatever else i want so if i made

//MainHand
			itemstack = playerIn.inventory.getStackInSlot(hand);
			if (ManoActiva != null && ManoActiva == EnumHand.MAIN_HAND && itemstack.getItem() == comparationItem) {

				chat.chatm(playerIn, "replacing the Item in MAIN_HAND");
				playerIn.inventory.setInventorySlotContents(40, replaceItemStack);
				//playerIn.inventory.markDirty();
			}

being the selected item in the right hand i can change the item in the off hand

and also i can consume the item in the main hand even if its the selected using

playerIn.inventory.decrStackSize(hand, 1);

but is not allow to change the item in the main hand for any other if this is the selected itemStack

Quote

March 28, 20169 yr

Author

well is more like it changes to activates but inmediately comes back to normal

alsi if make the change here works


@Override
public boolean onEntitySwing(EntityLivingBase entityLiving, ItemStack itemStackIn) {

	World worldIn = entityLiving.worldObj;

	if (entityLiving instanceof EntityPlayer) {
		EntityPlayer playerIn = (EntityPlayer) entityLiving;

	ItemStack replaceItemStack = new ItemStack(MMMateriales.materialesSuspensionderedstoneActivada, 1, 0);

	int handi = playerIn.inventory.currentItem;
	chat.chatm(playerIn, "replacing the Item in MAIN_HAND");
	playerIn.inventory.setInventorySlotContents(handi, replaceItemStack);

	}

	return true;
}

but if i made it here onItemRightClick like this


@Override
public ActionResult<ItemStack> onItemRightClick(ItemStack itemStackIn, World worldIn, EntityPlayer playerIn,
		EnumHand hand) {
	boolean sePudo = true;


	ItemStack replaceItemStack = new ItemStack(MMMateriales.materialesSuspensionderedstoneActivada, 1, 0);

	int handi = playerIn.inventory.currentItem;
	chat.chatm(playerIn, "replacing the Item in MAIN_HAND");
	playerIn.inventory.setInventorySlotContents(handi, replaceItemStack);



	return sePudo ? new ActionResult(EnumActionResult.PASS, itemStackIn) : new ActionResult(EnumActionResult.FAIL, itemStackIn);
}

is curious coze its visually changing for a brief moment but comes back to normal

Quote

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.

Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.

Reply to this topic...

https://forums.minecraftforge.net/topic/37406-19-you-cant-replace-item-from-the-inside-code-methods/

Followers

Go to topic listing

IPS spam blocked by CleanTalk.

Announcements

Guys, please help me get through this horrible NoClassDefFound exception.

Oh yeah, and one more thing I forgot to mention. My server runs on Java SE 8 from Graal VM Enterprise Edition. Not exactly sure if this will come in handy.

By Jackson Perr · 40 minutes ago40 min
Guys, please help me get through this horrible NoClassDefFound exception.

Hey fellas!

So I've been working on a mod for my Minecraft server that adds "admin tools" ( that's code for "trolling commands" ( : )

It's been pretty smooth so far: I've heard them scream as they were chased by every enemy in the entire world, I've watched as they mysteriously turned into potatoes, and I've even gotten to hear them say strange things like "Why th are my pants on my head?"

However, I've lately started running into an issue that I can't quite seem to get through... Like at all! It started when I thought it would be a really cool idea to implement a command that makes EVERYTHING invisible for a selected player. You know, the usual everyday kinda' tool needed for proper server administration. But I can't just set all the entities to be invisible with the setInvisible(boolean value) kind of deal, because that would mean that if you had either a hacked client or the entity had some kind of equipment, then you could still see it. I wanted it so that the client that I chose didn't even know the entity existed. So, I initially started by just trying to intercept packets. However, this only worked for vanilla entities, like zombies and basic items. The reason? Because forge is really annoying and decides that it's not going to spawn mod entities through the default channel and is instead going to use an entirely different channel, one that I can't conveniently connect a player entity to through a CustomPacketHandler because this channel remains static throughout the server's lifespan. So I though, jeez, well why don't I just add another CustomPacketHandler to this channel and see how it goes. So I did my first test and tried to prevent an entity spawn packet from being written to the channel. And can you guess what happened? Probably not, because it wasn't a NoClassDefFound exception. Instead, it was a NullReferenceException that lead to an error that said something like, "Don't know how to spawn in this entity." As it turns out, when the packet for spawning an entity is being created, it relies on trying to write the packet to this fml channel and then for some reason it just reads the packet from the fml channel before trying to send it again. So that means if you prevent it from being written then there's nothing that can be read from the channel, so the spawn packet ends up being null. So I though, damn, this is really stupid. So I searched around a bit and found a function called isVisibleTo, which is basically just called to check whether a player can or can't see an entity before it decides to make the packet to send to the player. Then I got an idea: what if I just hijack this function and add some code at the start to make it return false if the player has had this "invisible enemy" command ran on them. So I looked around a little more, and finally, I found out that if you make your mod a plugin, then you can access the transform method of the IClassTransformer interface. Basically, it's like the harmony transpiler patch, but with java! So I got to work.

Now, understand that everything before this point it basically just pretext to give you fellas some good context. From this point on, this is where my struggles are going to start to begin. I started off with writing my hijacking function. It was a static public function with one argument of the type EntityPlayerMP that returned a boolean value. I named this function "canPlayerSee". Now, on the AsmTranspiler end, I had gotten the class/method descriptors and function names from the obfuscated mappings and pulled apart the isVisibleTo function's disassembly. From here, I put together some quick code that injected at the start of the isVisibleTo function the following java assembly codes:

aload_1 //This loads the first, and only argument (excluding the this argument) of the isVisibleTo function
invokestatic //canPlayerSee
ifne //label_0
iconst_0 //This should load the 0 integer
ireturn //This should return the number at the top of the stack, zero in this case
label_0

So I loaded up the server, and nothing bad happened. I run the command on myself to make everything invisible and it works perfectly! So at this point, I'm thinking that this would be a pretty powerful tool if I could load an instance of the Entity that the isVisibleTo function concerns onto the stack and pass is as an argument to the canPlayerSee function for better use. This would let me implement a propper vanish command and a command that would let me turn players of my choice into a creeper that the player in question cannot see. So, I modified the assembly code again. I changed it to this instead: (Note, I did update the canPlayerSee function to accept the new Entity arg at the end)

aload_1 //This loads the first, and only argument (excluding the this argument) of the isVisibleTo function
aload_0 //Loads the this arg to the stack
getfield //EntityTrackerEntry.trackedEntity
invokestatic //canPlayerSee
ifne //label_0
iconst_0 //This should load the 0 integer
ireturn //This should return the number at the top of the stack, zero in this case
label_0

So I launched up the server again... And it crashes: NoClassDefFoundException. This is the error, I am currently getting, and I have tried almost everything I could think of to fix it. Hell, I even tried praying, which almost always works! None of those things I tried are of particular note here, but what is of particular note is the fact that it seems that the crashing will happen only when I make the canPlayerSee function two arguments and change the code to reflect loading the proper arguments. Now, I'm going to send some source code, so just be ready: my chunking skills are horrible and my coding "elegancy" might make you really want to puke. For that, I am sorry. Also, since it's like 3:00 AM rn, I'm just going to try and send images instead of providing text. I know this will make it a bit harder to help, so I am sorry for that too, but I'm really tired.

Nvm, I guess I'm sending text:

This is my entire class for modifying the asm code:
public class AsmTransformer implements IClassTransformer { @SuppressWarnings("deprecation") private static Logger logger = FMLLog.getLogger(); public byte[] transform(String name, String transformedName, byte[] bytes) { if (transformedName.startsWith("net.minecraft.entity.EntityTrackerEntry")) { ClassReader reader = new ClassReader(bytes); ClassNode node = new ClassNode(); reader.accept((ClassVisitor)node, 0); if (analyzeClass(node)) { ClassWriter writer = new ClassWriter(3); node.accept(writer); bytes = writer.toByteArray(); } } return bytes; } private boolean analyzeClass(ClassNode node) { boolean rval = false; for (MethodNode method : node.methods) { logger.info(method.name); logger.info(method.desc); if (method.name.equals("isVisibleTo") || (method.name.equals("c") && method.desc.equals("(Loq;)Z"))) { try { rval = hijackCanSeeFunc(method); } catch (NoSuchMethodException e) { e.printStackTrace(); } catch (SecurityException e) { e.printStackTrace(); } } } if(!rval) logger.info("COULD NOT FIND METHOD!"); return rval; } private boolean hijackCanSeeFunc(MethodNode node) throws NoSuchMethodException, SecurityException { InsnList insnList = node.instructions; logger.info("Hackin' it..."); Method meth = CustomHandler.class.getMethod("canPlayerSee", Entity.class, EntityPlayerMP.class); String methDesc = Type.getMethodDescriptor(meth); logger.info(methDesc); InsnList patch = new InsnList(); LabelNode ln = new LabelNode(new Label()); patch.add((AbstractInsnNode) new VarInsnNode(Opcodes.ALOAD, 0)); patch.add((AbstractInsnNode) new FieldInsnNode(Opcodes.GETFIELD, Type.getInternalName(EntityTrackerEntry.class), "d", "Lvg;")); patch.add((AbstractInsnNode) new VarInsnNode(Opcodes.ALOAD, 1)); patch.add((AbstractInsnNode) new MethodInsnNode(Opcodes.INVOKESTATIC, Type.getInternalName(CustomHandler.class), "canPlayerSee", methDesc, false)); patch.add((AbstractInsnNode) new JumpInsnNode(Opcodes.IFNE, ln)); patch.add((AbstractInsnNode) new InsnNode(Opcodes.ICONST_0)); patch.add((AbstractInsnNode) new InsnNode(Opcodes.IRETURN)); patch.add((AbstractInsnNode) ln); insnList.insertBefore(insnList.getFirst(), patch); return true; } }Of particular note here is that last hijack function. The second to last chunk contains the code that inserts my own instructions.

This is the custom function I am injecting at the start of the isVisibleTo function:
public static boolean canPlayerSee(Entity instance, EntityPlayerMP target) { if (playerInvisTrollList.contains(target.getName())) { return false; } else if (instance instanceof EntityPlayerMP && playerVanishList.get(((EntityPlayerMP) instance).getName()) != null) { return false; } else if (findListElem(Common.creepers, index -> index.creep != null && index.creep.getEntityId() == instance.getEntityId()) != null && findListElem(Common.creepers, index -> index.name.contentEquals(target.getName())) != null) { return false; } return true; }I know, it's not chunked like at all. My bad. Also, the findListElem function is basically just the removeIf function for lists, except for instead of removing the element when it matches the predicate, it will just return it instead. It's weird how lists don't already have this function (as far as I know).

Here is the stacktrace:
java.lang.NoClassDefFoundError: net/minecraft/entity/EntityTrackerEntry at net.minecraft.entity.EntityTracker.func_72785_a(EntityTracker.java:252) ~[ol.class:?] at net.minecraft.entity.EntityTracker.func_72786_a(EntityTracker.java:157) ~[ol.class:?] at net.minecraft.world.ServerWorldEventHandler.func_72703_a(ServerWorldEventHandler.java:37) ~[op.class:?] at net.minecraft.world.World.func_72923_a(World.java:1222) ~[amu.class:?] at net.minecraft.world.WorldServer.func_72923_a(WorldServer.java:1112) ~[oo.class:?] at net.minecraft.world.WorldServer.func_175650_b(WorldServer.java:1068) ~[oo.class:?] at net.minecraft.world.chunk.Chunk.func_76631_c(Chunk.java:860) ~[axw.class:?] at net.minecraftforge.common.chunkio.ChunkIOProvider.syncCallback(ChunkIOProvider.java:109) ~[ChunkIOProvider.class:?] at net.minecraftforge.common.chunkio.ChunkIOExecutor.syncChunkLoad(ChunkIOExecutor.java:94) ~[ChunkIOExecutor.class:?] at net.minecraft.world.gen.ChunkProviderServer.loadChunk(ChunkProviderServer.java:118) ~[on.class:?] at net.minecraft.world.gen.ChunkProviderServer.func_186028_c(ChunkProviderServer.java:89) ~[on.class:?] at net.minecraft.world.gen.ChunkProviderServer.func_186025_d(ChunkProviderServer.java:135) ~[on.class:?] at net.minecraft.server.MinecraftServer.func_71222_d(MinecraftServer.java:344) ~[MinecraftServer.class:?] at net.minecraft.server.MinecraftServer.func_71247_a(MinecraftServer.java:314) ~[MinecraftServer.class:?] at net.minecraft.server.dedicated.DedicatedServer.func_71197_b(DedicatedServer.java:270) ~[nz.class:?] at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:486) [MinecraftServer.class:?] at java.lang.Thread.run(Thread.java:750) [?:1.8.0_451]Not much I gotta' say about this.

Here, I even got the disassembled code for the isVisibleTo function in case it helps:
public boolean isVisibleTo(net.minecraft.entity.player.EntityPlayerMP); descriptor: (Lnet/minecraft/entity/player/EntityPlayerMP;)Z flags: ACC_PUBLIC Code: stack=6, locals=7, args_size=2 0: aload_1 1: getfield #148 // Field net/minecraft/entity/player/EntityPlayerMP.posX:D 4: aload_0 5: getfield #13 // Field encodedPosX:J 8: l2d 9: ldc2_w #149 // double 4096.0d 12: ddiv 13: dsub 14: dstore_2 15: aload_1 16: getfield #151 // Field net/minecraft/entity/player/EntityPlayerMP.posZ:D 19: aload_0 20: getfield #17 // Field encodedPosZ:J 23: l2d 24: ldc2_w #149 // double 4096.0d 27: ddiv 28: dsub 29: dstore 4 31: aload_0 32: getfield #7 // Field range:I 35: aload_0 36: getfield #8 // Field maxRange:I 39: invokestatic #152 // Method java/lang/Math.min:(II)I 42: istore 6 44: dload_2 45: iload 6 47: ineg 48: i2d 49: dcmpl 50: iflt 95 53: dload_2 54: iload 6 56: i2d 57: dcmpg 58: ifgt 95 61: dload 4 63: iload 6 65: ineg 66: i2d 67: dcmpl 68: iflt 95 71: dload 4 73: iload 6 75: i2d 76: dcmpg 77: ifgt 95 80: aload_0 81: getfield #6 // Field trackedEntity:Lnet/minecraft/entity/Entity; 84: aload_1 85: invokevirtual #153 // Method net/minecraft/entity/Entity.isSpectatedByPlayer:(Lnet/minecraft/entity/player/EntityPlayerMP;)Z 88: ifeq 95 91: iconst_1 92: goto 96 95: iconst_0 96: ireturn LineNumberTable: line 475: 0 line 476: 15 line 477: 31 line 478: 44 LocalVariableTable: Start Length Slot Name Signature 0 97 0 this Lnet/minecraft/entity/EntityTrackerEntry; 0 97 1 playerMP Lnet/minecraft/entity/player/EntityPlayerMP; 15 82 2 d0 D 31 66 4 d1 D 44 53 6 i I StackMapTable: number_of_entries = 2 frame_type = 254 /* append */ offset_delta = 95 locals = [ double, double, int ] frame_type = 64 /* same_locals_1_stack_item */ stack = [ int ]

This is pretty much everything I can think to provide. I'm not sure what else I can do for you fellas rn, but I hope this can get somebody somewhere. Even if you aren't confident you know exactly what's going on, if you think you have some kind of idea, please don't be shy about replyin'. I would appreciate ANY amount of help here with this, no matter how menial or small it is. Thank you.

By Jackson Perr · Posted 49 minutes ago49 min
Failure to Launch with 1.21.10-60.0.6 Linux server, Java 21.

Hey, sorry about this. There was an oversight with my fix so I've made a pull request to have it resolved. When LexManos or Paint_Ninja are available, it'll be merged. For now, please continue to use 60.0.5. Thanks for letting me know of the issue again. 🙏

By Jonathing · 8 hours ago8 hr
Failure to Launch with 1.21.10-60.0.6 Linux server, Java 21.

OK, got a similar warning as above with .5 but FallingTree is working fine. I'm not sure if it's a FallingTree or Forge issue.

By Richy_T · 9 hours ago9 hr
Failure to Launch with 1.21.10-60.0.6 Linux server, Java 21.

Thanks. May still be issues.

[19:51:42] [main/WARN] [ne.mi.co.ForgeHooks/WP]: The following mods have version differences that were not resolved:
forge (version 59.0.1 -> 1.21.10-60.0.7)
Things may not work well.

Also had trouble running FallingTree mod but currently verifying that's not a me thing.

Yep, not a me thing.

[19:54:52] [main/ERROR] [ne.mi.fm.lo.ModSorter/LOADING]: Missing or unsupported
mandatory dependencies: Mod ID: 'forge', Requested by: 'fallingtree', Expected range: '[60,)', A
ctual version: '1.21.10-60.0.7'

Going to see how .5 does.

By Richy_T · 9 hours ago9 hr

Choonster

Sign In

[1.9] you can't replace item from the inside code methods

Featured Replies

Join the conversation

Posts

Guys, please help me get through this horrible NoClassDefFound exception.

Guys, please help me get through this horrible NoClassDefFound exception.

Failure to Launch with 1.21.10-60.0.6 Linux server, Java 21.

Failure to Launch with 1.21.10-60.0.6 Linux server, Java 21.

Failure to Launch with 1.21.10-60.0.6 Linux server, Java 21.

Who's Online (See full list)

Important Information

Account

Navigation

Search

Configure browser push notifications

Chrome (Android)

Chrome (Desktop)

Safari (iOS 16.4+)

Safari (macOS)

Edge (Android)

Edge (Desktop)

Firefox (Android)

Firefox (Desktop)