Skip to content

Make Controlling Mixin only load if Controlling is installed #19

@BlayTheNinth

Description

@BlayTheNinth

Minecraft Version

1.21.11

Mod Loader

NeoForge / Fabric / Forge

Describe the Issue

Right now if Controlling is not installed it leads to this error message in the log:

[15:48:02] [main/WARN]: Error loading class: com/blamejared/controlling/client/NewKeyBindsList$KeyEntry (java.lang.ClassNotFoundException: com/blamejared/controlling/client/NewKeyBindsList$KeyEntry)
[15:48:02] [main/WARN]: @Mixin target com.blamejared.controlling.client.NewKeyBindsList$KeyEntry was not found kuma_api.mixins.json:controlling.ControllingKeyEntryMixin from mod kuma_api

It's harmless, but may lead users to make reports or incorrectly attribute errors to Kuma/Balm.

Initial attempt at adding a Mixin Plugin to make it conditional lead to weird class mismatches (possibly due to different Mixin versions/flavors in the subprojects?)... we'll have to see.

Logs

dev

Metadata

Metadata

Assignees

No one assigned

    Labels

    BugSomething isn't working as expectedFabricIssues occurring on the Fabric platformForgeIssues occurring on the Forge platformMinecraft 1.21.11NeoForgeIssues occurring on the NeoForge platform

    Type

    Projects

    Status

    📝 Backlog

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions