[1.14.3] Gui Containers

[Draco18s]

I'm trying to figure out how to set up gui containers so that the client side has a reference to the TE in question so that it can get things like the current cook time.

 

Due to all the obfuscation behind factories and container types, I can't figure out how on earth the Furnace (and like classes) actually get injected into the Container used on the client side.

 

For example, the Furnace gets its GuiContainer constructed using a Container that calls a constructor that does this:

 

this(containerTypeIn, recipeTypeIn, id, playerInventoryIn, new Inventory(3), new IntArray(4));

 

That new Inventory(3) is what gets referenced on the client side later on as the Furnace inventory and the new IntArray(4) is used as field_217064_e, referenced in getCookProgressionScaled(). But it's a blank array! It never gets filled with any values as far as I can find.

 

Going a step further it appears that furnaces are just plain broken (they don't smelt, at all) in the dev environment. Can anyone confirm this?

[Animefan8888]

10 minutes ago, Draco18s said:

Going a step further it appears that furnaces are just plain broken (they don't smelt, at all) in the dev environment. Can anyone confirm this?

Gonna check this now.

 

10 minutes ago, Draco18s said:

I'm trying to figure out how to set up gui containers so that the client side has a reference to the TE in question so that it can get things like the current cook time.

Yeah the way it is kinda seems weird to me, but I figured it out. So you need an INamedContainerProvider which can have a constructor. To open a gui you use one of the NetworkHooks.openGui which has three options all of which take in a ServerPlayer and a INamedContainerProvider. One takes in a BlockPos and another takes in a Consumer<PacketBuffer>.

 

And there is this example on the github.

[Draco18s]

7 minutes ago, Animefan8888 said:

Yeah the way it is kinda seems weird to me, but I figured it out. So you need an INamedContainerProvider which can have a constructor. To open a gui you use one of the NetworkHooks.openGui which has three options all of which take in a ServerPlayer and a INamedContainerProvider. One takes in a BlockPos and another takes in a Consumer<PacketBuffer>.

I have that. I extended the interface, actually, but my TE still implements it. createMenu, getDisplayName, and openGUI. All the vanilla TEs implement the interface directly.

 

I'm already using the NetworkHooks.openGui that takes in the player, container, and blockpos.

 

But I don't see where I get that blockpos back on the client end of things.

[Animefan8888]

28 minutes ago, Draco18s said:

Going a step further it appears that furnaces are just plain broken (they don't smelt, at all) in the dev environment. Can anyone confirm this?

I can confirm that they work in the latest version of forge  1.14. So you may want to update.

3 minutes ago, Draco18s said:

But I don't see where I get that blockpos back on the client end of things.

There might be some behind the scenes magic with registering the ContainerType you should use IForgeContainerType.create

[Draco18s]

20 minutes ago, Animefan8888 said:

I can confirm that they work in the latest version of forge  1.14. So you may want to update.

1.14.3 or 1.14.4?

20 minutes ago, Animefan8888 said:

There might be some behind the scenes magic with registering the ContainerType you should use IForgeContainerType.create

And how do you register the class as an IForgeContainerType?

[Animefan8888]

7 minutes ago, Draco18s said:

1.14.3 or 1.14.4?

1.14.4 I updated when it first came out.

 

7 minutes ago, Draco18s said:

And how do you register the class as an IForgeContainerType?

ContainerTypes are a registry.

[Draco18s]

3 minutes ago, Animefan8888 said:

1.14.4 I updated when it first came out.

Noted. I'll give that a try.

Quote

ContainerTypes are a registry.

Yes. I know. That's not what I am having a problem with.

ContainerType's constructor requires an IFactory parameter (which defines a create method), not an IForgeContainerType (which defines a different create method).

ContainerType is already an IForgeContainerType, so either:

a) I'm already doing it right and something else is wrong

b) I'm doing it wrong

What do I need to do to get the IForgeContainerType result?

Edited July 28, 2019 by Draco18s

[Animefan8888]

6 minutes ago, Draco18s said:

a) I'm already doing it right and something else is wrong

b) I'm doing it wrong

I'm not sure do you have your code on your github so I can look.

[Draco18s]

5 minutes ago, Animefan8888 said:

I'm not sure do you have your code on your github so I can look.

Will be here momentarily (need to push changes, will take a few minutes)

https://github.com/Draco18s/ReasonableRealism/blob/1.14.3/src/main/java/com/draco18s/harderores/HarderOres.java#L102

Done

Edited July 28, 2019 by Draco18s

[Animefan8888]

16 minutes ago, Draco18s said:

b) I'm doing it wrong

EasyRegistry.registerOther(new ContainerType<>(SifterContainer::new), new ResourceLocation(HarderOres.MODID,"sifter"));

Instead of new ContainerType<?> do IForgeContainerType.create(SifterContainer::new)

 

And then in your SifterContainer have a constructor with a PacketBuffer parameter. I think it gets passed in there. And then you can send whatever info you want to the container.

[Draco18s]

I swear I tried doing that and Eclipse wouldn't let me. I probably inferred something wrong and the autocompletion was (correctly, but unhelpfully) not suggesting things or otherwise complaining.