Jump to content

[SOLVED][1.8] Multitextures


Recommended Posts

Posted

So I am making several types of wood for my mod, and I am having a problem with the rendering of the block. The problem is not that I don't get to render the block, but the fact that one of the types of plank are using the wrong texture. Namely the alder plank.  Where it use the texture of the yew plank.  If you look at the code in the class BlockPlank you can see that I have placed in some System.out.println() for checking what values or strings are used. When placing the alderplank, it outputs

 

[11:12:27] [Client thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getStateFromMeta:51]: em:plank[woodtype=Alder]
[11:12:27] [server thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getStateFromMeta:51]: em:plank[woodtype=Alder]

which it should, but when broken it outputs

[11:14:37] [Client thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getMetaFromState:58]: Yew
16
[11:14:37] [Client thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getStateFromMeta:51]: em:plank[woodtype=Yew]
[11:14:37] [server thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getMetaFromState:58]: Yew
16
[11:14:37] [server thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getStateFromMeta:51]: em:plank[woodtype=Yew]
[11:14:37] [server thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getMetaFromState:58]: Yew
16
[11:14:37] [server thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getStateFromMeta:51]: em:plank[woodtype=Yew]
[11:14:37] [server thread/INFO] [sTDOUT]: [com.tommywithatiger.extendedminecraft.blocks.BlockPlank:getMetaFromState:58]: Yew
16

 

Which now means that the game thinks it is Yew plank. I have been trying to look for why it does this, but havn't really found the error. If someone could help me that would be great

 

 

Code:

 

BlockPlank:

package com.tommywithatiger.extendedminecraft.blocks;

import java.util.List;

import net.minecraft.block.Block;
import net.minecraft.block.material.Material;
import net.minecraft.block.properties.IProperty;
import net.minecraft.block.properties.PropertyEnum;
import net.minecraft.block.state.BlockState;
import net.minecraft.block.state.IBlockState;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.util.BlockPos;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.IStringSerializable;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

import com.tommywithatiger.extendedminecraft.handler.ExtendedMinecraftCreativeTabHandler;

public class BlockPlank extends Block{

public BlockPlank() {
	super(Material.wood);
	this.setCreativeTab(ExtendedMinecraftCreativeTabHandler.ExtendedMinecraftBlocksTab);
	this.setDefaultState(this.blockState.getBaseState().withProperty(PROPERTYTYPE, BlockPlank.EnumWoodType.ALDER));
}

public static final PropertyEnum PROPERTYTYPE = PropertyEnum.create("woodtype", EnumWoodType.class);


@Override
@SideOnly(Side.CLIENT)
public void getSubBlocks(Item itemIn, CreativeTabs tab, List list){
	for(EnumWoodType type: EnumWoodType.values()){
			list.add(new ItemStack(itemIn, 1, type.getMetadata()));
	}
}

@Override
protected BlockState createBlockState()  {
	return new BlockState(this, new IProperty[] {PROPERTYTYPE});
    }

@Override
public IBlockState getStateFromMeta(int meta){
	EnumWoodType type = EnumWoodType.byMetadata(meta);
	System.out.println(this.getDefaultState().withProperty(PROPERTYTYPE, type));
	return this.getDefaultState().withProperty(PROPERTYTYPE, type);
}

@Override
public int getMetaFromState(IBlockState state){
	EnumWoodType type = (EnumWoodType)state.getValue(PROPERTYTYPE);
	System.out.println(type);
	int typebit = type.getMetadata();
	System.out.println(typebit);
	return typebit;
}

@Override
public IBlockState onBlockPlaced(World worldIn, BlockPos pos, EnumFacing blockFaceClickedOn, float hitX, float hitY, float hitZ, int meta, EntityLivingBase placer)
{
	EnumWoodType type = EnumWoodType.byMetadata(meta);
	System.out.println(this.getDefaultState().withProperty(PROPERTYTYPE, type));
	return this.getDefaultState().withProperty(PROPERTYTYPE, type);
}

public static enum EnumWoodType implements IStringSerializable
{
    ALDER(0,"Alder","alder"),
    ASH(1,"Ash","ash"),
    ASPEN(2,"Aspen","aspen"),
    CEDAR(3,"Cedar","cedar"),
    CHERRY(4,"Cherry","cherry"),
    ELM(5,"Elm","elm"),
    EUCALYPTUS(6,"Eucalyptus","eucalyptus"),
    FIR(7,"Fir","fir"),
    HICKORY(8,"Hickory","hickory"),
    MAHOGANY(9,"Mahogany","mahogany"),
    MAPLE(10,"Maple","maple"),
    PINE(11,"Pine","pine"),
    POPLAR(12,"Poplar","poplar"),
    REDWOOD(13,"Redwood","redwood"),
    TEAK(14,"Teak","teak"),
    WILLOW(15,"Willow","willow"),
    YEW(16,"Yew","yew");
    
    private final int meta;
    private final String name;
    private final String substring;
    private static final EnumWoodType[] META_LOOKUP = new BlockPlank.EnumWoodType[values().length];
    
    private EnumWoodType(int i_meta, String i_name, String i_substring){
    	this.meta = i_meta;
    	this.name = i_name;
    	this.substring = i_substring;
    }
    
    public int getMetadata(){
    	return this.meta;
    }
    
    @Override
    public String toString()
    {
    	return this.name;
    }
    
    public String getSubstring(){
    	return this.substring;
    }
    
    public static EnumWoodType byMetadata(int meta)
    {
    	if (meta < 0 || meta >= META_LOOKUP.length)
    	{
    		meta = 0;
    	}
    	
    	return META_LOOKUP[meta];
    }
    
    public String getName(){
    	return this.name;
    }
    	    
    static
    {
    	for ( EnumWoodType woodType : values()){
    		META_LOOKUP[woodType.getMetadata()] = woodType;
    	}
    }

}

}

 

plank.json:

{
    "variants": {
        "woodtype=alder": { "model": "em:alderplank" },
        "woodtype=ash": {"model": "em:medium_crafting_table"},
        "woodtype=aspen": {"model": "em:medium_crafting_table"},
        "woodtype=cedar": {"model": "em:medium_crafting_table"},
        "woodtype=cherry": {"model": "em:medium_crafting_table"},
        "woodtype=elm": {"model": "em:medium_crafting_table"},
        "woodtype=eucalyptus": {"model": "em:medium_crafting_table"},
        "woodtype=fir": {"model": "em:medium_crafting_table"},
        "woodtype=hickory": {"model": "em:medium_crafting_table"},
        "woodtype=mahogany": {"model": "em:medium_crafting_table"},
        "woodtype=maple": {"model": "em:medium_crafting_table"},
        "woodtype=pine": {"model": "em:medium_crafting_table"},
        "woodtype=poplar": {"model": "em:medium_crafting_table"},
        "woodtype=redwood": {"model": "em:medium_crafting_table"},
        "woodtype=teak": {"model": "em:medium_crafting_table"},
        "woodtype=willow": {"model": "em:medium_crafting_table"},
        "woodtype=yew": {"model": "em:small_crafting_table"}
    }
}

 

 

Posted

You have 17 types. You can only have 16 of those being saved as metadata (metadata is still saved as 4 bits => 0-15).

The world now thinks the block is from the last type when it's actually the first one.

Don't ask for support per PM! They'll get ignored! | If a post helped you, click the "Thank You" button at the top right corner of said post! |

mah twitter

This thread makes me sad because people just post copy-paste-ready code when it's obvious that the OP has little to no programming experience. This is not how learning works.

Posted

So what you are saying is that I can only have 16 different types saved in metadata. So if you want more than 16 you have to register several blocks? Or is it possibly to setup another Enum?

Guest
This topic is now closed to further replies.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Version 1.19 - Forge 41.0.63 I want to create a wolf entity that I can ride, so far it seems to be working, but the problem is that when I get on the wolf, I can’t control it. I then discovered that the issue is that the server doesn’t detect that I’m riding the wolf, so I’m struggling with synchronization. However, it seems to not be working properly. As I understand it, the server receives the packet but doesn’t register it correctly. I’m a bit new to Java, and I’ll try to provide all the relevant code and prints *The comments and prints are translated by chatgpt since they were originally in Spanish* Thank you very much in advance No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. MountableWolfEntity package com.vals.valscraft.entity; import com.vals.valscraft.network.MountSyncPacket; import com.vals.valscraft.network.NetworkHandler; import net.minecraft.client.Minecraft; import net.minecraft.network.syncher.EntityDataAccessor; import net.minecraft.network.syncher.EntityDataSerializers; import net.minecraft.network.syncher.SynchedEntityData; import net.minecraft.server.MinecraftServer; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.EntityType; import net.minecraft.world.entity.Mob; import net.minecraft.world.entity.ai.attributes.AttributeSupplier; import net.minecraft.world.entity.ai.attributes.Attributes; import net.minecraft.world.entity.animal.Wolf; import net.minecraft.world.entity.player.Player; import net.minecraft.world.entity.Entity; import net.minecraft.world.InteractionHand; import net.minecraft.world.InteractionResult; import net.minecraft.world.item.ItemStack; import net.minecraft.world.item.Items; import net.minecraft.world.level.Level; import net.minecraft.world.phys.Vec3; import net.minecraftforge.event.TickEvent; import net.minecraftforge.eventbus.api.SubscribeEvent; import net.minecraftforge.network.PacketDistributor; public class MountableWolfEntity extends Wolf { private boolean hasSaddle; private static final EntityDataAccessor<Byte> DATA_ID_FLAGS = SynchedEntityData.defineId(MountableWolfEntity.class, EntityDataSerializers.BYTE); public MountableWolfEntity(EntityType<? extends Wolf> type, Level level) { super(type, level); this.hasSaddle = false; } @Override protected void defineSynchedData() { super.defineSynchedData(); this.entityData.define(DATA_ID_FLAGS, (byte)0); } public static AttributeSupplier.Builder createAttributes() { return Wolf.createAttributes() .add(Attributes.MAX_HEALTH, 20.0) .add(Attributes.MOVEMENT_SPEED, 0.3); } @Override public InteractionResult mobInteract(Player player, InteractionHand hand) { ItemStack itemstack = player.getItemInHand(hand); if (itemstack.getItem() == Items.SADDLE && !this.hasSaddle()) { if (!player.isCreative()) { itemstack.shrink(1); } this.setSaddle(true); return InteractionResult.SUCCESS; } else if (!level.isClientSide && this.hasSaddle()) { player.startRiding(this); MountSyncPacket packet = new MountSyncPacket(true); // 'true' means the player is mounted NetworkHandler.CHANNEL.sendToServer(packet); // Ensure the server handles the packet return InteractionResult.SUCCESS; } return InteractionResult.PASS; } @Override public void travel(Vec3 travelVector) { if (this.isVehicle() && this.getControllingPassenger() instanceof Player) { System.out.println("The wolf has a passenger."); System.out.println("The passenger is a player."); Player player = (Player) this.getControllingPassenger(); // Ensure the player is the controller this.setYRot(player.getYRot()); this.yRotO = this.getYRot(); this.setXRot(player.getXRot() * 0.5F); this.setRot(this.getYRot(), this.getXRot()); this.yBodyRot = this.getYRot(); this.yHeadRot = this.yBodyRot; float forward = player.zza; float strafe = player.xxa; if (forward <= 0.0F) { forward *= 0.25F; } this.flyingSpeed = this.getSpeed() * 0.1F; this.setSpeed((float) this.getAttributeValue(Attributes.MOVEMENT_SPEED) * 1.5F); this.setDeltaMovement(new Vec3(strafe, travelVector.y, forward).scale(this.getSpeed())); this.calculateEntityAnimation(this, false); } else { // The wolf does not have a passenger or the passenger is not a player System.out.println("No player is mounted, or the passenger is not a player."); super.travel(travelVector); } } public boolean hasSaddle() { return this.hasSaddle; } public void setSaddle(boolean hasSaddle) { this.hasSaddle = hasSaddle; } @Override protected void dropEquipment() { super.dropEquipment(); if (this.hasSaddle()) { this.spawnAtLocation(Items.SADDLE); this.setSaddle(false); } } @SubscribeEvent public static void onServerTick(TickEvent.ServerTickEvent event) { if (event.phase == TickEvent.Phase.START) { MinecraftServer server = net.minecraftforge.server.ServerLifecycleHooks.getCurrentServer(); if (server != null) { for (ServerPlayer player : server.getPlayerList().getPlayers()) { if (player.isPassenger() && player.getVehicle() instanceof MountableWolfEntity) { MountableWolfEntity wolf = (MountableWolfEntity) player.getVehicle(); System.out.println("Tick: " + player.getName().getString() + " is correctly mounted on " + wolf); } } } } } private boolean lastMountedState = false; @Override public void tick() { super.tick(); if (!this.level.isClientSide) { // Only on the server boolean isMounted = this.isVehicle() && this.getControllingPassenger() instanceof Player; // Only print if the state changed if (isMounted != lastMountedState) { if (isMounted) { Player player = (Player) this.getControllingPassenger(); // Verify the passenger is a player System.out.println("Server: Player " + player.getName().getString() + " is now mounted."); } else { System.out.println("Server: The wolf no longer has a passenger."); } lastMountedState = isMounted; } } } @Override public void addPassenger(Entity passenger) { super.addPassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(true)); } } } @Override public void removePassenger(Entity passenger) { super.removePassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is no longer mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(false)); } } } @Override public boolean isControlledByLocalInstance() { Entity entity = this.getControllingPassenger(); return entity instanceof Player; } @Override public void positionRider(Entity passenger) { if (this.hasPassenger(passenger)) { double xOffset = Math.cos(Math.toRadians(this.getYRot() + 90)) * 0.4; double zOffset = Math.sin(Math.toRadians(this.getYRot() + 90)) * 0.4; passenger.setPos(this.getX() + xOffset, this.getY() + this.getPassengersRidingOffset() + passenger.getMyRidingOffset(), this.getZ() + zOffset); } } } MountSyncPacket package com.vals.valscraft.network; import com.vals.valscraft.entity.MountableWolfEntity; import net.minecraft.network.FriendlyByteBuf; import net.minecraft.server.level.ServerLevel; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.Entity; import net.minecraft.world.entity.player.Player; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class MountSyncPacket { private final boolean isMounted; public MountSyncPacket(boolean isMounted) { this.isMounted = isMounted; } public void encode(FriendlyByteBuf buffer) { buffer.writeBoolean(isMounted); } public static MountSyncPacket decode(FriendlyByteBuf buffer) { return new MountSyncPacket(buffer.readBoolean()); } public void handle(NetworkEvent.Context context) { context.enqueueWork(() -> { ServerPlayer player = context.getSender(); // Get the player from the context if (player != null) { // Verifies if the player has dismounted if (!isMounted) { Entity vehicle = player.getVehicle(); if (vehicle instanceof MountableWolfEntity wolf) { // Logic to remove the player as a passenger wolf.removePassenger(player); System.out.println("Server: Player " + player.getName().getString() + " is no longer mounted."); } } } }); context.setPacketHandled(true); // Marks the packet as handled } } networkHandler package com.vals.valscraft.network; import com.vals.valscraft.valscraft; import net.minecraft.resources.ResourceLocation; import net.minecraftforge.network.NetworkRegistry; import net.minecraftforge.network.simple.SimpleChannel; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class NetworkHandler { private static final String PROTOCOL_VERSION = "1"; public static final SimpleChannel CHANNEL = NetworkRegistry.newSimpleChannel( new ResourceLocation(valscraft.MODID, "main"), () -> PROTOCOL_VERSION, PROTOCOL_VERSION::equals, PROTOCOL_VERSION::equals ); public static void init() { int packetId = 0; // Register the mount synchronization packet CHANNEL.registerMessage( packetId++, MountSyncPacket.class, MountSyncPacket::encode, MountSyncPacket::decode, (msg, context) -> msg.handle(context.get()) // Get the context with context.get() ); } }  
    • Do you use features of inventory profiles next (ipnext) or is there a change without it?
    • Remove rubidium - you are already using embeddium, which is a fork of rubidium
  • Topics

×
×
  • Create New...

Important Information

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