[1.16.5] Block entity invalid when implement ITickableEntity

[SomeDudeOnTheInternet]

When I place block with tickable tile entity, this happens

[13:36:59] [Render thread/WARN] [minecraft/TileEntity]: Block entity invalid: futurearmour:fabricator_controller_tet @ BlockPos{x=-149, y=4, z=341}
[13:36:59] [Server thread/WARN] [minecraft/TileEntity]: Block entity invalid: futurearmour:fabricator_controller_tet @ BlockPos{x=-149, y=4, z=341}

 

Tile Entity:

  Reveal hidden contents

package com.spu.futurearmour.content.tileentities;

import com.spu.futurearmour.setup.TileEntityTypeRegistry;
import net.minecraft.tileentity.ITickableTileEntity;
import net.minecraft.tileentity.TileEntity;

import java.util.UUID;

public class FabricatorControllerTileEntity extends TileEntity implements ITickableTileEntity {
    public FabricatorControllerTileEntity() {
        super(TileEntityTypeRegistry.FABRICATOR_CONTROLLER_TILE_ENTITY_TYPE.get());
    }

    @Override
    public void tick() {

    }
}

 

 

Block:

  Reveal hidden contents

package com.spu.futurearmour.content.blocks.fabricator;

import com.spu.futurearmour.content.tileentities.FabricatorControllerTileEntity;
import net.minecraft.block.*;
import net.minecraft.block.material.Material;
import net.minecraft.item.BlockItemUseContext;
import net.minecraft.state.DirectionProperty;
import net.minecraft.state.StateContainer;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.Direction;
import net.minecraft.world.IBlockReader;

import javax.annotation.Nullable;

public class FabricatorController extends Block {
    public static final DirectionProperty FACING = HorizontalBlock.FACING;

    public FabricatorController() {
        super(AbstractBlock.Properties.of(Material.HEAVY_METAL)
        .strength(5)
        .sound(SoundType.METAL));

        this.registerDefaultState(this.stateDefinition.any().setValue(FACING, Direction.NORTH));
    }

    @SuppressWarnings("deprecation")
    public BlockRenderType getRenderShape(BlockState blockState) {
        return BlockRenderType.MODEL;
    }

    public BlockState getStateForPlacement(BlockItemUseContext itemUseContext) {
        return this.defaultBlockState().setValue(FACING, itemUseContext.getHorizontalDirection().getOpposite());
    }

    protected void createBlockStateDefinition(StateContainer.Builder<Block, BlockState> stateBuilder) {
        stateBuilder.add(FACING);
    }

    @Override
    public boolean hasTileEntity(BlockState state) {
        return true;
    }

    @Nullable
    @Override
    public TileEntity createTileEntity(BlockState state, IBlockReader world) {
        return new FabricatorControllerTileEntity();
    }
}

 

 

Registration:

  Reveal hidden contents

package com.spu.futurearmour.setup;

import com.spu.futurearmour.content.ModItemModelsProperties;

public class Registration {
    public static void register(){
        BlockRegistry.register();
        TileEntityTypeRegistry.register();
        ItemRegistry.register();
    }

    public static void registerClientOnly(){
        ModItemModelsProperties.register();
    }
}

 

  Reveal hidden contents

package com.spu.futurearmour.setup;

import com.spu.futurearmour.FutureArmour;
import com.spu.futurearmour.content.blocks.*;
import com.spu.futurearmour.content.blocks.fabricator.FabricatorController;
import net.minecraft.block.Block;
import net.minecraft.item.BlockItem;
import net.minecraft.item.Item;
import net.minecraftforge.fml.RegistryObject;
import net.minecraftforge.fml.javafmlmod.FMLJavaModLoadingContext;
import net.minecraftforge.registries.DeferredRegister;
import net.minecraftforge.registries.ForgeRegistries;

public class BlockRegistry {
    public static final DeferredRegister<Block> BLOCKS = DeferredRegister.create(ForgeRegistries.BLOCKS, FutureArmour.MOD_ID);

    public static void register(){
        BLOCKS.register(FMLJavaModLoadingContext.get().getModEventBus());
    }

    public static final RegistryObject<FabricatorController> FABRICATOR_CONTROLLER_BLOCK =
            BLOCKS.register("fabricator_controller_block", () -> new FabricatorController());
    public static final RegistryObject<Item> FABRICATOR_CONTROLLER_BLOCK_ITEM =
            ItemRegistry.ITEMS.register("fabricator_controller_block", ()->
                    new BlockItem(FABRICATOR_CONTROLLER_BLOCK.get(), new Item.Properties().tab(FutureArmour.ITEM_GROUP)));
}

 

  Reveal hidden contents

package com.spu.futurearmour.setup;

import com.spu.futurearmour.FutureArmour;
import com.spu.futurearmour.content.tileentities.FabricatorControllerTileEntity;
import net.minecraft.tileentity.TileEntityType;
import net.minecraftforge.fml.RegistryObject;
import net.minecraftforge.fml.javafmlmod.FMLJavaModLoadingContext;
import net.minecraftforge.registries.DeferredRegister;
import net.minecraftforge.registries.ForgeRegistries;

public class TileEntityTypeRegistry {
    public static final DeferredRegister<TileEntityType<?>> TILE_ENTITY_TYPES = DeferredRegister.create(ForgeRegistries.TILE_ENTITIES, FutureArmour.MOD_ID);

    public static void register() {
        TILE_ENTITY_TYPES.register(FMLJavaModLoadingContext.get().getModEventBus());
    }

    public static final RegistryObject<TileEntityType<FabricatorControllerTileEntity>> FABRICATOR_CONTROLLER_TILE_ENTITY_TYPE =
            TILE_ENTITY_TYPES.register("fabricator_controller_tet", () ->
                    TileEntityType.Builder.of(FabricatorControllerTileEntity::new).build(null));
}

 

Any suggestions?

[SomeDudeOnTheInternet]

  On 7/1/2021 at 12:08 PM, diesieben07 said:

Do not put a DeferredRegister and its entries in separate classes (mainly FABRICATOR_CONTROLLER_BLOCK_ITEM and ItemRegistry.ITEMS). Either move FABRICATOR_CONTROLLER_BLOCK_ITEM or make a secondary DeferredRegister for items in BlockRegistry.

You have told your TileEntityType that no blocks are valid for it, hence it reports this fact. Pass the valid blocks to the of method.

Expand  

Wow that was quick, thanks a lot!