RandomPatches (Forge)

RandomPatches (Forge)

56M Downloads

Mixin config loads MC classes too early, causing other Mixins to fail

malte0811 opened this issue ยท 1 comments

commented

@Override
public boolean shouldApplyMixin(String targetClassName, String mixinClassName) {
return RandomPatches.config().misc.isMixinClassEnabled(mixinClassName);
}

That causes the main mod class and various other classes (including Identifier/ResourceLocation via autoconfig and the fabric Event class) to be loaded before all Mixin configs are loaded. In my case this causes malte0811/FerriteCore#43 (comment) since my accessors to ResourceLocation are not applied. In general only classes from "basic" libraries and classes in your own mod that only load such classes should be referenced directly in a Mixin config.

commented

This also breaks Rhino: https://www.curseforge.com/minecraft/mc-mods/rhino with the same issue