Jump to content

[1.18.1] Torch not dropping on break and not quite rendering properly


Mattah12

Recommended Posts

So I have two custom torches, both registered as below:

public static final RegistryObject<Block> LIGHTSTONE_TORCH = BLOCKS.register("lightstone_torch", () -> new TorchBlock(BlockBehaviour.Properties.of(Material.DECORATION)
            .noCollission().noOcclusion().instabreak().lightLevel((p_50886_) -> {return 10;})
            .sound(SoundType.GLASS), ParticleTypes.FLAME));
public static final RegistryObject<Block> WALL_LIGHTSTONE_TORCH = BLOCKS.register ("wall_lightstone_torch", () -> new WallTorchBlock((BlockBehaviour.Properties.of(Material.DECORATION)
            .noCollission().instabreak().lightLevel((p_152607_) -> {return 10;})
            .sound(SoundType.GLASS).dropsLike(Registration.LIGHTSTONE_TORCH.get())), ParticleTypes.FLAME)); public static final RegistryObject<StandingAndWallBlockItem> LIGHTSTONE_TORCH_ITEM = ITEMS.register("lightstone_torch", () ->
            new StandingAndWallBlockItem(Registration.LIGHTSTONE_TORCH.get(), Registration.WALL_LIGHTSTONE_TORCH.get(),
                    new Item.Properties().tab(ModSetup.ITEM_GROUP)));

I was originally erroneously using createSimpleTable, under the assumption that torches in part act like blocks. Obviously that did not work so I am trying the below method:

protected LootTable.Builder createItemTable(String name, Item item) {
        LootPool.Builder builder = LootPool.lootPool()
                .name(name)
                .setRolls(ConstantValue.exactly(1))
                .add(LootItem.lootTableItem(item));
        return LootTable.lootTable().withPool(builder);
    }

This doesn't work either so I'm now a bit stumped. Full repo is here https://github.com/Mattah12/Kanohi_Craft2022/tree/master/src/main/java/com/Mattah12/kanohicraft

 

My other issue is that the torches are rendering as in the linked pic, and not solid like a vanilla torch. https://imgur.com/a/Fo7TfzW

Link to comment
Share on other sites

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

For the torch rendering, not really my area of expertise.

But you are using RenderType.translucent() while vanilla uses RenderType.cutout()

https://github.com/Mattah12/Kanohi_Craft2022/blob/62dac5df0ba4f7e5e381ef78e01050721ed2dcd8/src/main/java/com/Mattah12/kanohicraft/setup/ClientSetup.java#L21

Edited by warjort

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

28 minutes ago, warjort said:

Not sure why you're not seeing my most up-to-dat repo, but I'm now using:

createItemTable("lightstone_torch", Registration.LIGHTSTONE_TORCH_ITEM.get());
createItemTable("lightstone_refined_torch", Registration.LIGHTSTONE_REFINED_TORCH_ITEM.get());
createItemTable("wall_lightstone_torch", Registration.LIGHTSTONE_TORCH_ITEM.get());
createItemTable("wall_lightstone_refined_torch", Registration.LIGHTSTONE_REFINED_TORCH_ITEM.get());



//AND


protected LootTable.Builder createItemTable(String name, Item item) {
        LootPool.Builder builder = LootPool.lootPool()
                .name(name)
                .setRolls(ConstantValue.exactly(1))
                .add(LootItem.lootTableItem(item));
        return LootTable.lootTable().withPool(builder);
    }

 

Link to comment
Share on other sites

I still don't see a lootTables.put() in the above code?

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

8 minutes ago, warjort said:

I still don't see a lootTables.put() in the above code?

You fixed both my issues, thanks. Working code in spoilers below should anyone else want to know.

Spoiler
lootTables.put(Registration.LIGHTSTONE_TORCH.get(), createItemTable("lightstone_torch", Registration.LIGHTSTONE_TORCH_ITEM.get()));
lootTables.put(Registration.LIGHTSTONE_REFINED_TORCH.get(), createItemTable("lightstone_refined_torch", Registration.LIGHTSTONE_REFINED_TORCH_ITEM.get()));
lootTables.put(Registration.WALL_LIGHTSTONE_TORCH.get(), createItemTable("wall_lightstone_torch", Registration.LIGHTSTONE_TORCH_ITEM.get()));
lootTables.put(Registration.WALL_LIGHTSTONE_REFINED_TORCH.get(),createItemTable("wall_lightstone_refined_torch",Registration.LIGHTSTONE_REFINED_TORCH_ITEM.get()));

 

 

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.



×
×
  • Create New...

Important Information

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