[1.16.5] Help with Capability (Inventory)

loordgek · February 18, 2021

1 hour ago, Luis_ST said:

IItemHandlerModifiable itemHandlerModifiable = (IModItemHandler) serverPlayer.getCapability(ModCapability.CAPABILITY);

1 does getCapability return a IModItemHandler ?

2 does IModItemHandler extend IItemHandlerModifiable ?

loordgek · February 18, 2021

20 hours ago, diesieben07 said:

At this point I doubt if you know basic Java.

Luis_ST · February 18, 2021

1 hour ago, diesieben07 said:

IT IS YOUR DAMN CLASS. Make the constructor however you need it. IT IS YOUR CODE.

i already have a constructor.

but what do I have to do with IItemHandlerModifiable,

how do I get an inventory from it beacuse when i want to add the solts i need an IInventory?

1 hour ago, loordgek said:

1 does getCapability return a IModItemHandler ?

nop but it return a LazyOptional which holds the IModItemHandler

loordgek · February 18, 2021

1 hour ago, Luis_ST said:

nop but it return a LazyOptional which holds the IModItemHandler

why do you do

4 hours ago, Luis_ST said:

IItemHandlerModifiable itemHandlerModifiable = (IModItemHandler) serverPlayer.getCapability(ModCapability.CAPABILITY);

then?

Luis_ST · February 18, 2021

34 minutes ago, loordgek said:

why do you do

then?

what are you telling me?

4 hours ago, diesieben07 said:

No, you just have to register it.

how do I register the IContainer Factory?

Luis_ST · February 18, 2021

19 minutes ago, diesieben07 said:

That the quoted code makes no sense.

I found the error myself

19 minutes ago, diesieben07 said:

It's a parameter to your ContainerType.

Sorry, I don't understand what you mean by parameters exactly, do I need to add something to my containertype?

this is my containerType:

	public static final DeferredRegister<ContainerType<?>> CONTAINERS = DeferredRegister.create(ForgeRegistries.CONTAINERS, Cave.Mod_Id);
	
	public static final RegistryObject<ContainerType<ModEnderChestContainer>> ENDER_CHEST = CONTAINERS.register("mod_ender_chest", 
			() -> IForgeContainerType.create(ModEnderChestContainer::new));

Edited February 18, 2021 by Luis_ST

Luis_ST · February 19, 2021

10 hours ago, diesieben07 said:

Great. Now look at what you are passing in there as parameters...

i just looked at the register class of ContainerType (IForgeContainerType) and this class only needs a single parameter, the IContainerFactory

therefore I don't understand what i should add as a parameter

Luis_ST · February 19, 2021

37 minutes ago, diesieben07 said:

This was your question. This is the answer. You register it together with your ContainerType, as it is a parameter there.

then it should work that way...?

another thing i use AtomicReference to get the IItemHandlerModifiable:

AtomicReference<IItemHandlerModifiable> itemHandlerModifiable = new AtomicReference<>();
serverPlayer.getCapability(ModCapability.CAPABILITY, null).ifPresent(capability -> itemHandlerModifiable.set(capability));

it should work like this because I've done it before with CapabilityItemHandler.

But when I then right click the enderchest i got an error the IItemHandlerModifiable is null.

when i use the debugger i get this:

Cave.LOGGER.debug("is my Present: " + serverPlayer.getCapability(ModCapability.CAPABILITY, null).isPresent()); 
[10:11:46] [Server thread/DEBUG] [ne.lu.ca.Cave/]: is my Present: false

but my AttachCapabilitiesEvent is executed

so why is my capability null?

Luis_ST · February 19, 2021

49 minutes ago, diesieben07 said:

Whyyyyy?

I've tried a long time to find out

is there another possibility?

49 minutes ago, diesieben07 said:

Unknown with just code snippets posted.

AttachCapabilitiesEvent:

	@SubscribeEvent
	public static void AttachCapabilities(AttachCapabilitiesEvent<Entity> event) {
		
		if (event.getObject() instanceof PlayerEntity && !(event.getObject() instanceof FakePlayer)) {
			
			PlayerEntity player = (PlayerEntity) event.getObject();
			event.addCapability(new ResourceLocation("cave:extended_enderchest_gui"), new ModCapability.Provider(player));
			
		}
		
	}

PlayerInteractEvent:

	@SubscribeEvent
	public static void PlayerInteract(PlayerInteractEvent.RightClickBlock event) {
		
		PlayerEntity player = event.getPlayer();
		BlockPos pos = event.getPos();
		World world = event.getWorld();
		BlockState state = world.getBlockState(pos);
		
		if (state.getBlock() == Blocks.ENDER_CHEST) {
			
			if (player instanceof ServerPlayerEntity) {
				
				event.setCanceled(true);
				ServerPlayerEntity serverPlayer = (ServerPlayerEntity) player;
				AtomicReference<IItemHandlerModifiable> itemHandlerModifiable = new AtomicReference<>();
				Cave.LOGGER.debug("is my Present: " + serverPlayer.getCapability(ModCapability.CAPABILITY, null).isPresent()); 
				serverPlayer.getCapability(ModCapability.CAPABILITY, null).ifPresent(capability -> itemHandlerModifiable.set(capability));
				NetworkHooks.openGui(serverPlayer, new SimpleNamedContainerProvider((id, inventory, playerIn) -> {
					return new ModEnderChestContainer(id, inventory, itemHandlerModifiable.get());
				}, CONTAINER_NAME), pos);
				
			}
			
		}

	}

and my Capability (without Storage and Factory):

public class ModCapability {
	
	@CapabilityInject(IModItemHandler.class)
	public static Capability<IModItemHandler> CAPABILITY = null;
	
	public ModCapability() {
		
		CapabilityManager.INSTANCE.register(IModItemHandler.class, new Storage(), new Factory());
		
	}
	
	public static class Provider implements ICapabilitySerializable<CompoundNBT> {
		
		private ModItemStackHandler inventory = new ModItemStackHandler();
		private CombinedInvWrapper invWrapper;
		private LazyOptional<CombinedInvWrapper> optional = LazyOptional.of(() -> invWrapper);
		private PlayerEntity player;
		
		public Provider(PlayerEntity playerIn) {
			
			this.player = playerIn;
			
		}
		
		@Override
		@SuppressWarnings({ "unchecked" })
		public <T> LazyOptional<T> getCapability(Capability<T> cap, Direction side) {
			
			if (optional == null) {
				
				EnderChestInventory enderChestInventory = player.getInventoryEnderChest();
				IItemHandlerModifiable enderChestModifiable = new InvWrapper(enderChestInventory);
				invWrapper = new CombinedInvWrapper(enderChestModifiable, inventory);
				optional = LazyOptional.of(() -> invWrapper);
			}
			
			return cap == CAPABILITY && cap != null ? (LazyOptional<T>) optional : LazyOptional.empty();
			
		}

		@Override
		public CompoundNBT serializeNBT() {
			
			return inventory.serializeNBT();
			
		}

		@Override
		public void deserializeNBT(CompoundNBT nbt) {
			
			inventory.deserializeNBT(nbt);
			
		}
		
	}
	
}

Edited February 19, 2021 by Luis_ST

Luis_ST · February 19, 2021

13 minutes ago, diesieben07 said:

To do, what?

um den inhalt einer LazyOptional zu bekommen

17 minutes ago, diesieben07 said:

This will never be true, you initialize the optional field immediately.

Thinking about this now, LazyOptional already has this lazy initialization built in, so you could just do your lazy initializtion in the Supplier you give to LazyOptional

makes sense that means that should work now:

		private ModItemStackHandler inventory = new ModItemStackHandler();
		private CombinedInvWrapper invWrapper;
		private LazyOptional<CombinedInvWrapper> optional = LazyOptional.of(() -> invWrapper);
		private PlayerEntity player;
		
		public Provider(PlayerEntity playerIn) {
			
			this.player = playerIn;
			
		}
		
		@Override
		@SuppressWarnings({ "unchecked" })
		public <T> LazyOptional<T> getCapability(Capability<T> cap, Direction side) {
			
			if (!optional.isPresent()) {
				
				EnderChestInventory enderChestInventory = player.getInventoryEnderChest();
				IItemHandlerModifiable enderChestModifiable = new InvWrapper(enderChestInventory);
				invWrapper = new CombinedInvWrapper(enderChestModifiable, inventory);
				optional = LazyOptional.of(() -> invWrapper);
			}
			
			return cap == CAPABILITY && cap != null ? (LazyOptional<T>) optional : LazyOptional.empty();
			
		}

Luis_ST · February 19, 2021

1 hour ago, diesieben07 said:

You can't, directly. It could be absent (that's why it's an Optional!). Use orElse or orElseThrow.

like this:

IItemHandlerModifiable itemHandlerModifiable = serverPlayer.getCapability(ModCapability.CAPABILITY, null).orElseThrow(
	() -> new NullPointerException("The mod Capability<IModItemHandler> is null"));

1 hour ago, diesieben07 said:

Just do the lazy initialization in the LazyOptional-supplier.

why in the supplier? does it work on this way:

			if (!optional.isPresent()) {
				
				optional = LazyOptional.of(() -> {
					
					EnderChestInventory enderChestInventory = player.getInventoryEnderChest();
					IItemHandlerModifiable enderChestModifiable = new InvWrapper(enderChestInventory);
					invWrapper = new CombinedInvWrapper(enderChestModifiable, inventory);
					return invWrapper;
					
				});
				
			}
			
			return cap == CAPABILITY && cap != null ? (LazyOptional<T>) optional : LazyOptional.empty();

loordgek · February 19, 2021

private LazyOptional<CombinedInvWrapper> optional = LazyOptional.of(() -> this will lazy initialized);

EnderChestInventory enderChestInventory = player.getInventoryEnderChest();
IItemHandlerModifiable enderChestModifiable = new InvWrapper(enderChestInventory);
invWrapper = new CombinedInvWrapper(enderChestModifiable, inventory);
return invWrapper;

that can be reduced to 1 line

Edited February 19, 2021 by loordgek

Luis_ST · February 19, 2021

1 hour ago, loordgek said:

that can be reduced to 1 line

this is now my Capability Provider:

	public static class Provider implements ICapabilitySerializable<CompoundNBT> {
		
		private ModItemStackHandler inventory = new ModItemStackHandler();
		private PlayerEntity player;
		private LazyOptional<CombinedInvWrapper> optional = LazyOptional.of(() -> new CombinedInvWrapper(new InvWrapper(player.getInventoryEnderChest()), inventory));
		
		public Provider(PlayerEntity playerIn) {
			
			this.player = playerIn;
			
		}
		
		@Override
		@SuppressWarnings({ "unchecked" })
		public <T> LazyOptional<T> getCapability(Capability<T> cap, Direction side) {
			
			return cap == CAPABILITY && cap != null ? (LazyOptional<T>) optional : LazyOptional.empty();
			
		}

		@Override
		public CompoundNBT serializeNBT() {
			
			return inventory.serializeNBT();
			
		}

		@Override
		public void deserializeNBT(CompoundNBT nbt) {
			
			inventory.deserializeNBT(nbt);
			
		}
		
	}

and now i got this error:

java.lang.NullPointerException: The mod Capability<IModItemHandler> is null
	at net.luis.cave.events.entity.player.interact.block.OnEnderchest.lambda$PlayerInteract$0(OnEnderchest.java:41) ~[main/:?] {re:classloading}
	at net.minecraftforge.common.util.LazyOptional.orElseThrow(LazyOptional.java:292) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading}
	at net.luis.cave.events.entity.player.interact.block.OnEnderchest.PlayerInteract(OnEnderchest.java:40) ~[main/:?] {re:classloading}
	at net.minecraftforge.eventbus.ASMEventHandler_29_OnEnderchest_PlayerInteract_RightClickBlock.invoke(.dynamic) ~[?:?] {}
	at net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:85) ~[eventbus-4.0.0.jar:?] {}
	at net.minecraftforge.eventbus.EventBus.post(EventBus.java:302) ~[eventbus-4.0.0.jar:?] {}
	at net.minecraftforge.eventbus.EventBus.post(EventBus.java:283) ~[eventbus-4.0.0.jar:?] {}
	at net.minecraftforge.common.ForgeHooks.onRightClickBlock(ForgeHooks.java:803) ~[forge:?] {re:classloading}
	at net.minecraft.server.management.PlayerInteractionManager.func_219441_a(PlayerInteractionManager.java:327) ~[forge:?] {re:classloading}
	at net.minecraft.network.play.ServerPlayNetHandler.processTryUseItemOnBlock(ServerPlayNetHandler.java:986) ~[forge:?] {re:classloading}
	at net.minecraft.network.play.client.CPlayerTryUseItemOnBlockPacket.processPacket(CPlayerTryUseItemOnBlockPacket.java:45) ~[forge:?] {re:classloading}
	at net.minecraft.network.play.client.CPlayerTryUseItemOnBlockPacket.processPacket(CPlayerTryUseItemOnBlockPacket.java:12) ~[forge:?] {re:classloading}
	at net.minecraft.network.PacketThreadUtil.lambda$checkThreadAndEnqueue$0(PacketThreadUtil.java:19) ~[forge:?] {re:classloading}
	at net.minecraft.util.concurrent.TickDelayedTask.run(TickDelayedTask.java:20) ~[forge:?] {re:classloading}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.run(ThreadTaskExecutor.java:139) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.util.concurrent.RecursiveEventLoop.run(RecursiveEventLoop.java:22) ~[forge:?] {re:classloading}
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:758) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:159) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.driveOne(ThreadTaskExecutor.java:109) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.driveOneInternal(MinecraftServer.java:741) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.driveOne(MinecraftServer.java:735) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.driveUntil(ThreadTaskExecutor.java:122) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.runScheduledTasks(MinecraftServer.java:721) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.func_240802_v_(MinecraftServer.java:667) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.lambda$startServer$0(MinecraftServer.java:233) ~[forge:?] {re:classloading,pl:accesstransformer:B}
	at java.lang.Thread.run(Thread.java:832) [?:?] {}

RightClickBlock event:

	@SubscribeEvent
	public static void PlayerInteract(PlayerInteractEvent.RightClickBlock event) {
		
		PlayerEntity player = event.getPlayer();
		BlockPos pos = event.getPos();
		World world = event.getWorld();
		BlockState state = world.getBlockState(pos);
		
		if (state.getBlock() == Blocks.ENDER_CHEST) {
			
			if (player instanceof ServerPlayerEntity) {
				
				event.setCanceled(true);
				ServerPlayerEntity serverPlayer = (ServerPlayerEntity) player;
				IItemHandlerModifiable itemHandlerModifiable = serverPlayer.getCapability(ModCapability.CAPABILITY, null).orElseThrow(
						() -> new NullPointerException("The mod Capability<IModItemHandler> is null"));
				
				if (itemHandlerModifiable != null) {
					
					NetworkHooks.openGui(serverPlayer, new SimpleNamedContainerProvider((id, inventory, playerIn) -> {
						return new ModEnderChestContainer(id, inventory, itemHandlerModifiable);
					}, CONTAINER_NAME), pos);
					
				}
				
			}
			
		}

	}

Luis_ST · February 19, 2021

1 hour ago, diesieben07 said:

Use the debugger to check why it return the empty optional.

I'm not sure, but I think the LazyOptional isn't initialized because i tried this:

when i search for the message ("this is a test message to find you on the console") on the console i can't find it

CapabilityProvider:

		private ModItemStackHandler inventory = new ModItemStackHandler();
		private PlayerEntity player;
		private LazyOptional<CombinedInvWrapper> optional = LazyOptional.of(() -> {
			
			// i think this is not called
			
			Cave.LOGGER.debug("this is a test message to find you on the console");
			// not on the console
			
			EnderChestInventory enderChestInventory = player.getInventoryEnderChest();
			Cave.LOGGER.debug("enderChestInventory: " + enderChestInventory != null);
			// not on the console
			
			InvWrapper invWrapper = new InvWrapper(enderChestInventory);
			Cave.LOGGER.debug("invWrapper: " + invWrapper != null);
			// not on the console
			
			CombinedInvWrapper combinedInvWrapper = new CombinedInvWrapper(invWrapper, inventory);
			Cave.LOGGER.debug("inventory: " + inventory != null);
			Cave.LOGGER.debug("combinedInvWrapper: " + combinedInvWrapper != null);
			// not on the console
			
			return combinedInvWrapper;
			
		});
		
		public Provider(PlayerEntity playerIn) {
			
			this.player = playerIn;
			
		}
		
		@Override
		@SuppressWarnings({ "unchecked" })
		public <T> LazyOptional<T> getCapability(Capability<T> cap, Direction side) {
			
			Cave.LOGGER.debug("getCapability: cap == CAPABILITY: " + (cap == CAPABILITY));
			// on the console: return true
			
			Cave.LOGGER.debug("getCapability: cap != null: " + cap != null);
			// on the console: return true
			
			Cave.LOGGER.debug("optional: " + optional.isPresent());
			// on the console: return true
			
			return cap == CAPABILITY && cap != null ? (LazyOptional<T>) optional : LazyOptional.empty();
			
		}

and the event:

			if (player instanceof ServerPlayerEntity) {
				
				event.setCanceled(true);
				ServerPlayerEntity serverPlayer = (ServerPlayerEntity) player;
				IItemHandlerModifiable itemHandlerModifiable = serverPlayer.getCapability(ModCapability.CAPABILITY, null).orElseThrow(
				() -> new NullPointerException("The mod Capability<IModItemHandler> is null"));
				
				Cave.LOGGER.debug("getCapability isPresent:  " + serverPlayer.getCapability(ModCapability.CAPABILITY, null).isPresent());
				// on the console: return false
				
				if (itemHandlerModifiable != null) {
					
					NetworkHooks.openGui(serverPlayer, new SimpleNamedContainerProvider((id, inventory, playerIn) -> {
						return new ModEnderChestContainer(id, inventory, itemHandlerModifiable);
					}, CONTAINER_NAME), pos);
					
				}
				
			}

if someone still has an idea what i can do to find the problem i am open to suggestions

Edited February 19, 2021 by Luis_ST

Luis_ST · February 19, 2021

2 minutes ago, diesieben07 said:

Post a Git repo of your mod.

https://github.com/Luis-st/Forge-1.16.5-36.0.1-mdk/tree/main/forge-1.16.5-36.0.1-mdk

Luis_ST · February 19, 2021

50 minutes ago, diesieben07 said:

You never register your capability.

if i understand the forge doc correctly

i have to run the constructor in my capability class but in which FML setup do i have to do this (FMLCommonSetupEvent)

53 minutes ago, diesieben07 said:

Oh and by the way, adding debug logs is not "using the debugger".

I took a little too literally

Luis_ST · February 19, 2021

8 minutes ago, diesieben07 said:

Why are you doing it in the constructor? It makes no sense to do it there.

FMLCommonSetupEvent is the correct event.

I get an error again (cannot be cast)

now my question do i need this method in my container:

assertInventorySize((IInventory) itemHandlerModifiable, 6 * 9);

because I cannot cast IItemHandlerModifiable to an IInventory

when i need them what do i have to use there instead

loordgek · February 19, 2021

1 hour ago, Luis_ST said:

because I cannot cast IItemHandlerModifiable to an IInventory

why do you cast?

Luis_ST · February 19, 2021

37 minutes ago, loordgek said:

why do you cast?

my ide says that I need an IInventory and not an IItemHandlerModifiable

loordgek · February 19, 2021

where

Luis_ST · February 19, 2021

1 minute ago, loordgek said:

where

in my container at:

assertInventorySize((IInventory) itemHandlerModifiable, 6 * 9);

	public ModEnderChestContainer(int id, PlayerInventory playerInventoryIn, IItemHandlerModifiable itemHandlerModifiable) {
		
		// TODO: fix
		
		super(ModContainerType.ENDER_CHEST.get(), id);
		assertInventorySize((IInventory) itemHandlerModifiable, 6 * 9);
		int i = (6 - 4) * 18;
		
		for (int j = 0; j < 6; ++j) {

			for (int k = 0; k < 9; ++k) {

				this.addSlot(new SlotItemHandler(itemHandlerModifiable, k + j * 9, 8 + k * 18, 18 + j * 18));

			}

		}

		for (int l = 0; l < 3; ++l) {
			
			for (int j1 = 0; j1 < 9; ++j1) {
				
				this.addSlot(new Slot(playerInventoryIn, j1 + l * 9 + 9, 8 + j1 * 18, 103 + l * 18 + i));
				
			}
			
		}

		for (int i1 = 0; i1 < 9; ++i1) {
			
			this.addSlot(new Slot(playerInventoryIn, i1, 8 + i1 * 18, 161 + i));
			
		}

	}

loordgek · February 19, 2021

go and look what assertInventorySize does and

A: replicate it using IItemHandler or

B: remove it

Edited February 19, 2021 by loordgek

Luis_ST · February 19, 2021

2 minutes ago, loordgek said:

B: remove it

if I understand it correctly, the method only looks if the inventory doesn't have too many inventory slots. if there is too much, it will issue an error

now i got another error the container is opened but has no slots:

[20:15:14] [Render thread/FATAL] [minecraft/ThreadTaskExecutor]: Error executing task on Client
java.lang.IndexOutOfBoundsException: Index 0 out of bounds for length 0
	at jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64) ~[?:?] {}
	at jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70) ~[?:?] {}
	at jdk.internal.util.Preconditions.checkIndex(Preconditions.java:248) ~[?:?] {}
	at java.util.Objects.checkIndex(Objects.java:359) ~[?:?] {}
	at java.util.ArrayList.get(ArrayList.java:427) ~[?:?] {}
	at net.minecraft.inventory.container.Container.getSlot(Container.java:165) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading}
	at net.minecraft.inventory.container.Container.setAll(Container.java:486) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading}
	at net.minecraft.client.network.play.ClientPlayNetHandler.handleWindowItems(ClientPlayNetHandler.java:1224) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading,pl:runtimedistcleaner:A}
	at net.minecraft.network.play.server.SWindowItemsPacket.processPacket(SWindowItemsPacket.java:61) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading}
	at net.minecraft.network.play.server.SWindowItemsPacket.processPacket(SWindowItemsPacket.java:13) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading}
	at net.minecraft.network.PacketThreadUtil.lambda$checkThreadAndEnqueue$0(PacketThreadUtil.java:19) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.run(ThreadTaskExecutor.java:139) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.util.concurrent.RecursiveEventLoop.run(RecursiveEventLoop.java:22) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.driveOne(ThreadTaskExecutor.java:109) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.drainTasks(ThreadTaskExecutor.java:97) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading,pl:accesstransformer:B}
	at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:973) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading,pl:accesstransformer:B,pl:runtimedistcleaner:A}
	at net.minecraft.client.Minecraft.run(Minecraft.java:612) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading,pl:accesstransformer:B,pl:runtimedistcleaner:A}
	at net.minecraft.client.main.Main.main(Main.java:184) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {re:classloading,pl:runtimedistcleaner:A}
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?] {}
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:64) ~[?:?] {}
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?] {}
	at java.lang.reflect.Method.invoke(Method.java:564) ~[?:?] {}
	at net.minecraftforge.userdev.FMLUserdevClientLaunchProvider.lambda$launchService$0(FMLUserdevClientLaunchProvider.java:52) ~[forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {}
	at cpw.mods.modlauncher.LaunchServiceHandlerDecorator.launch(LaunchServiceHandlerDecorator.java:37) [modlauncher-8.0.9.jar:?] {}
	at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:54) [modlauncher-8.0.9.jar:?] {}
	at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:72) [modlauncher-8.0.9.jar:?] {}
	at cpw.mods.modlauncher.Launcher.run(Launcher.java:82) [modlauncher-8.0.9.jar:?] {}
	at cpw.mods.modlauncher.Launcher.main(Launcher.java:66) [modlauncher-8.0.9.jar:?] {}
	at net.minecraftforge.userdev.LaunchTesting.main(LaunchTesting.java:105) [forge-1.16.5-36.0.1_mapped_snapshot_20201028-1.16.3-recomp.jar:?] {}

Luis_ST · February 21, 2021

14 hours ago, diesieben07 said:

The constructor used by your IContainerFactory (here) does not add any slots. This means the client side container won't have any slots. You must add identical slots on client and server.

okay if I want to add slots there (those of the container) I need an IItemHandlerModifiable.

Can I save the IItemHandlerModifiable from the server constructor in a field and then use it in the client constructor?

Luis_ST · February 21, 2021

22 minutes ago, diesieben07 said:

No. You need to get it from the player, same way you do on the server.

okay i now have slots in the container they are not in the correct position yet but that should be a matter of setting

unfortunately it still doesn't do perfect:

the enderchest inventory works but in my inventory only the first slot works,

the others are displayed but I can't put any items in the slots

Sign In

[1.16.5] Help with Capability (Inventory)

Recommended Posts

Top Posters In This Topic

Popular Days

Top Posters In This Topic

Popular Days

Popular Posts

loordgek

Posted Images

Join the conversation

Announcements

Recently Browsing

Posts

Topics

Who's Online (See full list)

Important Information