[1.13.2] Set block harvest level and tool without overriding functions

[Daniel Rollins]

Hi, is there a way to set the block harvest level and tool without having to override the getHarvestTool and getHarvestLevel functions? Basically, how can I set the internal harvestTool and harvestLevel variables? Preferably I would like to be able to set them without putting them in the custom block classes. In my mod I tried to have less actual block classes by basically doing this:

BlockList.new_block1 = new Block(Block.Properties.create(Material.IRON).hardnessAndResistance(20.0F, 27.0F).lightValue(0).sound(SoundType.METAL)).setRegistryName(location("new_block1")) # Using vanilla block class

BlockList.new_block2 = new Block(Block.Properties.create(Material.IRON).hardnessAndResistance(5.0F, 3.0F).lightValue(0).sound(SoundType.METAL)).setRegistryName(location("new_block2")) # Using vanilla block class

or

BlockList.custom_block1 = new CustomBlock(Block.Properties.create(Material.IRON).hardnessAndResistance(5.0F, 1.0F).lightValue(15).sound(SoundType.GLASS)).setRegistryName(location("custom_block1")) # Using custom block class

BlockList.custom_block2= new CustomBlock(Block.Properties.create(Material.IRON).hardnessAndResistance(30.0F, 15.0F).lightValue(0).sound(SoundType.GLASS)).setRegistryName(location("custom_block2")) # Using same custom block class

 

Is there a way to set the harvest info similar to how I set the hardness and resistance so I don't have to create custom classes just for different harvest settings?

 

Thanks

[V0idWa1k3r]

6 minutes ago, Daniel Rollins said:

Is there a way to set the harvest info similar to how I set the hardness and resistance so I don't have to create custom classes just for different harvest settings?

 

No. In this case a custom block class is acceptable.

 

I mean, the fields are technically not final, just private, so you could use reflection but why?

see https://github.com/MinecraftForge/MinecraftForge/issues/5560 for reasons as to why there isn't a setter.

[Daniel Rollins]

Thank you for your reply. If I have to create more classes then I guess I have to but I was wondering if there was anything I could do with this (found in Block Class):

static {
      net.minecraftforge.common.ForgeHooks.setBlockToolSetter((block, tool, level) -> {
         block.harvestTool = tool;
         block.harvestLevel = level;
      });

 

I'm not super great with Java yet so I'm not sure how I would use this or if it would be easier to just create the extra classes.

 

Thanks

[ItsWormy]

I am only coding for version 1.12.2 and the net.minecraftforge.common.ForgeHooks.setBlockToolSetter() does not seem to exist, so I cannot help you with that. But I do have some alternatives for setting the harvest level without making custom classes (that may or may not work).

First Option:

// First Option
Block option1 = new Block(Material.STONE) {
    @Override
    public int getHarvestLevel(IBlockState state) {
      //Return whatever
    }
};

Second Option:

// Second Option (might not work, never tried this before)
Block option2 = new Block(Material.STONE) {
    private int[] harvestLevel = new int[] {/* Your values here */};
};

Third Option:

// Third Option (might be 1.12.2 only)
Block option3 = new Block(Material.STONE).setHarvestLevel(String, int/*, IBlockState*/);

 

I hope one of these works!

[V0idWa1k3r]

17 minutes ago, ItsWormy said:

I am only coding for version 1.12.2 and the net.minecraftforge.common.ForgeHooks.setBlockToolSetter() does not seem to exist, so I cannot help you with that.

The OP stated that they are using 1.13.2 in the title.

 

17 minutes ago, ItsWormy said:

First Option:

18 minutes ago, ItsWormy said:

without making custom classes

You are making a custom class in your first option...

 

18 minutes ago, ItsWormy said:

Second Option:

Yeah, this does nothing.

 

18 minutes ago, ItsWormy said:

Third Option:

This is 1.12 only.

 

1 hour ago, Daniel Rollins said:

Thank you for your reply. If I have to create more classes then I guess I have to but I was wondering if there was anything I could do with this (found in Block Class):

This is an internal forge hook for setting harvest tool/level for vanilla blocks. The method even says that modders shouldn't use it so...

[ItsWormy]

14 minutes ago, V0idWa1k3r said:

The OP stated that they are using 1.13.2 in the title.

 

That is why I stated that I used 1.12.2

14 minutes ago, V0idWa1k3r said:

This is 1.12 only.

And I said that it might only exist for 1.12.2

 

Edit: I thank you for the feedback about the other options though, I had no idea what the second one would do. And I had no idea that the first option was considered making a new custom class.

Edited May 9, 2019 by ItsWormy

[Daniel Rollins]

2 hours ago, V0idWa1k3r said:

This is an internal forge hook for setting harvest tool/level for vanilla blocks. The method even says that modders shouldn't use it so...

 

Sorry, didn't see that. I'll just go with more classes and using the getHarvestLevel & getHarvestTool methods. Thank you

[Cadiboo]

4 hours ago, ItsWormy said:

without making custom classes

Options 1 & 2 are making new anonymous classes. Anonymous classes are custom classes