Draco18s

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)...

Because these are not the same: public void onArmorTick(World world, LivingEntity entity, ItemStack itemStack) { public void onArmorTick(ItemStack itemStack, World world, PlayerEntity entity) {

It is one of YOUR data files that is causing problems. It should be mentioned in the log. But if you can't find it, remove all of your assets inside /data and put them back in binary chunks (add half of the most recently removed if the problem isn't exhibited, remove half of what was most recently added if it does) and narrow it down.

One of your data files is bad and it caused everything to be discarded.