Update gradle.properties

Fixed incompatibility with mod Dynmap

README.md

@@ -2,8 +2,8 @@
  A Simple mod add bukkit style with an "&" and "[item]" display for Fabric Server.
  
  It's a **server-side** mod and doesn't need to be installed on the client.
- 
- Current Minecraft version: 1.20.1
+
+Currently supported Minecraft versions: 1.20.1, 1.20.4
 
 ## Screenshot
 ![image](https://github.com/CPTProgrammer/ChatPlus/assets/46586216/dcd2ca4b-79e9-4692-a8de-02fddfe4a392)

build.gradle

@@ -1,9 +1,9 @@
 plugins {
-	id 'fabric-loom' version '1.3-SNAPSHOT'
+	id 'fabric-loom' version '1.5-SNAPSHOT'
 	id 'maven-publish'
 }
 
-version = project.mod_version
+version = project.mod_version + "-mc" + project.minecraft_version
 group = project.maven_group
 
 base {

gradle.properties

@@ -4,14 +4,14 @@ org.gradle.parallel=true
 
 # Fabric Properties
 # check these on https://fabricmc.net/develop
-minecraft_version=1.20.1
-yarn_mappings=1.20.1+build.9
-loader_version=0.14.21
+minecraft_version=1.20.4
+yarn_mappings=1.20.4+build.3
+loader_version=0.15.6
 
 # Mod Properties
-mod_version=0.20.0
+mod_version=0.20.3
 maven_group=cn.revaria.chatplus
 archives_base_name=chat-plus
 
 # Dependencies
-fabric_version=0.85.0+1.20.1
+fabric_version=0.95.1+1.20.4

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.2-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.5-bin.zip
 networkTimeout=10000
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME

src/main/java/cn/revaria/chatplus/ChatPlus.java

@@ -1,11 +1,6 @@
 package cn.revaria.chatplus;
 
 import net.fabricmc.api.ModInitializer;
-
-import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
-import net.minecraft.network.message.MessageType;
-import net.minecraft.network.message.SignedMessage;
-import net.minecraft.server.network.ServerPlayerEntity;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -13,7 +8,7 @@ public class ChatPlus implements ModInitializer {
 	// This logger is used to write text to the console and the log file.
 	// It is considered best practice to use your mod id as the logger's name.
 	// That way, it's clear which mod wrote info, warnings, and errors.
-    public static final Logger LOGGER = LoggerFactory.getLogger("chat-plus");
+    public static final Logger LOGGER = LoggerFactory.getLogger("Chat Plus");
 
 	@Override
 	public void onInitialize() {

src/main/java/cn/revaria/chatplus/mixin/MixinChat.java

@@ -1,75 +1,50 @@
 package cn.revaria.chatplus.mixin;
 
-import io.netty.util.internal.StringUtil;
-import net.fabricmc.fabric.api.object.builder.v1.block.FabricBlockSettings;
+import net.minecraft.MinecraftVersion;
 import net.minecraft.SharedConstants;
-import net.minecraft.block.AbstractBlock;
-import net.minecraft.block.AirBlock;
-import net.minecraft.block.Block;
-import net.minecraft.entity.EquipmentSlot;
-import net.minecraft.item.Item;
 import net.minecraft.item.ItemStack;
-import net.minecraft.item.Items;
-import net.minecraft.network.message.LastSeenMessageList;
-import net.minecraft.network.message.MessageChain;
-import net.minecraft.network.message.MessageType;
-import net.minecraft.network.message.SignedMessage;
-import net.minecraft.network.message.MessageChainTaskQueue;
-import net.minecraft.server.MinecraftServer;
+import net.minecraft.network.ClientConnection;
+import net.minecraft.network.message.*;
 import net.minecraft.network.packet.c2s.play.ChatMessageC2SPacket;
-import net.minecraft.network.listener.ServerPlayPacketListener;
+import net.minecraft.server.MinecraftServer;
 import net.minecraft.server.filter.FilteredMessage;
-import net.minecraft.server.network.ServerPlayerEntity;
+import net.minecraft.server.network.ConnectedClientData;
+import net.minecraft.server.network.ServerCommonNetworkHandler;
 import net.minecraft.server.network.ServerPlayNetworkHandler;
-import net.minecraft.text.LiteralTextContent;
+import net.minecraft.server.network.ServerPlayerEntity;
 import net.minecraft.text.MutableText;
 import net.minecraft.text.Text;
-import net.minecraft.item.AirBlockItem;
-import net.minecraft.text.TextContent;
-import net.minecraft.util.Formatting;
-import net.minecraft.nbt.NbtCompound;
-import org.apache.commons.lang3.StringUtils;
-import org.apache.logging.log4j.core.jmx.Server;
-import org.slf4j.Logger;
 import org.spongepowered.asm.mixin.Final;
 import org.spongepowered.asm.mixin.Mixin;
-import org.spongepowered.asm.mixin.Mutable;
 import org.spongepowered.asm.mixin.Shadow;
 import org.spongepowered.asm.mixin.injection.At;
 import org.spongepowered.asm.mixin.injection.Inject;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
-import com.google.gson.Gson;
-import com.google.gson.GsonBuilder;
 
-import java.time.Instant;
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
 import java.util.ArrayDeque;
 import java.util.Deque;
-import java.util.Objects;
 import java.util.Optional;
 import java.util.concurrent.CompletableFuture;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
 @Mixin(ServerPlayNetworkHandler.class)
-public abstract class MixinChat {
-	@Shadow @Final
-	static Logger LOGGER;
-
-	@Final
-	@Shadow
-	private MinecraftServer server;
-
-	@Shadow
-	public abstract void disconnect(Text reason);
+public abstract class MixinChat extends ServerCommonNetworkHandler {
+	public MixinChat(MinecraftServer server, ClientConnection connection, ConnectedClientData clientData) {
+		super(server, connection, clientData);
+	}
 
 	@Final
 	@Shadow
 	private MessageChainTaskQueue messageChainTaskQueue;
 
 	@Shadow
-	private ServerPlayerEntity player;
+	public ServerPlayerEntity player;
 
-	@Shadow protected abstract Optional<LastSeenMessageList> validateMessage(String message, Instant timestamp, LastSeenMessageList.Acknowledgment acknowledgment);
+	@Shadow protected abstract Optional<LastSeenMessageList> validateMessage(LastSeenMessageList.Acknowledgment acknowledgment);
 
 	@Shadow protected abstract SignedMessage getSignedMessage(ChatMessageC2SPacket packet, LastSeenMessageList lastSeenMessages) throws MessageChain.MessageChainException;
 
@@ -86,7 +61,7 @@ public abstract class MixinChat {
 		if (hasIllegalCharacter(packet.chatMessage())) {
 			disconnect(Text.translatable("multiplayer.disconnect.illegal_characters"));
 		} else {
-			Optional<LastSeenMessageList> optional = this.validateMessage(packet.chatMessage(), packet.timestamp(), packet.acknowledgment());
+			Optional<LastSeenMessageList> optional = this.validateMessage(packet.acknowledgment());
 			if (optional.isPresent()) {
 				if (!packet.chatMessage().startsWith("/")){
 
@@ -105,9 +80,9 @@ public abstract class MixinChat {
 						}
 					}
 
-					MutableText changedText = MutableText.of(TextContent.EMPTY);
-					for (int i = 0; i < messages.length; i++) {
-						changedText.append(Text.of(messages[i]));
+					MutableText changedText = Text.empty();
+					for (String message : messages) {
+						changedText.append(Text.of(message));
 						if (!itemDeque.isEmpty()) {
 							ItemStack itemStack;
 							if (itemDeque.getFirst() == -1) {
@@ -125,6 +100,34 @@ public abstract class MixinChat {
 						server.getPlayerManager().broadcast(signedMessage.withUnsignedContent(
 								changedText
 						), player, MessageType.params(MessageType.CHAT, player));
+
+						// Compatible with mod "Discord Integration"
+						try {
+							Class<?> DiscordIntegrationMod = Class.forName("de.erdbeerbaerlp.dcintegration.fabric.DiscordIntegrationMod");
+							Method handleChatMessage = DiscordIntegrationMod.getMethod("handleChatMessage", SignedMessage.class, ServerPlayerEntity.class);
+							handleChatMessage.invoke(null, signedMessage.withUnsignedContent(changedText), player);
+						} catch (ClassNotFoundException | NoSuchMethodException | InvocationTargetException |
+								 IllegalAccessException ignored) { }
+
+						// Compatible with mod "Dynmap"
+						try {
+							/*
+							  Warning: Using reflection can make the code harder to maintain, debug, and understand.
+							  Statement: DynmapMod.plugin.chathandler.handleChat()
+							 */
+							Class<?> DynmapMod = Class.forName("org.dynmap.fabric_" + MinecraftVersion.CURRENT.getName().replaceAll("\\.", "_") + ".DynmapMod");
+							Field pluginField = DynmapMod.getField("plugin");
+							Object plugin = pluginField.get(null);
+							Class<?> pluginClass = plugin.getClass();
+							Field chatHandlerField = pluginClass.getDeclaredField("chathandler");
+							chatHandlerField.setAccessible(true);
+							Object chatHandler = chatHandlerField.get(plugin);
+							Class<?> chatHandlerClass = chatHandler.getClass();
+							Method handleChat = chatHandlerClass.getMethod("handleChat", ServerPlayerEntity.class, String.class);
+							handleChat.invoke(chatHandler, player, signedMessage.withUnsignedContent(changedText).getContent().getString());
+						} catch (ClassNotFoundException | NoSuchMethodException | NoSuchFieldException | NullPointerException |
+								 IllegalAccessException | InvocationTargetException ignored) { }
+
 					} catch (MessageChain.MessageChainException e) {
 						handleMessageChainException(e);
 					}
@@ -132,19 +135,17 @@ public abstract class MixinChat {
 					this.server.submit(() -> {
 						SignedMessage signedMessage;
 						try {
-							signedMessage = this.getSignedMessage(packet, (LastSeenMessageList) optional.get());
+							signedMessage = this.getSignedMessage(packet, optional.get());
 						} catch (MessageChain.MessageChainException var6) {
 							handleMessageChainException(var6);
 							return;
 						}
 
 						CompletableFuture<FilteredMessage> completableFuture = filterText(signedMessage.getSignedContent());
-						CompletableFuture<Text> completableFuture2 = this.server.getMessageDecorator().decorate(this.player, signedMessage.getContent());
-						messageChainTaskQueue.append((executor) -> {
-							return CompletableFuture.allOf(completableFuture, completableFuture2).thenAcceptAsync((void_) -> {
-								SignedMessage signedMessage2 = signedMessage.withUnsignedContent((Text) completableFuture2.join()).withFilterMask(((FilteredMessage) completableFuture.join()).mask());
-								handleDecoratedMessage(signedMessage2);
-							}, executor);
+						Text decoratedMessage = this.server.getMessageDecorator().decorate(this.player, signedMessage.getContent());
+						messageChainTaskQueue.append(completableFuture, filteredMessage -> {
+							SignedMessage message = signedMessage.withUnsignedContent(decoratedMessage).withFilterMask(filteredMessage.mask());
+							this.handleDecoratedMessage(message);
 						});
 					});
 				}

src/main/resources/fabric.mod.json

@@ -8,8 +8,9 @@
 		"Rev_Aria"
 	],
 	"contact": {
-		"homepage": "https://fabricmc.net/",
-		"sources": "https://github.com/FabricMC/fabric-example-mod"
+		"homepage": "https://modrinth.com/mod/chatplus",
+		"sources": "https://github.com/CPTProgrammer/ChatPlus/",
+		"issues": "https://github.com/CPTProgrammer/ChatPlus/issues"
 	},
 	"license": "CC0-1.0",
 	"icon": "assets/chat-plus/icon.png",
@@ -28,7 +29,7 @@
 	],
 	"depends": {
 		"fabricloader": ">=0.14.21",
-		"minecraft": "~1.20.1",
+		"minecraft": "~1.20.4",
 		"java": ">=17",
 		"fabric-api": "*"
 	},