How to implement inventories

[rackodo]

Description

Following this tutorial to implement an inventory, but the readNbt and writeNbt methods are asking for an extra argument that the tutorial doesn't address. Any ideas?

@Override
public void readNbt(NbtCompound nbt) {
    super.readNbt(nbt);                            [1]
    Inventories.readNbt(nbt, items);               [2]
}

@Override
public NbtCompound writeNbt(NbtCompound nbt) {
    Inventories.writeNbt(nbt, items);              [3]
    return super.writeNbt(nbt);                    [4]
}

[1] 'readNbt(net.minecraft.nbt.NbtCompound, net.minecraft.registry.RegistryWrapper.WrapperLookup)' in 'net.minecraft.block.entity.BlockEntity' cannot be applied to '(net.minecraft.nbt.NbtCompound)'

[2] 'readNbt(net.minecraft.nbt.NbtCompound, net.minecraft.util.collection.DefaultedList<net.minecraft.item.ItemStack>, net.minecraft.registry.RegistryWrapper.WrapperLookup)' in 'net.minecraft.inventory.Inventories' cannot be applied to '(net.minecraft.nbt.NbtCompound, net.minecraft.util.collection.DefaultedList<net.minecraft.item.ItemStack>)'

[3] Cannot resolve method 'writeNbt(NbtCompound, DefaultedList<ItemStack>)'

[4] 'writeNbt(net.minecraft.nbt.NbtCompound, net.minecraft.registry.RegistryWrapper.WrapperLookup)' in 'net.minecraft.block.entity.BlockEntity' cannot be applied to '(net.minecraft.nbt.NbtCompound)'

[melontini]

Hi, in 1.20.5 the signature of nbt methods changed to include RegistryWrapper.WrapperLookup .

Your code has to look like this:

@Override
public void readNbt(NbtCompound nbt, RegistryWrapper.WrapperLookup lookup) {
    super.readNbt(nbt, lookup); 
    Inventories.readNbt(nbt, items, lookup);
}

@Override
public NbtCompound writeNbt(NbtCompound nbt, RegistryWrapper.WrapperLookup lookup) {
    Inventories.writeNbt(nbt, items, lookup);
    return super.writeNbt(nbt, lookup);
}