Merge pull request #128 from MultiChat/v19

MultiChat v1.9.5 - Fix RGB codes not working when set in MultiChat formats

Author: OllieMartin

multichat/dependency-reduced-pom.xml

@@ -3,7 +3,7 @@
   <modelVersion>4.0.0</modelVersion>
   <groupId>xyz.olivermartin.multichat</groupId>
   <artifactId>multichat</artifactId>
-  <version>1.9.4</version>
+  <version>1.9.5</version>
   <build>
     <plugins>
       <plugin>

multichat/pom.xml

@@ -5,7 +5,7 @@
 
 	<groupId>xyz.olivermartin.multichat</groupId>
 	<artifactId>multichat</artifactId>
-	<version>1.9.4</version>
+	<version>1.9.5</version>
 
 	<repositories>
 

multichat/releases/1.9.5/multichat-1.9.5-dev.1.jar

@@ -37,11 +37,12 @@
  */
 public class MultiChat extends Plugin implements Listener {
 
-	public static final String LATEST_VERSION = "1.9.4";
+	public static final String LATEST_VERSION = "1.9.5";
 
 	public static final String[] ALLOWED_VERSIONS = new String[] {
 
 			LATEST_VERSION,
+			"1.9.4",
 			"1.9.3",
 			"1.9.2",
 			"1.9.1",

multichat/src/main/java/xyz/olivermartin/multichat/bungee/MultiChat.java

@@ -8,6 +8,7 @@
 
 import net.milkbowl.vault.chat.Chat;
 import xyz.olivermartin.multichat.bungee.MultiChatUtil;
+import xyz.olivermartin.multichat.local.common.LocalConsoleLogger;
 import xyz.olivermartin.multichat.local.common.LocalMetaManager;
 import xyz.olivermartin.multichat.local.common.MultiChatLocal;
 import xyz.olivermartin.multichat.local.spigot.hooks.LocalSpigotVaultHook;
@@ -72,22 +73,40 @@ public String getWorld(UUID uuid) {
 	@Override
 	public String getDisplayName(UUID uuid) {
 
+		LocalConsoleLogger logger = MultiChatLocal.getInstance().getConsoleLogger();
+
 		Player player = Bukkit.getPlayer(uuid);
 
 		if (player == null) return "";
 
 		// If MultiChat is setting the display name...
 		if (MultiChatLocal.getInstance().getDataStore().isSetDisplayName()) {
 
+			logger.debug("[LocalSpigotMetaManager] We are setting the display name!");
+
 			String displayNameFormat = MultiChatLocal.getInstance().getDataStore().getDisplayNameFormatLastVal();
 
+			logger.debug("[LocalSpigotMetaManager] Format = " + displayNameFormat);
+			logger.debug("[LocalSpigotMetaManager] Format (using & only) = " + displayNameFormat.replaceAll("(?i)§(?=[a-f,0-9,k-o,r,x])", "&"));
+
 			// TODO This stuff could be refactored as it is duplicated between Spigot and Sponge
 			displayNameFormat = displayNameFormat.replaceAll("%NICK%", getNick(uuid));
 			displayNameFormat = displayNameFormat.replaceAll("%NAME%", player.getName());
 			displayNameFormat = displayNameFormat.replaceAll("%PREFIX%", getPrefix(uuid));
 			displayNameFormat = displayNameFormat.replaceAll("%SUFFIX%", getSuffix(uuid));
+
+			logger.debug("[LocalSpigotMetaManager] Format with placeholders = " + displayNameFormat);
+			logger.debug("[LocalSpigotMetaManager] Format with placeholders (using & only) = " + displayNameFormat.replaceAll("(?i)§(?=[a-f,0-9,k-o,r,x])", "&"));
+
 			displayNameFormat = MultiChatUtil.reformatRGB(displayNameFormat);
-			displayNameFormat = displayNameFormat.replaceAll("&(?=[a-f,0-9,k-o,r,x])", "§");
+
+			logger.debug("[LocalSpigotMetaManager] Format after reformatting RGB = " + displayNameFormat);
+			logger.debug("[LocalSpigotMetaManager] Format after reformatting RGB (using & only) = " + displayNameFormat.replaceAll("(?i)§(?=[a-f,0-9,k-o,r,x])", "&"));
+
+			displayNameFormat = displayNameFormat.replaceAll("(?i)&(?=[a-f,0-9,k-o,r,x])", "§");
+
+			logger.debug("[LocalSpigotMetaManager] FINAL = " + displayNameFormat);
+			logger.debug("[LocalSpigotMetaManager] FINAL (using & only) = " + displayNameFormat.replaceAll("(?i)§(?=[a-f,0-9,k-o,r,x])", "&"));
 
 			// LEGACY HACK
 			if (MultiChatLocal.getInstance().getDataStore().isLegacy()) {

multichat/src/main/java/xyz/olivermartin/multichat/local/spigot/LocalSpigotMetaManager.java

@@ -2,6 +2,7 @@
 
 import java.util.UUID;
 
+import xyz.olivermartin.multichat.bungee.MultiChatUtil;
 import xyz.olivermartin.multichat.local.common.LocalPlaceholderManager;
 import xyz.olivermartin.multichat.local.common.MultiChatLocalPlatform;
 
@@ -14,6 +15,9 @@ public LocalSpigotPlaceholderManager() {
 	@Override
 	public String buildChatFormat(UUID uuid, String format) {
 
+		// Reformat any hex codes in the format
+		format = MultiChatUtil.reformatRGB(format);
+
 		// RESPECT OTHER PLUGIN'S DISPLAY NAMES FIRST! (Allows for factions etc.)
 		format = format.replace("%DISPLAYNAME%", "%1$s");
 

multichat/src/main/java/xyz/olivermartin/multichat/local/spigot/LocalSpigotPlaceholderManager.java

@@ -93,7 +93,7 @@ public String getDisplayName(UUID uuid) {
 			displayNameFormat = displayNameFormat.replaceAll("%PREFIX%", getPrefix(uuid));
 			displayNameFormat = displayNameFormat.replaceAll("%SUFFIX%", getSuffix(uuid));
 			displayNameFormat = MultiChatUtil.reformatRGB(displayNameFormat);
-			displayNameFormat = displayNameFormat.replaceAll("&(?=[a-f,0-9,k-o,r,x])", "§");
+			displayNameFormat = displayNameFormat.replaceAll("(?i)&(?=[a-f,0-9,k-o,r,x])", "§");
 
 			displayNameFormat = MultiChatUtil.approximateHexCodes(displayNameFormat);
 

multichat/src/main/java/xyz/olivermartin/multichat/local/sponge/LocalSpongeMetaManager.java

@@ -2,6 +2,7 @@
 
 import java.util.UUID;
 
+import xyz.olivermartin.multichat.bungee.MultiChatUtil;
 import xyz.olivermartin.multichat.local.common.LocalPlaceholderManager;
 import xyz.olivermartin.multichat.local.common.MultiChatLocalPlatform;
 
@@ -13,7 +14,11 @@ public LocalSpongePlaceholderManager() {
 
 	@Override
 	public String buildChatFormat(UUID uuid, String format) {
-		return processMultiChatPlaceholders(uuid, format).replaceAll("&(?=[a-f,0-9,k-o,r])", "§");
+		// Reformat any hex codes in the format
+		format = MultiChatUtil.reformatRGB(format);
+		format = processMultiChatPlaceholders(uuid, format).replaceAll("(?i)&(?=[a-f,0-9,k-o,r,x])", "§");
+		format = MultiChatUtil.approximateHexCodes(format);
+		return format;
 	}
 
 }

multichat/src/main/java/xyz/olivermartin/multichat/local/sponge/LocalSpongePlaceholderManager.java

@@ -56,7 +56,7 @@
 import xyz.olivermartin.multichat.local.sponge.listeners.communication.LocalSpongePlayerChannelListener;
 import xyz.olivermartin.multichat.local.sponge.listeners.communication.LocalSpongePlayerMetaListener;
 
-@Plugin(id = "multichat", name = "MultiChat", version = "1.9.4", dependencies = { @Dependency(id = "placeholderapi", optional = true) })
+@Plugin(id = "multichat", name = "MultiChat", version = "1.9.5", dependencies = { @Dependency(id = "placeholderapi", optional = true) })
 public class MultiChatLocalSpongePlugin {
 
 	@Inject

multichat/src/main/java/xyz/olivermartin/multichat/local/sponge/MultiChatLocalSpongePlugin.java

@@ -1,6 +1,6 @@
 name: MultiChat
 main: xyz.olivermartin.multichat.bungee.MultiChat
-version: 1.9.4
+version: 1.9.5
 author: Revilo410
 api-version: 1.16
 softdepends: [PremiumVanish]