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?
Currently Wires and MiniMOTD both redirect the same method for handling server list MOTDs.
Wires/src/main/java/net/quiltservertools/wires/mixins/MixinServerQueryNetworkHandler.java
Line 13 in 4965e63
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?