Skip to content

Commit b0043a1

Browse files
DevSerendipityZabuzard
authored andcommitted
Resolved the issue#240, with making all the tags be on the new line to make it look like a bullet point, and sorted the tags alphabetically, with adding a WARNING to when the lines reach above 200
1 parent 1a87811 commit b0043a1

1 file changed

Lines changed: 20 additions & 5 deletions

File tree

application/src/main/java/org/togetherjava/tjbot/commands/tags/TagsCommand.java

Lines changed: 20 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,12 @@
88
import org.togetherjava.tjbot.commands.SlashCommandVisibility;
99
import org.togetherjava.tjbot.commands.utils.MessageUtils;
1010

11+
import java.util.ArrayList;
1112
import java.util.List;
1213
import java.util.Objects;
14+
import java.util.logging.Level;
15+
import java.util.logging.Logger;
16+
import java.util.stream.Collectors;
1317

1418
/**
1519
* Implements the {@code /tags} command which lets the bot respond with all available tags.
@@ -42,11 +46,22 @@ public TagsCommand(TagSystem tagSystem) {
4246
@Override
4347
public void onSlashCommand(@NotNull SlashCommandEvent event) {
4448
// TODO A list might be better than comma separated, which is hard to read
45-
event.replyEmbeds(MessageUtils.generateEmbed("All available tags",
46-
String.join(", ", tagSystem.getAllIds()), event.getUser(), TagSystem.AMBIENT_COLOR))
47-
.addActionRow(
48-
TagSystem.createDeleteButton(generateComponentId(event.getUser().getId())))
49-
.queue();
49+
50+
ArrayList<String> list = new ArrayList<>(tagSystem.getAllIds());
51+
if (list.size() > 200) {
52+
Logger logger = Logger.getLogger(TagsCommand.class.getName());
53+
logger.setLevel(Level.WARNING);
54+
logger.warning("- WARNING - TAGS ARE BEYOND 200 LINES ");
55+
}
56+
57+
event
58+
.replyEmbeds(MessageUtils.generateEmbed("All available tags",
59+
"* " + String.join("\n ",
60+
list.stream().sorted().collect(Collectors.joining("\n * "))),
61+
event.getUser(), TagSystem.AMBIENT_COLOR))
62+
.addActionRow(
63+
TagSystem.createDeleteButton(generateComponentId(event.getUser().getId())))
64+
.queue();
5065
}
5166

5267
@Override

0 commit comments

Comments
 (0)