BlockStates - Multipart

[cmchenry]

package com.minelife.drugs.block;

import com.minelife.Minelife;
import com.minelife.drugs.ModDrugs;
import net.minecraft.block.BlockCrops;
import net.minecraft.block.properties.IProperty;
import net.minecraft.block.properties.PropertyBool;
import net.minecraft.block.properties.PropertyInteger;
import net.minecraft.block.state.BlockStateContainer;
import net.minecraft.block.state.IBlockState;
import net.minecraft.item.Item;
import net.minecraft.util.EnumBlockRenderType;
import net.minecraft.util.math.BlockPos;
import net.minecraft.world.IBlockAccess;
import net.minecraft.world.World;

public class BlockHempCrop extends BlockCrops {

    public static final PropertyInteger FEMALE = PropertyInteger.create("female", 0, 1);

    public BlockHempCrop() {
        setRegistryName(Minelife.MOD_ID, "hemp_crop");
        setUnlocalizedName(Minelife.MOD_ID + ":hemp_crop");
        setTickRandomly(true);
        setCreativeTab(null);
        setHardness(0.0F);
        disableStats();
    }

    // TODO: Still grows in non farmland

    @Override
    public void grow(World worldIn, BlockPos pos, IBlockState state) {
        super.grow(worldIn, pos, state);
//        if(getAge(state) == 7)
            worldIn.setBlockState(pos, this.withAge(7).withProperty(FEMALE, 1));

        System.out.println(worldIn.getBlockState(pos).getProperties().get(FEMALE));
    }

    @Override
    protected BlockStateContainer createBlockState() {
        return new BlockStateContainer(this, new IProperty[] {AGE, FEMALE});
    }

    @Override
    protected Item getCrop() {
        return ModDrugs.itemHempBuds;
    }

    @Override
    protected Item getSeed() {
        return ModDrugs.itemHempSeed;
    }

}

 

{
  "multipart": [
    {
      "when": { "age": 0 },
      "apply": { "model": "minelife:hemp_stage0"}
    },
    {
      "when": { "age": 1 },
      "apply": { "model": "minelife:hemp_stage1"}
    },
    {
      "when": { "age": 2 },
      "apply": { "model": "minelife:hemp_stage1"}
    },
    {
      "when": { "age": 3 },
      "apply": { "model": "minelife:hemp_stage2"}
    },
    {
      "when": { "age": 4 },
      "apply": { "model": "minelife:hemp_stage2"}
    },
    {
      "when": { "age": 5 },
      "apply": { "model": "minelife:hemp_stage3"}
    },
    {
      "when": { "age": 6 },
      "apply": { "model": "minelife:hemp_stage4"}
    },
    {
      "when": { "age": 7, "female": 1 },
      "apply": { "model": "minelife:hemp_stage4"}
    },
    {
      "when": { "age": 7, "female": 0 },
      "apply": { "model": "minelife:hemp_male_stage4"}
    }
  ]
}

 

For some reason, no matter what it always renders as the female=0 scenario for age 7. What am I doing wrong?

[Draco18s]

I am not sure how the multistate format works, so I'm going to suggest going the rout of a custom state mapper.

 

I have a value that is only true for some states, otherwise, its ignored. So here's the state-mapper:

https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/flowers/states/StateMapperFlowers.java

And the resulting blockstate files:

https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/resources/assets/oreflowers/blockstates/oreflowers1.json

https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/resources/assets/oreflowers/blockstates/oreflowers1_tall.json

 

Notice all of the "oreflowers:items/na" entries? Those will never ever be seen. I know that, but Minecraft doesn't, and so rather than specifying a bunch of state pairs that'll never be used, I used the one value to split the states out into two different files.

 

You'd do the same thing on your FEMALE value, and the two files will be 99% the same, but one will have the "stage 7" be different.

 

Don't forget to register your custom state mapper on the client.

Edited April 27, 2018 by Draco18s

[cmchenry]

45 minutes ago, Draco18s said:

I am not sure how the multistate format works, so I'm going to suggest going the rout of a custom state mapper.

 

I have a value that is only true for some states, otherwise, its ignored. So here's the state-mapper:

https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/flowers/states/StateMapperFlowers.java

And the resulting blockstate files:

https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/resources/assets/oreflowers/blockstates/oreflowers1.json

https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/resources/assets/oreflowers/blockstates/oreflowers1_tall.json

 

Notice all of the "oreflowers:items/na" entries? Those will never ever be seen. I know that, but Minecraft doesn't, and so rather than specifying a bunch of state pairs that'll never be used, I used the one value to split the states out into two different files.

 

You'd do the same thing on your FEMALE value, and the two files will be 99% the same, but one will have the "stage 7" be different.

 

Don't forget to register your custom state mapper on the client.

Thanks for the respond! I think this will help a lot of people, but the issue was storing the correct metadata with storing two values in an integer. Here is the code for that.

 

@Override
public int getMetaFromState(IBlockState state) {
    int i = 0;
    i = i | state.getValue(FEMALE);
    i = i | state.getValue(AGE) << 1;
    return i;
}

@Override
public IBlockState getStateFromMeta(int meta) {
    int female = meta & 1;
    int age = Integer.valueOf((meta & 15) >> 1);
    return withAge(age).withProperty(FEMALE, female);
}

[Draco18s]

That would also do it