From 767aec7729a326ff1ccc225d8623a24dd1e698d5 Mon Sep 17 00:00:00 2001 From: Vitaly Takmazov Date: Thu, 5 Apr 2018 22:17:57 +0300 Subject: server: markdown command reply * also fixes new messages with one tag and attachment --- .../main/java/com/juick/server/CommandsManager.java | 20 ++++++++++++++------ .../java/com/juick/server/helpers/CommandResult.java | 13 ++++++++++--- .../main/java/com/juick/server/util/TagUtils.java | 4 ++-- 3 files changed, 26 insertions(+), 11 deletions(-) (limited to 'juick-common/src/main/java/com') diff --git a/juick-common/src/main/java/com/juick/server/CommandsManager.java b/juick-common/src/main/java/com/juick/server/CommandsManager.java index 9bde0884..4f115577 100644 --- a/juick-common/src/main/java/com/juick/server/CommandsManager.java +++ b/juick-common/src/main/java/com/juick/server/CommandsManager.java @@ -90,12 +90,15 @@ public class CommandsManager { groups.add(matcher.group(i)); } } - return (CommandResult) getClass().getMethod(cmd.get().getName(), User.class, Jid.class, URI.class, String[].class) + CommandResult commandResult = (CommandResult) getClass().getMethod(cmd.get().getName(), User.class, Jid.class, URI.class, String[].class) .invoke(this, user, from, attachment, groups.toArray(new String[groups.size()])); + if (StringUtils.isNotEmpty(commandResult.getText())) { + return commandResult; + } } // new message List tags = tagService.fromString(input, false); - String body = input.substring(TagUtils.toString(tags).length()); + String body = TagUtils.toString(tags).equals(input) ? StringUtils.EMPTY : input.substring(TagUtils.toString(tags).length()).trim(); String attachmentType = StringUtils.isNotEmpty(attachment.toString()) ? attachment.toString().substring(attachment.toString().length() - 3) : null; int mid = messagesService.createMessage(user.getUid(), body, attachmentType, tags); subscriptionService.subscribeMessage(mid, user.getUid()); @@ -107,7 +110,7 @@ public class CommandsManager { } com.juick.Message msg = messagesService.getMessage(mid); applicationEventPublisher.publishEvent(new MessageEvent(this, msg)); - return CommandResult.fromMessage(msg, StringUtils.EMPTY); + return CommandResult.build(msg, StringUtils.EMPTY, String.format("[New message](%s) posted", PlainTextFormatter.formatUrl(msg))); } @UserCommand(pattern = "^ping$", patternFlags = Pattern.CASE_INSENSITIVE, @@ -407,9 +410,13 @@ public class CommandsManager { } @UserCommand(pattern = "^\\*(\\S+)(\\+?)$", help = "*tag - Show last messages with tag") public CommandResult commandShowTag(User user, Jid from, URI attachment, String... arguments) { + if (StringUtils.isNotEmpty(attachment.toString())) { + // new message with tag + return CommandResult.fromString(StringUtils.EMPTY); + } Tag tag = tagService.getTag(arguments[0], false); if (tag != null) { - // TODO: synonims + // TODO: synonyms List mids = messagesService.getTag(tag.TID, user.getUid(), 0, 10); return CommandResult.fromString("Last messages with *" + tag.getName() + ":\n" + printMessages(mids, true)); } @@ -480,8 +487,9 @@ public class CommandsManager { } Message reply = messagesService.getReply(mid, newrid); applicationEventPublisher.publishEvent(new MessageEvent(this, reply)); - return CommandResult.fromMessage(reply,"Reply posted.\n#" + mid + "/" + newrid + " " - + "https://juick.com/" + mid + "#" + newrid); + return CommandResult.build(reply,"Reply posted.\n#" + mid + "/" + newrid + " " + + "https://juick.com/" + mid + "#" + newrid, + String.format("[Reply](%s) posted", PlainTextFormatter.formatUrl(reply))); } } diff --git a/juick-common/src/main/java/com/juick/server/helpers/CommandResult.java b/juick-common/src/main/java/com/juick/server/helpers/CommandResult.java index 29244115..a772153b 100644 --- a/juick-common/src/main/java/com/juick/server/helpers/CommandResult.java +++ b/juick-common/src/main/java/com/juick/server/helpers/CommandResult.java @@ -2,21 +2,28 @@ package com.juick.server.helpers; import com.juick.Message; +import java.util.Optional; + public class CommandResult { private String text; + private String markdown; private Message newMessage; public String getText() { return text; } + public String getMarkdown() { + return markdown; + } - public Message getNewMessage() { - return newMessage; + public Optional getNewMessage() { + return Optional.of(newMessage); } - public static CommandResult fromMessage(Message newMessage, String text) { + public static CommandResult build(Message newMessage, String text, String markdown) { CommandResult result = new CommandResult(); result.newMessage = newMessage; result.text = text; + result.markdown = markdown; return result; } public static CommandResult fromString(String text) { diff --git a/juick-common/src/main/java/com/juick/server/util/TagUtils.java b/juick-common/src/main/java/com/juick/server/util/TagUtils.java index 9edeab32..cb828933 100644 --- a/juick-common/src/main/java/com/juick/server/util/TagUtils.java +++ b/juick-common/src/main/java/com/juick/server/util/TagUtils.java @@ -36,7 +36,7 @@ public class TagUtils { if (CollectionUtils.isEmpty(tags)) return StringUtils.EMPTY; - return tags.stream().map(t -> " *" + t.getName()) - .collect(Collectors.joining()); + return tags.stream().map(t -> "*" + t.getName()) + .collect(Collectors.joining(" ")); } } -- cgit v1.2.3