warjort

You have REI and jei You can't have both.

It is the slot that was clicked. You decide how the slots are layed out in your constructor. Though vanilla has a "convention" of sorts. e.g. see BeanconMenu which puts the payment slot at slot 0 then the player's main inventory in the next 27 slots (1 to 27) and finally the 9 hotbar slots (28 to 37). You can then see these numbers used in quickMoveStack() where it decides the priority of where things should go based on the slot clicked. If you look at the top of BeaconMenu you will also see these same constants defined. This is a failure of the deobfuscation where obviously the real minecraft code used those symbolic constants instead of the numbers. You can do the same to make the code more readable and less error prone. If you want a more complicated example, see the BrewingStandMenu which has different customs slots with mayPlace() tests.

Post a link to your logs/debug.log

This means you have 2 mods using the same mod id such as jade and wthit (both forks of Waila) JEI and Roughly Enough Items is another common example. If you post the logs/debug.log we can see which mods have which ids.

Either define the property or hard code the version in the dependency. You are also following some very old and out-dated documentation or tutorial, Your file has examples in a comment for the correct way to do it directly beneath your broken config. See also: https://forge.gemwire.uk/wiki/Dependencies

Ok, not an old data issue then. 🙂

I think I know what your issue is. @Override public void load(CompoundTag nbt) { super.load(nbt); itemHandler.deserializeNBT(nbt.getCompound("inventory")); progress = nbt.getInt("altarofessence.progress"); } The number of slots is stored in the NBT of the block entity's item handler. That deserialize will be loading old data that have item handlers with only 3 slots. Add some temporary code that converts old data to new data in your load, or just create a new test world.

Try doing running the "clean" gradle task. Since you are using "static final" to define some constants it might be the compiler didn't recompile something that references a constant when it changed. Such constants get "inlined" by the compiler. The clean task will force everything to be rebuilt from scratch the next time you build.

If this is a networking error, mojang log them at debug level Add the following property to your run configuration in your build.gradle Otherwise you can use a debugger to see the full error including its stacktrace.

Why don't you post the full error so we can see where it is crashing?

You are replacing the button on every screen event (including mouse clicked). There are many subclasses for specific events, you probably want ScreenEvent.Init.Post which fires after the initialisation of the screen. Also you can get the screen using event.getScreen()

Post a link to your logs/debug.log

Looks like an issue with the physicsmod. The curseforge page says you have to download up-to-date versions from the physics mod website: https://www.curseforge.com/minecraft/mc-mods/physics-mod If the latest version doesn't fix it, contact the mod author.

https://github.com/BobMowzie/MowziesMobs/issues/673

Post a link to your logs/debug.log

Post a link to your logs/debug.log

Post a link to your logs/debug.log

The issue with the old school way was you get people referencing client classes in their BlockEntity for the client side ticks. Which would then cause crashes on a dedicated server.

It's just an interface implement it how you like. I think you will find a lot of mods were written a while ago, when handling a block entity's ticks required different code. Instead of rewriting all their code, they probably just wrote some "glue code" that let them use the old code with the new way. Personally I liked the old school way where the BlockEntity just implemented a tick() method. You can see Immersive Engineering kind of emulating that here: https://github.com/BluSunrize/ImmersiveEngineering/blob/1.18.2/src/main/java/blusunrize/immersiveengineering/common/blocks/ticking/IEServerTickableBE.java

If you are creating a new world, then removing the mod is ok. If you already have a save where that mod alters the nether, you will need that mod installed to use it.

The error says the problem is with BYG Caused by: java.lang.NullPointerException: Cannot invoke "net.minecraft.world.phys.shapes.VoxelShape.m_83263_(net.minecraft.core.Direction)" because "p_49919_" is null at net.minecraft.world.level.block.Block.m_49918_(Block.java:237) ~[client-1.19.2-20220805.130853-srg.jar%23250!/:?] at potionstudios.byg.common.block.FlatVegetationBlock.m_6266_(FlatVegetationBlock.java:35) ~[Oh_The_Biomes_You'll_Go-forge-1.19.2-2.0.0.13.jar%23211!/:2.0.0.13] at net.minecraft.world.level.block.Block.canSustainPlant(Block.java:537) ~[client-1.19.2-20220805.130853-srg.jar%23250!/:?] at net.minecraftforge.common.extensions.IForgeBlockState.canSustainPlant(IForgeBlockState.java:277) ~[forge-1.19.2-43.1.43-universal.jar%23255!/:?] It is an issue with a test to see if a BYG's "FlatVegetationBlock" can have a plant on top of it. You will need to contact the BYG mod authors. It also looks like canary mod wasn't the problem. It just wasn't handling the real problem in a good way. So I guess you can reinstall that mod. Although IIRC the canary mod was crashing with something to do with trees rather than vegetation?

You should create a new world that is just vanilla.

This is the forge support. If you are not using forge you are in the wrong place. But you can't use a save created using forge with vanilla. Vanilla has no idea what the changes to the world gen mean or all the modded blocks and items, etc.

You are not running forge. It looks like you are using vanilla minecraft with a save from forge?

There is an error just before the one that causes the crash. Which looks like it is trying to reference a block that doesn't exist: minecraft:air is Mojang's way of saying "not registered" You will need to contact the netherdepthsupgrade mod author to figure out is happening.