Skip to content

Commit 515f051

Browse files
Strip Discord Formatting & More Color Codes
1 parent 83de2ae commit 515f051

6 files changed

Lines changed: 12 additions & 12 deletions

File tree

src/main/java/com/firecontroller1847/truediscordlink/DiscordManager.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -324,8 +324,8 @@ private void sendDiscordBotMessage(String content, boolean blocking, Player play
324324
if (player != null) {
325325
CompletableFuture<Message> future = channel.sendMessage(TrueDiscordLink.stripColorCodes(discordlink.getTranslation("messages.from_mc_bot_format", false,
326326
new String[] { "%message%", content },
327-
new String[] { "%name%" , player.getName() },
328-
new String[] { "%displayName%", player.getDisplayName() },
327+
new String[] { "%name%" , TrueDiscordLink.escapeDiscordFormatting(player.getName()) },
328+
new String[] { "%displayName%", TrueDiscordLink.escapeDiscordFormatting(player.getDisplayName()) },
329329
new String[] { "%uuid%", player.getUniqueId().toString() }
330330
)));
331331
if (blocking) {
@@ -362,8 +362,8 @@ private void sendDiscordWebhookMessage(String content, Player player) {
362362
if (player != null) {
363363
this.makeWebhookRequest(url, TrueDiscordLink.stripColorCodes(discordlink.getTranslation("messages.from_mc_webhook_format", false,
364364
new String[] { "%message%", content },
365-
new String[] { "%name%" , player.getName() },
366-
new String[] { "%displayName%", player.getDisplayName() },
365+
new String[] { "%name%" , TrueDiscordLink.escapeDiscordFormatting(player.getName()) },
366+
new String[] { "%displayName%", TrueDiscordLink.escapeDiscordFormatting(player.getDisplayName()) },
367367
new String[] { "%uuid%", player.getUniqueId().toString() }
368368
)), TrueDiscordLink.stripColorCodes(player.getDisplayName()), skin); // TODO: Why is the username not customizable
369369
} else {

src/main/java/com/firecontroller1847/truediscordlink/TrueDiscordLink.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ private void migrateConfigurations() throws IOException {
101101
}
102102

103103
// Escapes formatting for Discord messages
104-
public String escapeDiscordFormatting(String content) {
104+
public static String escapeDiscordFormatting(String content) {
105105
return content.replace("*", "\\*").replace("_", "\\_").replace("~", "\\~");
106106
}
107107

src/main/java/com/firecontroller1847/truediscordlink/listeners/discord/DiscordChatListener.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ public void onMessageCreate(MessageCreateEvent event) {
120120
// Placeholder API
121121
if (Bukkit.getPluginManager().getPlugin("PlaceholderAPI") != null) {
122122
value = PlaceholderAPI.setPlaceholders(null, value);
123-
value = TrueDiscordLink.stripColorCodes(value);
123+
value = TrueDiscordLink.escapeDiscordFormatting(TrueDiscordLink.stripColorCodes(value));
124124
}
125125

126126
// Send Message

src/main/java/com/firecontroller1847/truediscordlink/listeners/minecraft/PlayerDeathListener.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ public void onPlayerDeath(PlayerDeathEvent event) {
2626
discordlink.getDiscordManager().sendDiscordMessage(
2727
discordlink.getTranslation("events.player_death", false,
2828
new String[] { "%message%", event.getDeathMessage() },
29-
new String[] { "%name%", event.getEntity().getName() },
30-
new String[] { "%displayName%", event.getEntity().getDisplayName() },
29+
new String[] { "%name%", TrueDiscordLink.escapeDiscordFormatting(TrueDiscordLink.stripColorCodes(event.getEntity().getName())) },
30+
new String[] { "%displayName%", TrueDiscordLink.escapeDiscordFormatting(TrueDiscordLink.stripColorCodes(event.getEntity().getDisplayName())) },
3131
new String[] { "%uuid%", event.getEntity().getUniqueId().toString() }
3232
)
3333
);

src/main/java/com/firecontroller1847/truediscordlink/listeners/minecraft/PlayerJoinListener.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ public void onPlayerJoin(PlayerJoinEvent event) {
2525
// Send Discord Message
2626
discordlink.getDiscordManager().sendDiscordMessage(
2727
discordlink.getTranslation("events.player_join", false,
28-
new String[] { "%name%", event.getPlayer().getName() },
29-
new String[] { "%displayName%", event.getPlayer().getDisplayName() },
28+
new String[] { "%name%", TrueDiscordLink.escapeDiscordFormatting(event.getPlayer().getName()) },
29+
new String[] { "%displayName%", TrueDiscordLink.escapeDiscordFormatting(event.getPlayer().getDisplayName()) },
3030
new String[] { "%uuid%", event.getPlayer().getUniqueId().toString() }
3131
)
3232
);

src/main/java/com/firecontroller1847/truediscordlink/listeners/minecraft/PlayerQuitListener.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ public void onPlayerQuit(PlayerQuitEvent event) {
2525
// Send Discord Message
2626
discordlink.getDiscordManager().sendDiscordMessage(
2727
discordlink.getTranslation("events.player_quit", false,
28-
new String[] { "%name%", event.getPlayer().getName() },
29-
new String[] { "%displayName%", event.getPlayer().getDisplayName() },
28+
new String[] { "%name%", TrueDiscordLink.escapeDiscordFormatting(event.getPlayer().getName()) },
29+
new String[] { "%displayName%", TrueDiscordLink.escapeDiscordFormatting(event.getPlayer().getDisplayName()) },
3030
new String[] { "%uuid%", event.getPlayer().getUniqueId().toString() }
3131
)
3232
);

0 commit comments

Comments
 (0)