Jump to content

[1.8.9] FMLMissingMappingsEvent and missing blocks


ShetiPhian

Recommended Posts

I've gone through all my mods and changed their modid's to lowercase, and cleaned up all block and item registered names and made them lowercase.

Thankfully I've only got one that has a non-alpha/public release so this doesn't affect the rest.

 

I've used FMLMissingMappingsEvent to remap the id's, but it doesn't seem to be fully working.

 

There is no missing blocks/items message on world load.

The item and itemblock are being replaced, as they still exist in inventory.

The block however is deleted from the world.

 

The log doesn't have any errors and even appears to be working.

[FML]: Injecting existing block and item data into this server instance
[FML]: Found a missing id from the world endertanks:blockEnderTank
[FML]: Found a missing id from the world endertanks:blockEnderTank
[FML]: Found a missing id from the world endertanks:itemEnderBucket
[endertanks]: Block Remap: endertanks:blockEnderTank > endertanks:ender_tank
[endertanks]: ItemBlock Remap: endertanks:blockEnderTank > endertanks:ender_tank
[endertanks]: Item Remap: endertanks:itemEnderBucket > endertanks:ender_bucket
[FML]: Fixed block id mismatch endertanks:ender_tank: -1 (init) -> 198 (map).
[FML]: Fixed item id mismatch endertanks:ender_tank: -1 (init) -> 4096 (map).
[FML]: Fixed item id mismatch endertanks:ender_bucket: -1 (init) -> 4097 (map).

 

Here is the remap code, its in the main class of my mod (with preInit, init, etc.) and is being executed as seen by log output and partly working remap.

@Mod.EventHandler
public void fmlMissingMappingsEvent(FMLMissingMappingsEvent event)
{
    for (FMLMissingMappingsEvent.MissingMapping mapping : event.getAll()) {
        if (Values.blockEnderTank != null && mapping.name.equalsIgnoreCase("EnderTanks:blockEnderTank")) {
            if (mapping.type == GameRegistry.Type.BLOCK) {
                Values.logEnderTanks.info("Block Remap: " + mapping.name + " > " + Values.blockEnderTank.getRegistryName());
                mapping.remap(Values.blockEnderTank);
            } else {
                Values.logEnderTanks.info("ItemBlock Remap: " + mapping.name + " > " + Values.blockEnderTank.getRegistryName());
                mapping.remap(Item.getItemFromBlock(Values.blockEnderTank));
            }
            continue;
        }
        if (Values.itemEnderBucket != null && mapping.name.equalsIgnoreCase("EnderTanks:itemEnderBucket")) {
            if (mapping.type == GameRegistry.Type.ITEM) {
                Values.logEnderTanks.info("Item Remap: " + mapping.name + " > " + Values.itemEnderBucket.getRegistryName());
                mapping.remap(Values.itemEnderBucket);
            }
        }
    }
}

 

 

EDIT:

I think the problem has been located.

PersistentRegistryManager.injectSnapshot

After the missed mapping have been handled in 'if (injectFrozenData)'

 

The remapping is ignored and any missing block is set to BlockDummyAir

 

EDIT2:

Well either I'm missing something or my options are between two "wrongs"; use mixed case naming, or use reflection to correct the registry.

I don't want to be responsible for accidentally messing up the registry on someones world, so I'll stick with the mixed case naming.

Link to comment
Share on other sites

When a brand-spanking new mod is added to a brand-spanking new world, there shouldn't be any of its blocks in that world needing remapping, so why are you doing remapping? (There's something missing from your story that means "the world" to us. What are you trying to do???)

The debugger is a powerful and necessary tool in any IDE, so learn how to use it. You'll be able to tell us more and get better help here if you investigate your runtime problems in the debugger before posting.

Link to comment
Share on other sites

I never specified if the worlds where new or not because there is no point in remapping on a new world.

 

I'll expand on my previous statement of "Thankfully I've only got one that has a non-alpha/public release so this doesn't affect the rest."

One of my mods is publicly released and used by many, I don't want to mess up those existing worlds.

The rest are either private rebuilds or public alpha, so I don't care about those worlds.

 

 

The problem is the remap process is broken; even after a successful remap in world blocks are replaced with dummy air.

I have two options; edit the registry via reflection (I'm not willing to do that) or leave the mixed case id's.

 

The mod id's case is automatically made lowercase in the registry so changing that required no remaps, and the remap works correctly for items.

That just leaves only the block that is going to remain with the 'wrong' mixed case id.

Link to comment
Share on other sites

Indeed, you didn't say anything about worlds, so there were too many possibilities to guess at.

 

So you're changing names in a mod but not moving from one Forge (or MC) version to another. There's hope, or at least there should have been. If what you say about injectSnapshot is correct and block remappings are all being ignored, then there may be a bug in Forge that needs to be reported (though it may have been fixed since 1.8.9). You might want to check the change logs.

 

Speaking of later versions, it is generally hopeless to "upgrade" existing modded worlds to later MC versions of Minecraft (possibly because block remapping is broken), so you can posit that it simply shouldn't even be attempted. That frees you to rename things with impunity when porting your mods to newer MC versions -- only new worlds will run with the new naming. Therefore, make a note to self to complete your case-change renaming when you next upgrade this mod (e.g. to 1.9.2)

The debugger is a powerful and necessary tool in any IDE, so learn how to use it. You'll be able to tell us more and get better help here if you investigate your runtime problems in the debugger before posting.

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

    • They were already updated, and just to double check I even did a cleanup and fresh update from that same page. I'm quite sure drivers are not the problem here. 
    • i tried downloading the drivers but it says no AMD graphics hardware has been detected    
    • Update your AMD/ATI drivers - get the drivers from their website - do not update via system  
    • As the title says i keep on crashing on forge 1.20.1 even without any mods downloaded, i have the latest drivers (nvidia) and vanilla minecraft works perfectly fine for me logs: https://pastebin.com/5UR01yG9
    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
  • Topics

×
×
  • Create New...

Important Information

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