Jump to content

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


Daniel Rollins

Recommended Posts

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

Link to comment
Share on other sites

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.

  • Thanks 1
Link to comment
Share on other sites

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

Link to comment
Share on other sites

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!

Link to comment
Share on other sites

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...

Link to comment
Share on other sites

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 by ItsWormy
Link to comment
Share on other sites

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

Link to comment
Share on other sites

4 hours ago, ItsWormy said:

without making custom classes

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

About Me

Spoiler

My Discord - Cadiboo#8887

My WebsiteCadiboo.github.io

My ModsCadiboo.github.io/projects

My TutorialsCadiboo.github.io/tutorials

Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support.

When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible.

Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org

Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

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