Draco18s

You don't show where this tileentity is coming from, but odds are either it doesn't exist (is null) or isn't a tileentity.

Get stack Insert item Set stack (probably not clean code, however; looking at it made me wonder if I could have written it better) Extract item Get slot limit

Why are you overriding a function that you do nothing but call the existing implementation?

Well, yes. Because vanilla did things in a way that aren't extendable. I'm not debating that. I'm saying that the method is returning a sensible value and that it's the wrong thing to look at.

Items, in general, by default, are not furnace fuel. If an item is fuel, it will override that method.

You don't need to sync very much or very often. The client, provided that the amount of fuel has been synced as the result of an inventory change, can calculate the RF conversion process all on its own using the exact same code as the server side tile entity. There is no point or need to sync the progress.

Where is the packet coming from? Why aren't you writing code to directly give the player an item?

TiC is open source.

https://minecraft.gamepedia.com/Tag Those are items, not tags. If you want to use items, you need to use "item". If you want to use tags, you need to pass a valid tag.

That's the advancement that grants the recipe, not the recipe.

This does not require code, all you have to do is create a loot table JSON file and place it in the correct directory.

Try any vanilla block and flowing water. None of them function with it either.

This is because the "waterlogged" system isn't finished.

As an example: https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderores/client/ClientEventHandler.java#L19 A "factory" is just a method that returns a given type, in this case your GuiScreen. Most of the time you will be able to do this through a 0-parameter constructor.

Yes, that is valid. But my point was that's not how (generic) you do the thing (generic) you are trying to do right now. Which I found by searching this (which is in a banner at the top of every page here on the forums) for "container"

Vanilla code looks like that because the decompiler isn't smart enough to know the difference (5 and Integer.valueOf(5) compile to the same byte code).

Looks similar to what I do, check out my model and block: https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/resources/assets/harderores/models/block/ore_hardiron_1.json https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderores/block/ore/HardOreBlock.java

This is not how you open GUIs.

oh christ, why all this boxing? Do you want an int or a bool? Use the value you want, done. with(FILLED, false). with(STAGE, 0). Also, nothing in your code ever sets FILLED to true.

/me does not see updated code or an updated error message

IHasModel is completely and utterly pointless. All it does is make you write the same 4 lines of code all over again for every item so that you can pass public values to a public function.

The only constructor that SwordItem has takes parameters, you aren't giving it those parameters, go look at the class to see what parameters it wants.

Or as a visual aid: register(new TestDoor(Material.WOOD)).setHardness(... vs: register(new TestOre().setHardness(3.0F)...