Draco18s

onEntityCollission is only called for blocks that the player can actually be inside of, so blocks that take up the full 1x1x1 cube (and are solid) have no space for the player to actually "enter" it.

Seriously, search first. https://www.minecraftforge.net/forum/search/?q=deprecated&type=forums_topic&updated_after=any&sortby=relevancy&search_in=titles

My ores aren't "ores" either: https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderores/block/ore/HardOreBlock.java#L40

I don't see any code, or what line number that the array bounds exception occurred at. So. I'm not really sure how to help you. (Also, why do people think that the last thing the game spits out, "finished with non-zero exit value -1," is the actual problem? All that line says is "The program Java was running had a problem. Check the log.")

This is useless and pointless. All items need models.

From https://gist.github.com/williewillus/353c872bcf1a6ace9921189f6100d09a Loader.isModLoaded => ModList.get().isLoaded()

That's my thought, too.

Update. 1.12 is no longer supported.

Read the docs on events. @SubscribeEvent is only part of what you need in order to get events (you should also use a dedicated event handler class). But yes, 1.12 is no longer supported.

Contact the author of the pack, or the author(s) of the mods. Forge's team can't help here.

Point 1 still remains relevant. You don't need to check again.

You don't need to do this. 1) getContainerItem is only called if hasContainerItem returns true 2) Its your own god damn item and you already always return true...

Return something other than PASS? Pass means "I didn't do anything." Check the other values and return the obviously correct one.

You call a method here, but the code you've shown does not contain this method. What line is that?

Your blockstate file is still in the wrong place or incorrectly named.

Post new logs.

You're missing your modID directory.

Also post your latest log file.

Vanilla already has a gamerule for this (keepInventory). Just turn it on.

You're not supposed to edit vanilla source files. Use reflection.

data and assets are siblings. Also don't put your mod's recipes in the minecraft folder, use your own mod ID

handIn.OFF_HAND is equivalent to saying "4" as an integer. Is "4"....what?

Because leaves don't have a "correct tool" per the tool type enum. They check for shears differently. And when the tool isn't correct the harvest speed is ignored (and is basically the same as punching blocks). You'll also find that webs won't break instantly either.

Check any vanilla monster's loot table file, one where there's really basic loot (so not cows and pigs that deal with being on fire). Squid is a good example. Take that file, change the item dropped, put it in your own resource directory, and then add it during the event (as my code shows you how to do).

Option (1) completely override the loot table by creating your own and putting it in the same resource directory as the vanilla file (e.g src/resources/data/minecraft/loot_tables). This is a last-mod-wins solution. Option (2) append the loot table using the LootTableLoadEvent. By appending you don't interfere with other mods, but also can't prevent existing drops (so you'd get bones AND your item instead of bone OR your item). Option (3) use a Global Loot Modifier. This is by far the most flexible and powerful option.