[1.14.4]world.getTileEntity() doesn't work correctly

[Yurim64]

Hi guys, i don't understand why but when i use world.getTileEntity(pos) on a position in which there is a TileEntity setted with world.setTileEntity(pos, entity), it returns me null even if, using debug, the TileEntity exists in the List of the Client and Server World.

I leave the class of the block in which I use the two methods.

 

package com.yurim.alotofthings.block;

import com.yurim.alotofthings.tileentity.LumberjackTileEntity;
import net.minecraft.block.Block;
import net.minecraft.block.BlockState;
import net.minecraft.block.Blocks;
import net.minecraft.block.material.Material;
import net.minecraft.entity.LivingEntity;
import net.minecraft.entity.player.PlayerEntity;
import net.minecraft.entity.player.ServerPlayerEntity;
import net.minecraft.inventory.container.INamedContainerProvider;
import net.minecraft.item.ItemStack;
import net.minecraft.state.StateContainer;
import net.minecraft.state.properties.BlockStateProperties;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.Direction;
import net.minecraft.util.Hand;
import net.minecraft.util.math.BlockPos;
import net.minecraft.util.math.BlockRayTraceResult;
import net.minecraft.world.World;
import net.minecraftforge.fml.network.NetworkHooks;

import javax.annotation.Nullable;

/**
 * @author Yurim
 * @version 1.0A
 **/
@SuppressWarnings("NullableProblems")
public class Lumberjack extends Block {

    public Lumberjack() {
        super(net.minecraft.block.Block.Properties.create(Material.IRON).hardnessAndResistance(2.0f));
    }


    @Override
    public boolean onBlockActivated(BlockState state, World worldIn, BlockPos pos, PlayerEntity player, Hand handIn, BlockRayTraceResult hit) {
        if (!worldIn.isRemote) {
            TileEntity entity = worldIn.getTileEntity(pos);
            if (entity instanceof LumberjackTileEntity && entity.getPos().equals(pos)) {
                NetworkHooks.openGui((ServerPlayerEntity) player, (INamedContainerProvider) entity, entity.getPos());
                return true;
            }
        }
        return false;
    }

    @Override
    public void onBlockPlacedBy(World world, BlockPos pos, BlockState state, @Nullable LivingEntity entity, ItemStack stack) {
        if (entity != null) {
            Direction direction = getFacingFromEntity(pos, entity);
            world.setBlockState(pos, state.with(BlockStateProperties.FACING, direction), 2);
            world.setTileEntity(pos, new LumberjackTileEntity(direction, pos));
        }
    }

    private static Direction getFacingFromEntity(BlockPos clickedBlock, LivingEntity entity) {
        return Direction.getFacingFromVector((float) (entity.posX - clickedBlock.getX()), (float) (entity.posY - clickedBlock.getY()), (float) (entity.posZ - clickedBlock.getZ()));
    }

    @Override
    protected void fillStateContainer(StateContainer.Builder<Block, BlockState> builder) {
        super.fillStateContainer(builder);
        builder.add(BlockStateProperties.FACING);
    }

    @Override
    public void onBlockHarvested(World worldIn, BlockPos pos, BlockState state, PlayerEntity player) {
        TileEntity e = worldIn.getTileEntity(pos);
        if (e instanceof LumberjackTileEntity) {
            ((LumberjackTileEntity) e).dropToWorld(worldIn, pos);
        }
        super.onBlockHarvested(worldIn, pos, state, player);
    }

}

 

[Draco18s]

  On 12/15/2019 at 6:38 PM, Yurim64 said:

world.setTileEntity(pos, new LumberjackTileEntity(direction, pos));

Expand  

You can't just do this. Override hasTileEntity and CreateTileEntity in your block class.

[Yurim64]

  On 12/15/2019 at 6:52 PM, Draco18s said:

You can't just do this. Override hasTileEntity and CreateTileEntity in your block class.

Expand  

yes, but if I do this then I have another problem I was trying to solve. I cannot pass the block direction to TileEntity in the CreateTileEntity method, because the onBlockPlacedBy method, in which I also set the direction, is called later.

Any suggestions?

Edited December 15, 2019 by Yurim64

[Draco18s]

You don't need to. The TileEntity can query the blockstate of its own position any time it wants.

[Yurim64]

  On 12/15/2019 at 7:38 PM, Draco18s said:

You don't need to. The TileEntity can query the blockstate of its own position any time it wants.

Expand  

Oh, ok. Thank you very much