Crafting menu not opening on right click after update to vanilla crafting table

[Capricocious]

Hi, I'm creating a mod that involved reskinning the default crafting table to look like a real table, I have re-registered the item with the method .noOcclusion() so you can't see the rest of the world under the table legs:

public static final RegistryObject<Block> CraftingTableBlock = VANILLA_BLOCKS.register("crafting_table",
() -> new Block(BlockBehaviour.Properties.of(Material.WOOD).destroyTime(5).sound(SoundType.WOOD).noOcclusion()));

However, on right-click the table no longer brings up the crafting menu. I've had to re-register the crafting table block item to ensure it shows up in the creative menu. I have looked in the java file for the craftingTableBlock and found the use and getMenuProvider methods, but I'm not sure how to call them. Is it a similar task I need to do to re-register the crafting menu so it can be opened? If anyone could provide some guidance it would be much appreciated <3

Minecraft version: 1.19.2
Forge version: 43.1.7

[warjort]

You can't just replace the crafting table. Look at the CraftingMenu.stillValid() check

or see for example: https://forums.minecraftforge.net/topic/114789-best-way-to-create-a-custom-crafting-table-in-119/?do=findComment&comment=508881

 

In general it is a bad idea to replace vanilla objects in the registries.

You will just create conflicts with other mods trying to do the same thing or run into code where Mojang hard wires things.

 

What you should do is create your own block and let users or modpack developers decide whether to use your crafting table.

e.g. they can change the recipe so it crafts your block instead of the vanilla one using a datapack.

[Capricocious]

Hi warjort, thanks for the input!

Is there a better way to turn off culling for blocks surrounding the vanilla crafting table rather than trying to overwrite the registries? I'd like to just use a resource pack to reskin it if I can, but I wasn't able to find a way to stop the faces of the surrounding blocks from culling.

Or is it most sensible to just create my table as a new block?