Jump to content

[1.13.2] IItemHandler and Capabilities


Saucier

Recommended Posts

I'm trying to setup my tile entity capabilities for automation, but I'm a bit stuck.

I have a tile with an inventory of 4 slots. Two slots (0, 1) for input items from the side and two slots (2, 3) for output items to the bottom. I'm using an ItemStackHandler for my inventory.

private final IItemHandler inv = new ItemStackHandler(4) {
	@Override
	protected void onContentsChanged(int slot) {
		super.onContentsChanged(slot);
		markDirty();
	}
};

 

How would I implement the ITEM_HANDLER_CAPABILITY here?

 

My problem:

- A hopper/pipe that is inserting items should only access slots 0,1 and a hopper/pipe that is extracting items should only access slots 2,3

 

If I'm returning my inv directly, it is returning all 4 slots.

Do I need to separate my ItemStackHandler into two inventories like invInput and invOutput or can I still use one ItemStackHandler and return only half of the inventory at the getCapability method?

Edited by Saucier
Link to comment
Share on other sites

Use a custom implementation of IItemHandler#insertItem and IItemHandler#extractItem. Both methods give you a slot as a parameter.

If you want extraction/insertion to be based on the side you can return a wrapper around your IItemHandler based on the side that only exposes the slots you want to expose for necessarry operation in getCapability. Note that you would still need to use a custom implementation of IItemHandler#insertItem and IItemHandler#extractItem.

  • Like 1
Link to comment
Share on other sites

6 hours ago, LTNightshade said:

Override TileEntity#hasCapability and TileEntity#getCapability

This stuff changed in 1.13.2

About Me

Spoiler

My Discord - Cadiboo#8887

My WebsiteCadiboo.github.io

My ModsCadiboo.github.io/projects

My TutorialsCadiboo.github.io/tutorials

Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support.

When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible.

Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org

Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)

Link to comment
Share on other sites

On 4/14/2019 at 4:28 AM, V0idWa1k3r said:

Use a custom implementation of IItemHandler#insertItem and IItemHandler#extractItem. Both methods give you a slot as a parameter.

If you want extraction/insertion to be based on the side you can return a wrapper around your IItemHandler based on the side that only exposes the slots you want to expose for necessarry operation in getCapability. Note that you would still need to use a custom implementation of IItemHandler#insertItem and IItemHandler#extractItem.

Thank you. Could you tell me the right way to return my overridden methods at getCapability?

That's what I have so far:

private final IItemHandler inv = new ItemStackHandler(4) {
	@Override
	protected void onContentsChanged(int slot) {
		super.onContentsChanged(slot);
		markDirty();
	}
	
	@Override
	public boolean isItemValid(int slot, @Nonnull ItemStack stack) {
		// My checks to see if an item is valid in the slot
	}
};

private final LazyOptional<IItemHandler> capItemHandler = LazyOptional.of(() -> createCapabilityHandler());

protected IItemHandler createCapabilityHandler() {
	return new CombinedInvWrapper(this.inv) {
		@Override
		public ItemStack insertItem(int slot, @Nonnull ItemStack stack, boolean simulate) {
			// Only insert, if stack is valid for the slot
			return (isItemValid(slot, stack)) ? super.insertItem(slot, stack, simulate) : stack;
		}
			
		@Override
		public ItemStack extractItem(int slot, int amount, boolean simulate) {
			// Only pull items from the output slots
			return (slot == 2 || slot == 3) ? super.extractItem(slot, amount, simulate) : ItemStack.EMPTY;
		}
	};
}

public <T> LazyOptional<T> getCapability(Capability<T> cap, @Nullable EnumFacing side) {
	return CapabilityItemHandler.ITEM_HANDLER_CAPABILITY.orEmpty(cap, capItemHandler);
}

 

It's working so far, but I used the CombinedInvWrapper even tho I'm just using one IItemHandler. I figured I don't need to use different sides, I'll allow inserting/extracting on every side.

Is there any better way to do this or is my attempt fine?

 

Thanks so far.

Link to comment
Share on other sites

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.

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

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.