Skip to content

Incompatible with MiniMOTD #4

@jpenilla

Description

@jpenilla

Currently Wires and MiniMOTD both redirect the same method for handling server list MOTDs.

@Redirect(method = "onRequest", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/MinecraftServer;getServerMetadata()Lnet/minecraft/server/ServerMetadata;"))

https://github.com/jpenilla/MiniMOTD/blob/80ae413c9d3ba4fecb31a2dfa0ddafb011c428d4/fabric/src/main/java/xyz/jpenilla/minimotd/fabric/mixin/ServerStatusPacketListenerImplMixin.java#L48

I'm not sure what the best course of action is here for compatibility, given we both need to modify the return value... Since this seems like a common mixin, maybe we should try and get something into Fabric API?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions