Jump to content

[1.16.5]Overriding vanilla Block doesn't work as expected


Cratthorax

Recommended Posts

Good Morning,

I tried overriding one of the vanilla rock blocks according to instructions in this old thread, but failed for the same reason the org TM did, even though I got a tiny bit farther than he did.

I can remove the original block entirely from world spawn, and replace it with custom version, which I want to prevent due to following up workload(adapting recipes etc. to custom block).

What I tried solving the issue/actually making replacing(partially) work:

1. I had to use the original Forge variant of registering the block in main class, like that:

which actually did use the custom granite block as replacer(not exactly illustrated in the old thread), but didn't consider it on world feature generation...

 
    @Mod.EventBusSubscriber(bus=Mod.EventBusSubscriber.Bus.MOD)
    public static class RegistryEvents {
        
        @SubscribeEvent
        public static void onBlocksRegistry(final RegistryEvent.Register<Block> blockRegEvt) {
            blockRegEvt.getRegistry().register(new BlockGranite(null).setRegistryName(new ResourceLocation("minecraft", "granite")));
        }
        
    }

 

2. I then believed it was due to not using correct path variables for assets texture/models/blockstates etc., so I created missing asset packets for both, vanilla asset path, and my mod path. This also did not fix the issue with creating holes filled up with Items.AIR on world feature generation.

3. I then tried to remove the vanilla granite block during feature generation, and reimported it after with my replacement set up, during feature generation as well(according to @Luis_ST advice). But the holes were still there, only this time I got my replaced granite block spawned as well.

What do I want to achieve(asking myself and you in hope maybe there's a better solution)?

Mojang found it a good idea, to give one of the hardest rock materials on this planet, the .harvestLevel(soft as a pillow), so I can harvest it with a wooden pickaxes(which technically is even worth, but doesn't matter since wooden "pickaxe" tools do not exist in my mod). I really just want to compare harvestLevel of any of my tools, against the harvestLevel of the targetBlock, so I can automatically disallow harvesting it, if the tool doesn't match the harvestLevel(which for whatever reason does not work as intended in 1.16.5 Minecraft).

Edit: and before you shout "doPlayerHarvestCheck". Yes, I do that!

 
    @SubscribeEvent
    public static void doPlayerHarvestCheck(PlayerEvent.HarvestCheck event) {
        
       ...
            
        int toolLevel = itemRef.getHarvestLevel(heldItems, blockTool, playerRef, blockState);
    	int blockLevel = targetBlock.getHarvestLevel(blockState);
       
            if((toolLevel < blockLevel) && (toolLevel >= 0)) {
                event.setCanHarvest(false);
                playerRef.sendStatusMessage(new TranslationTextComponent("This Material is to hard for your Tool!"), true);
            } else {
                event.setCanHarvest(true);
            }
        }
    }

 

Edit2: Also, why are there Manga Comic Girls, on the right upper corner of my Monitor, stating how can it be that big?(whatever "it" is?)

Edited by Cratthorax
Link to comment
Share on other sites

2 hours ago, Cratthorax said:

3. I then tried to remove the vanilla granite block during feature generation, and reimported it after with my replacement set up, during feature generation as well(according to @Luis_ST advice). But the holes were still there, only this time I got my replaced granite block spawned as well.

What do I want to achieve(asking myself and you in hope maybe there's a better solution)?

show the code your use for this

2 hours ago, Cratthorax said:

Mojang found it a good idea, to give one of the hardest rock materials on this planet, the .harvestLevel(soft as a pillow), so I can harvest it with a wooden pickaxes(which technically is even worth, but doesn't matter since wooden "pickaxe" tools do not exist in my mod). I really just want to compare harvestLevel of any of my tools, against the harvestLevel of the targetBlock, so I can automatically disallow harvesting it, if the tool doesn't match the harvestLevel(which for whatever reason does not work as intended in 1.16.5 Minecraft).

Edit: and before you shout "doPlayerHarvestCheck". Yes, I do that!

is the Event called?, use debugger to check the values in the condition

  • Like 1
Link to comment
Share on other sites

The final solution was to learn how to register and use a loot serializer, like this:

 
        @SubscribeEvent
        public static void onModifierSerializersRegistry(final RegistryEvent.Register<GlobalLootModifierSerializer<?>> evt) {
            evt.getRegistry().register(new RelEqtLootModifier.Serializer().setRegistryName(new ResourceLocation(MODID, "stone_replace")));
        }   

 

...and then extend on LootModifier like this:

 
	public class RelEqtLootModifier extends LootModifier {
    
    private final int numDropsToConvert;
    private final Item itemToCheck;
    private final Item itemReward;
    
    public RelEqtLootModifier(ILootCondition[] conditionsIn, int numDrops, Item itemCheck, Item reward) {
        super(conditionsIn);
        numDropsToConvert = numDrops;
        itemToCheck = itemCheck;
        itemReward = reward;
    }
	    @Nonnull
    @Override
    public List<ItemStack> doApply(List<ItemStack> generatedLoot, LootContext context) {
	        int numDrops = 0;
        for (ItemStack stack : generatedLoot) {
            if (stack.getItem() == itemToCheck)
                numDrops += stack.getCount();
        }
        if (numDrops >= numDropsToConvert) {
            generatedLoot.removeIf(x -> x.getItem() == itemToCheck);
            generatedLoot.add(new ItemStack(itemReward, (numDrops / numDropsToConvert)));
            numDrops = numDrops % numDropsToConvert;
            if (numDrops > 0)
                generatedLoot.add(new ItemStack(itemToCheck, numDrops));
        }
        return generatedLoot;
    }
	    public static class Serializer extends GlobalLootModifierSerializer<RelEqtLootModifier> {
	        @Override
        public RelEqtLootModifier read(ResourceLocation name, JsonObject object, ILootCondition[] conditionsIn) {
            int numDrops = JSONUtils.getInt(object, "numDrops");
            Item drop = ForgeRegistries.ITEMS.getValue(new ResourceLocation((JSONUtils.getString(object, "dropItem"))));
            Item stone = ForgeRegistries.ITEMS.getValue(new ResourceLocation(JSONUtils.getString(object, "replacement")));
            return new RelEqtLootModifier(conditionsIn, numDrops, drop, stone);
        }
	        @Override
        public JsonObject write(RelEqtLootModifier instance) {
            JsonObject json = makeConditions(instance.conditions);
            json.addProperty("numDrops", instance.numDropsToConvert);
            json.addProperty("dropItem", ForgeRegistries.ITEMS.getKey(instance.itemToCheck).toString());
            json.addProperty("replacement", ForgeRegistries.ITEMS.getKey(instance.itemReward).toString());
            return json;
        }
    }
}

 

The registration happens on the Mod bus(not Forge bus), in your main class's RegistryEvents.

I still can't get over the inconvenience, how cryptic the Forge Docs uses to be though.

Edited by Cratthorax
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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • In the rapidly evolving landscape of cryptocurrency, scams are on the rise, leaving many investors vulnerable to fraudulent schemes. If you've found yourself a victim of a crypto scam, navigating the complex process of recovering your lost funds can be daunting and overwhelming. However, there are some professionals in cryptocurrency recovery, such as Captain WebGenesis, who have specific knowledge and experience in handling complex blockchain technology and comprehending how scammers function in the cryptocurrency space. Captain WebGenesis has extensive expertise helping people who have been duped by investment scams to effectively retrieve their stolen or lost cryptocurrency.  
    • Looking for a semi-vanilla Minecraft server? Maybe one with keep inventory and land claims? I have one for you! Let me introduce you to Safe Survival! Come on and join us at mc.safesurvival.net. It is bedrock supported too!
    • Hi. I'm trying to play RLCraft with Optifine but my game keeps crashing on startup with Exit Code 1. The modpack was working fine yesterday, not sure what changed to make it not wanna work anymore. I was hoping someone here could help me figure out what's going wrong. If I remove Optifine, nothing changes, I still get the same errors. Thanks in advance. Crash Report (I only included the errors+fatal, I’m not sure if more is needed, please let me know): https://pastebin.ai/66ivknz0f4  
    • I am trying to port my mod from 1.20.2 to 1.21, but I am encountering difficulties regarding the classes for rendering the GUI and related events. What are the equivalents of the following classes? net.minecraftforge.client.event.RenderGuiOverlayEvent net.minecraftforge.client.gui.overlay.NamedGuiOverlay net.minecraftforge.client.gui.overlay.ForgeGui net.minecraftforge.client.gui.overlay.GuiOverlayManager net.minecraftforge.client.gui.overlay.IGuiOverlay   thx
    • Hi all! Love the Custom npc mod, it is my favorite mod! That said, I have been using this mod since finding it a few years ago (I only play on 1.12.2) and instantly loved that I caould reskin all of the npc's I created. That worked great until recently. I got locked out of my outlook account, which is what I was logged into windows with at the time, and they locked me out of Windows 10. So, I had to reinstall, and since I have digital entitlements to Windows 10 ( I am part of the insider program) on all three of my outlook accounts, I ust swapped to the one I bought Minecraft for. Reinstalled everything, started MC, crerated my favorite fighter (set to kill everything) to clear an area) set up the stats, went back to the main screen, clicked skin so I can give him the grim reaper skin and crash . . . wtf? Thought it was another mod, so, over the course of the last 2 days, I removed every single mod, reinstalled forge 2860 and then 2859, replaced optifine G5 with G6 and vice versa, all with the same result. So, I created a new world, with only the two mods, and same thing. https://pastebin.com/8raJVHq1 That is the log, I have been through the visual settings and all sorts of stuff and can not figure this out, and being that I only just started a bachelors in software engineer / computer programming, I have no idea what I am looking at, so please, and thank you!  If any one of you know what is causing this, I will be forever grateful for the help! Mor9heus
  • Topics

×
×
  • Create New...

Important Information

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