Infinite XP Problem about minecraftforge HOT 9 CLOSED

I think this is a buildcraft problem. Go report it there ;)

from minecraftforge.

This is indeed an issue - it wouldn't be so bad if we could ask the inventory if we are allowed to pull from, or place in an item. The only way round it right now is to special case furnaces and other mods output slots - which is not ideal.

from minecraftforge.

Agree. There should be a boolean for the "acess type".

from minecraftforge.

Is it time to import BC's ISpecialInventory?

from minecraftforge.

You shouldn't be able to put anything in the output slot, only in the two input slots. Probably something is messed up and instead of adding the item to the fuel slot it adds it to the output slot.

from minecraftforge.

ISidedInventory doesn't have a concept of a "remove-only" inventory. So if you can pipe out, you can pipe in. That's the thing about the exploit. Fixing that requires either extending ISidedInventory or using a new interface like ISpecialInventory.

from minecraftforge.

An other version would be adding a new Interface IRestrictiveInventory (or something like that) with a method:
boolean isItemValid(int slot, ItemStack stack)
This would allow a much more detailed control over item additions. The restrictive gui slots are already implemented that way (look at net.minecraft.src.Slot.isItemValid). If wanted, the slot class could also be patched to call the new method if available.

from minecraftforge.

I just made up some drafts for the IRestrictiveInventory class and the TileEntityFurnace:
https://dl.dropbox.com/u/63263434/Minecraft/IRestrictiveInventory.java
https://dl.dropbox.com/u/63263434/Minecraft/TileEntityFurnace.java.patch

from minecraftforge.

This is not a robust enough solution, I'll start riding Overmind about this he was working on a better solution.

from minecraftforge.