add optional comments to /help ping

Author: danthe1st

src/main/java/net/javadiscord/javabot/systems/help/commands/HelpPingSubcommand.java

@@ -11,6 +11,8 @@
 import net.dv8tion.jda.api.entities.emoji.EmojiUnion;
 import net.dv8tion.jda.api.events.interaction.command.SlashCommandInteractionEvent;
 import net.dv8tion.jda.api.events.interaction.component.ButtonInteractionEvent;
+import net.dv8tion.jda.api.interactions.commands.OptionMapping;
+import net.dv8tion.jda.api.interactions.commands.OptionType;
 import net.dv8tion.jda.api.interactions.commands.build.SubcommandData;
 import net.dv8tion.jda.api.interactions.components.buttons.Button;
 import net.dv8tion.jda.api.interactions.components.buttons.ButtonStyle;
@@ -21,6 +23,8 @@
 import net.javadiscord.javabot.data.config.guild.HelpConfig;
 import net.javadiscord.javabot.util.Pair;
 import net.javadiscord.javabot.util.Responses;
+import net.javadiscord.javabot.util.StringUtils;
+
 import org.jetbrains.annotations.NotNull;
 import xyz.dynxsty.dih4jda.interactions.commands.application.SlashCommand;
 import xyz.dynxsty.dih4jda.interactions.components.ButtonHandler;
@@ -55,7 +59,8 @@ public class HelpPingSubcommand extends SlashCommand.Subcommand implements Butto
 	 * @param botConfig The main configuration of the bot
 	 */
 	public HelpPingSubcommand(BotConfig botConfig, ScheduledExecutorService asyncPool) {
-		setCommandData(new SubcommandData("ping", "Notify potential helpers that your question is urgent."));
+		setCommandData(new SubcommandData("ping", "Notify potential helpers that your question is urgent.")
+				.addOption(OptionType.STRING, "comment", "Optionally enter the reason you used this to be seen by helpers (e.g. 'no response')", false));
 		lastPingTimes = new ConcurrentHashMap<>();
 		this.botConfig = botConfig;
 		asyncPool.scheduleWithFixedDelay(this::cleanTimeoutCache, CACHE_CLEANUP_DELAY, CACHE_CLEANUP_DELAY, TimeUnit.SECONDS);
@@ -92,29 +97,26 @@ public void execute(@NotNull SlashCommandInteractionEvent event) {
 		if (isHelpPingTimeoutElapsed(member.getIdLong(), config)) {
 			lastPingTimes.put(event.getMember().getIdLong(), new Pair<>(System.currentTimeMillis(), config.getGuild()));
 			TextChannel notifChannel = config.getHelpConfig().getHelpNotificationChannel();
-			notifChannel.sendMessageEmbeds(new EmbedBuilder().setDescription("""
-					%s requested help in %s
-					
-					Tags:
-					%s
-					
-					[Click to view](%s)
-					"""
-					.formatted(
-							event.getUser().getAsMention(),
-							post.getAsMention(),
-							getTagString(post),
-							post.getJumpUrl()
-					))
+			EmbedBuilder eb = new EmbedBuilder()
+					.setDescription("%s requested help in %s"
+						.formatted(
+								event.getUser().getAsMention(),
+								post.getAsMention()
+						))
 					.setAuthor(member.getEffectiveName(), null, member.getEffectiveAvatarUrl())
 					.setFooter(event.getUser().getId())
-					.setColor(Color.YELLOW)
-					.build())
+					.setColor(Color.YELLOW);
+
+			appendComment(eb, event.getOption("comment", null, OptionMapping::getAsString));
+			appendTags(eb, post);
+			eb.appendDescription("\n\n[Click to view]("+post.getJumpUrl()+")");
+
+			notifChannel.sendMessageEmbeds(eb.build())
 				.addActionRow(createAcknowledgementButton())
 				.queue();
 			event.reply("""
 					Successfully requested help.
-					
+
 					Note that this does NOT gurantee that anybody here has the time and knowledge to help you.
 					Abusing this command might result in moderative action taken against you.
 					""")
@@ -125,23 +127,31 @@ public void execute(@NotNull SlashCommandInteractionEvent event) {
 		}
 	}
 
-	private String getTagString(ThreadChannel post) {
-		String text = post
-			.getAppliedTags()
-			.stream()
-			.map(this::getForumTagText)
-			.map(tag -> "- " + tag)
-			.collect(Collectors.joining("\n"));
-		if(text.isEmpty()) {
-			text = "- <no tags>";
+	private void appendTags(EmbedBuilder eb, ThreadChannel post) {
+		List<ForumTag> tags = post.getAppliedTags();
+		if (!tags.isEmpty()) {
+			String text = tags
+					.stream()
+					.map(this::getForumTagText)
+					.map(tag -> "- " + tag)
+					.collect(Collectors.joining("\n"));
+			eb.appendDescription("\n\nTags:\n")
+			.appendDescription(text);
+		}
+	}
+
+	private void appendComment(EmbedBuilder eb, String comment) {
+		if (comment != null) {
+			eb
+				.appendDescription("\n\ncomment:\n")
+				.appendDescription("```\n"+StringUtils.standardSanitizer().compute(comment) + "\n```");
 		}
-		return text;
 	}
 
 	private Button createAcknowledgementButton() {
 		return Button.of(ButtonStyle.SECONDARY, ComponentIdBuilder.build("help-ping", "acknowledge"), "Mark as acknowledged");
 	}
-	
+
 	private Button createUndoAcknowledgementButton() {
 		return Button.of(ButtonStyle.SECONDARY, ComponentIdBuilder.build("help-ping", "unacknowledge"), "Mark as unacknowledged");
 	}
@@ -202,11 +212,11 @@ public void handleButton(ButtonInteractionEvent event, Button button) {
 		switch(id[1]) {
 		case "acknowledge" ->
 			acknowledgeChangeAction(event, true);
-		case "unacknowledge" -> 
+		case "unacknowledge" ->
 			acknowledgeChangeAction(event, false);
 		default -> event.reply("Unknown button").setEphemeral(true).queue();
 		}
-		
+
 	}
 
 	private void acknowledgeChangeAction(ButtonInteractionEvent event, boolean acknowledged) {
@@ -224,7 +234,7 @@ private void acknowledgeChangeAction(ButtonInteractionEvent event, boolean ackno
 		.setActionRow(acknowledged?createUndoAcknowledgementButton():createAcknowledgementButton())
 		.queue();
 	}
-	
+
 	private String getCurrentFormattedTimestamp() {
 		return TimeFormat.RELATIVE.format(Instant.now().toEpochMilli());
 	}
@@ -238,7 +248,7 @@ private String getForumTagText(ForumTag tag) {
 				.append(" ");
 		}
 		sb.append(tag.getName());
-		
+
 		return sb.toString();
 	}