Skip to content

Commit d5e7745

Browse files
committed
Defer edit for better UX and log false-positives
1 parent eeef578 commit d5e7745

File tree

1 file changed

+5
-2
lines changed
  • application/src/main/java/org/togetherjava/tjbot/commands/moderation/scam

1 file changed

+5
-2
lines changed

application/src/main/java/org/togetherjava/tjbot/commands/moderation/scam/ScamBlocker.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -276,6 +276,7 @@ private List<ActionRow> createConfirmDialog(@NotNull GuildMessageReceivedEvent e
276276

277277
@Override
278278
public void onButtonClick(@NotNull ButtonClickEvent event, @NotNull List<String> argsRaw) {
279+
ComponentIdArguments args = ComponentIdArguments.fromList(argsRaw);
279280
if (event.getMember().getRoles().stream().map(Role::getName).noneMatch(hasRequiredRole)) {
280281
event.reply(
281282
"You can not handle scam in this guild, since you do not have the required role.")
@@ -285,11 +286,13 @@ public void onButtonClick(@NotNull ButtonClickEvent event, @NotNull List<String>
285286
}
286287

287288
MessageUtils.disableButtons(event.getMessage());
289+
event.deferEdit().queue();
288290
if (event.getButton().getStyle() == ButtonStyle.DANGER) {
291+
logger.info(
292+
"Identified a false-positive scam (id '{}', hash '{}') in guild '{}' sent by author '{}'",
293+
args.messageId, args.contentHash, args.guildId, args.authorId);
289294
return;
290295
}
291-
292-
ComponentIdArguments args = ComponentIdArguments.fromList(argsRaw);
293296
Guild guild = event.getJDA().getGuildById(args.guildId);
294297
if (guild == null) {
295298
logger.debug(

0 commit comments

Comments
 (0)