Jump to content

Getting panes to show up differently in inventory depending on metadata


Lunakki

Recommended Posts

I created colored glass panes by extending the BlockPane class. All the colors share the same blockID but display different textures depending on their metadata, similar to wool. This works great except that the inventory, handheld item, and object on ground all use the texture for metadata 0. With some playing around, it seems to be a result of the getRenderType() returning 18 (aka pane). I looked through the function renderBlockAsItem in RenderBlocks, but it looks like it uses the function that returns a texture based on the metadata, so I don't know why it's not working. It appears that the metadata it's using for that function is passed into from somewhere else, and presumably it's correct or things like wool would not render correctly in the inventory, but yet it doesn't work for my panes...

 

My objects all use the correct textures when placed, and they use the right textures in the inventory when using a different render type, so I'm pretty sure it's not me returning the wrong texture or anything like that.

 

I can get around it by creating individual objects for each color, but I'd really rather not do it that way if I can help it. Any ideas on a more elegant solution?

Link to comment
Share on other sites

tl;dr:

 

I need to override a function in GlassPane that is declared public final. Removing the final label in GlassPane.java lets my game run exactly as I want it to. Is there some way to do this that won't interfere with other mods that may want to edit the same class?

 

Long version which may give you an idea of a solution that I haven't thought of:

 

I changed my panes to all be individual items so that they did not have to use metadata. To do this, I had to make a new class that extended Block instead of BlockPane. This is because I need them to connect to each other, and the canThisPaneConnectToThisBlockID cannot be overridden because it's final in BlockPane (why???) and it checks for opaque blocks and certain block IDs (including its own) to see if it can connect.  Since my panes no longer all have the same blockID, they no longer connect with that method.

 

It all looked good until I tried to place the block. Then I got this error:

Unexpected error

 

java.lang.ClassCastException: krv.minecraft.moreblocks.BlockThinTintedGlass cannot be cast to net.minecraft.block.BlockPane

 

So now it can render the inventory icon correctly, but not the actual block.  After looking at the rendering code, sure enough it requires a BlockPane to render as a pane, which my block obviously isn't.

 

This would be easy to fix if I could just override that one function in BlockPane, because then I could just extend that class and it would all work. This is the function I need to override:

 

    public final boolean canThisPaneConnectToThisBlockID(int par1)
    {
        return Block.opaqueCubeLookup[par1] || par1 == this.blockID || par1 == Block.glass.blockID;
    }

 

I want it to be this in my class:

 

    public boolean canThisPaneConnectToThisBlockID(int id)
    {
        for (int i = 0; i < 16; i++) {
        	if (id == MoreBlocks.thinTintedGlass[i].blockID)
        		return true;
        }
    	return Block.opaqueCubeLookup[id] || id == Block.glass.blockID || id == MoreBlocks.tintedGlass.blockID;
    }

 

What are my options here without actually editing any of the core files? Do I have any?

Link to comment
Share on other sites

  • 11 months later...

What I resorted to was having people overwrite the BlockPane class file with my own version when installing my mod. This is obviously not an ideal solution, especially if you are distributing it to a lot of people. Fortunately my users are limited to my real life friends, so I can install it for them if they run into issues.

 

There is an alternate method using access transformers, but I could never get it to work. Here is the tutorial I was working off of: http://www.minecraftforge.net/wiki/Using_Access_Transformers

 

I also tried using Reflection but was unable to get that to work as well. Here's a tutorial I tried for that (looks outdated now): http://www.minecraftforum.net/topic/1854988-tutorial-162-changing-vanilla-without-editing-base-classes-coremods-and-events-very-advanced/

 

Here's a general Java Reflection tutorial: http://docs.oracle.com/javase/tutorial/reflect/

 

I imagine it's probably unnecessary in the current version since there are now colored panes in vanilla, but I haven't had the time to look into it myself yet. Let me know what you find out.

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

    • So I saw that mixin is shipped as a library with forge, but is it available for 1.7.10 ?  
    • So I've read the EULA, and lets be straight...     If I split my modpack(of my mods, yeah I'm nuts) into several(many) individual mods(like just one boss) with minor additions(plus not working together), then have a complete/modpack version on patreon/onlyfans having each addon work together... Would people buy my idea?
    • German A1 – C1, TestDAF, Goethe B1, B2, C1, C2, valid GOETHE certificate German A1 – C1, TestDAF, Goethe B1, B2, C1, C2, valid GOETHE certificate(+27(838-80-8170
    • Done, it still crashed. New log https://paste.ee/p/kYv6e
    • I am migrating a mod from 1.16.5 to 1.20.2 The version for 1.16.5 can be found here https://github.com/beothorn/automataCraft For the block called automata_start, it uses TileEntities and has blockstates, model/block and textures on json files. This is currently working fine on 1.16.5 https://github.com/beothorn/automataCraft/tree/master/src/main/resources/assets/automata For 1.20.2 I migrated the logic from TileEntities to BlockEntity. The mod is working fine. All blocks and Items are working with the correct textures except for the textures for each state of the automata_start block. No changes where made to the json files. This is the branch I am working on (there were some refactorings, but all is basically the same): https://github.com/beothorn/automataCraft/tree/1_20/src/main/resources/assets/automata The only difference I can think that may be related is that i had to implement createBlockStateDefinition on the BaseEntityBlock: https://github.com/beothorn/automataCraft/blob/1_20/src/main/java/br/com/isageek/automata/automata/AutomataStartBlock.java#L43 This is driving me crazy. I know the jsons are being loaded as I put a breakpoint at `net.minecraft.client.resources.model.ModelBakery#loadModel` and I can see BlockModelDefinition.fromJsonElement being called with automata_start. I also printed the state from the arguments of the tick function call and they look correct (https://github.com/beothorn/automataCraft/blob/1_20/src/main/java/br/com/isageek/automata/automata/Ticker.java#L32 ): blockState Block{automata:automata_start}[state=loadreplaceables] In game, all I see is the no textures. I think it is weird it is not the "missing texture" texture so I think it may be related to the material, but I had no success tweaking it (https://github.com/beothorn/automataCraft/blob/1_20/src/main/java/br/com/isageek/automata/automata/AutomataStartBlock.java#L37).   public static final Property<AutomataStartState> state = EnumProperty.create("state", AutomataStartState.class); private final AtomicReference<RegistryObject<BlockEntityType<?>>> blockEntityType; private final Map<String, RegistryObject<Block>> registeredBlocks; public AutomataStartBlock( final AtomicReference<RegistryObject<BlockEntityType<?>>> blockEntityType, final Map<String, RegistryObject<Block>> registeredBlocks ) { super(BlockBehaviour.Properties.of().mapColor(MapColor.STONE).strength(1.5F, 6.0F)); this.blockEntityType = blockEntityType; this.registeredBlocks = registeredBlocks; this.registerDefaultState(this.getStateDefinition().any().setValue(state, AutomataStartState.LOAD_REPLACEABLES)); } @Override protected void createBlockStateDefinition(StateDefinition.Builder<Block, BlockState> stateBuilder) { stateBuilder.add(state); }     So my cry for help is, anyone has any ideas? Is there a way to easily debug this, for example somewhere where I can list the textures for a given state, or make sure this is loaded?   Thanks in advance for the hints
  • Topics

×
×
  • Create New...

Important Information

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