Repository: james-project Updated Branches: refs/heads/master 06ddc93c5 -> f2005bf0f
JAMES-2244 Solve (some) IntelliJ warnings Project: http://git-wip-us.apache.org/repos/asf/james-project/repo Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/d1ea4a20 Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/d1ea4a20 Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/d1ea4a20 Branch: refs/heads/master Commit: d1ea4a20629692f4a1ff92e02e8a0340e5bb3097 Parents: 439e747 Author: benwa <btell...@linagora.com> Authored: Thu Nov 30 10:49:54 2017 +0700 Committer: Antoine Duprat <adup...@linagora.com> Committed: Fri Dec 1 13:51:42 2017 +0100 ---------------------------------------------------------------------- .../james/jmap/methods/SetMessagesUpdateProcessor.java | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/james-project/blob/d1ea4a20/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/SetMessagesUpdateProcessor.java ---------------------------------------------------------------------- diff --git a/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/SetMessagesUpdateProcessor.java b/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/SetMessagesUpdateProcessor.java index f7cbe62..9086ea6 100644 --- a/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/SetMessagesUpdateProcessor.java +++ b/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/SetMessagesUpdateProcessor.java @@ -123,9 +123,7 @@ public class SetMessagesUpdateProcessor implements SetMessagesProcessor { if (messages.isEmpty()) { addMessageIdNotFoundToResponse(messageId, builder); } else { - setInMailboxes(messageId, updateMessagePatch, - messages.stream().map(MessageResult::getFlags), - mailboxSession); + setInMailboxes(messageId, updateMessagePatch, mailboxSession); Optional<MailboxException> updateError = messages.stream() .flatMap(message -> updateFlags(messageId, updateMessagePatch, mailboxSession, message)) .findAny(); @@ -252,7 +250,7 @@ public class SetMessagesUpdateProcessor implements SetMessagesProcessor { private List<Flags> flagsFromOldKeyword(List<MessageResult> messagesToBeUpdated, OldKeyword oldKeyword) { return messagesToBeUpdated.stream() .map(MessageResult::getFlags) - .map(flags -> oldKeyword.applyToState(flags)) + .map(oldKeyword::applyToState) .collect(Guavate.toImmutableList()); } @@ -320,7 +318,7 @@ public class SetMessagesUpdateProcessor implements SetMessagesProcessor { .orElse(null); } - private void setInMailboxes(MessageId messageId, UpdateMessagePatch updateMessagePatch, Stream<Flags> originalFlags, MailboxSession mailboxSession) throws MailboxException { + private void setInMailboxes(MessageId messageId, UpdateMessagePatch updateMessagePatch, MailboxSession mailboxSession) throws MailboxException { Optional<List<String>> serializedMailboxIds = updateMessagePatch.getMailboxIds(); if (serializedMailboxIds.isPresent()) { List<MailboxId> mailboxIds = serializedMailboxIds.get() --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org