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?

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, 20187 yr by Draco18s

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);
}

That would also do it