Cloth API (Fabric)

Cloth API (Fabric)

12M Downloads

Bad ServerPlayerEntity mixin causes crash on container update

unascribed opened this issue ยท 0 comments

commented

Ref: unascribed/NotEnoughCreativity#28

When making a Mixin extend things to provide access to the target's class hierarchy, you must not implement interfaces that already exist, or you will overwrite the target's implementations of that interface with abstract stubs.

public abstract class MixinServerPlayerEntity extends PlayerEntity implements ScreenHandlerListener {

The implements ScreenHandlerListener must be removed. I have confirmed removing it and recompiling the mod prevents the crash, however there may be other bad mixins in the project I don't know of.

Something about NEC+ causes this otherwise-dormant issue to be activated. With NEC+ and Cloth Common Events installed, using /give or entering Creative+ mode causes the crash in the linked issue. Here's a crash from my minimal repro, using a version of Cloth Common Events built from the latest commit on the 1.17 branch: https://paste.ee/p/9Antr