X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fnotify%2FListNotificationFilters.java;h=0a2a518078297a0407ef24acdc41f9a2d3f117e4;hb=3da1daa6488c642430858a4baacc0aa986a6ebc1;hp=fe024d6083d7b80daffce5c66430c02f53a36dfd;hpb=f0fefbfbc2318e460d5e1f3bd2879a6a21c53ce3;p=Sone.git diff --git a/src/main/java/net/pterodactylus/sone/notify/ListNotificationFilters.java b/src/main/java/net/pterodactylus/sone/notify/ListNotificationFilters.java index fe024d6..0a2a518 100644 --- a/src/main/java/net/pterodactylus/sone/notify/ListNotificationFilters.java +++ b/src/main/java/net/pterodactylus/sone/notify/ListNotificationFilters.java @@ -1,5 +1,5 @@ /* - * Sone - ListNotificationFilters.java - Copyright © 2010 David Roden + * Sone - ListNotificationFilters.java - Copyright © 2010–2015 David Roden * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -17,22 +17,39 @@ package net.pterodactylus.sone.notify; +import static com.google.common.collect.FluentIterable.from; + import java.util.ArrayList; import java.util.Collection; import java.util.List; +import javax.annotation.Nonnull; +import javax.inject.Inject; +import javax.inject.Singleton; import net.pterodactylus.sone.data.Post; -import net.pterodactylus.sone.data.Reply; +import net.pterodactylus.sone.data.PostReply; import net.pterodactylus.sone.data.Sone; import net.pterodactylus.util.notify.Notification; +import com.google.common.base.Optional; + /** * Filter for {@link ListNotification}s. * * @author David ‘Bombe’ Roden */ +@Singleton public class ListNotificationFilters { + private final PostVisibilityFilter postVisibilityFilter; + private final ReplyVisibilityFilter replyVisibilityFilter; + + @Inject + public ListNotificationFilters(@Nonnull PostVisibilityFilter postVisibilityFilter, @Nonnull ReplyVisibilityFilter replyVisibilityFilter) { + this.postVisibilityFilter = postVisibilityFilter; + this.replyVisibilityFilter = replyVisibilityFilter; + } + /** * Filters new-post and new-reply notifications in the given list of * notifications. If {@code currentSone} is null, new-post and @@ -42,67 +59,83 @@ public class ListNotificationFilters { * itself will be retained in the notifications. * * @param notifications - * The notifications to filter + * The notifications to filter * @param currentSone - * The current Sone, or {@code null} if not logged in + * The current Sone, or {@code null} if not logged in * @return The filtered notifications */ - public static List filterNotifications(List notifications, Sone currentSone) { - ListNotification newPostNotification = getNotification(notifications, "new-post-notification", Post.class); - if (newPostNotification != null) { - ListNotification filteredNotification = filterNewPostNotification(newPostNotification, currentSone); - int notificationIndex = notifications.indexOf(newPostNotification); - if (filteredNotification == null) { - notifications.remove(notificationIndex); - } else { - notifications.set(notificationIndex, filteredNotification); - } - } - ListNotification newReplyNotification = getNotification(notifications, "new-replies-notification", Reply.class); - if (newReplyNotification != null) { - ListNotification filteredNotification = filterNewReplyNotification(newReplyNotification, currentSone); - int notificationIndex = notifications.indexOf(newReplyNotification); - if (filteredNotification == null) { - notifications.remove(notificationIndex); + @SuppressWarnings("unchecked") + public List filterNotifications(Collection notifications, Sone currentSone) { + List filteredNotifications = new ArrayList(); + for (Notification notification : notifications) { + if (notification.getId().equals("new-sone-notification")) { + if ((currentSone != null) && !currentSone.getOptions().isShowNewSoneNotifications()) { + continue; + } + filteredNotifications.add(notification); + } else if (notification.getId().equals("new-post-notification")) { + if (currentSone == null) { + continue; + } + if (!currentSone.getOptions().isShowNewPostNotifications()) { + continue; + } + Optional> filteredNotification = filterNewPostNotification((ListNotification) notification, currentSone); + if (filteredNotification.isPresent()) { + filteredNotifications.add(filteredNotification.get()); + } + } else if (notification.getId().equals("new-reply-notification")) { + if (currentSone == null) { + continue; + } + if (!currentSone.getOptions().isShowNewReplyNotifications()) { + continue; + } + Optional> filteredNotification = + filterNewReplyNotification((ListNotification) notification, currentSone); + if (filteredNotification.isPresent()) { + filteredNotifications.add(filteredNotification.get()); + } + } else if (notification.getId().equals("mention-notification")) { + Optional> filteredNotification = filterNewPostNotification((ListNotification) notification, null); + if (filteredNotification.isPresent()) { + filteredNotifications.add(filteredNotification.get()); + } } else { - notifications.set(notificationIndex, filteredNotification); + filteredNotifications.add(notification); } } - return notifications; + return filteredNotifications; } /** * Filters the new posts of the given notification. If {@code currentSone} - * is {@code null}, {@code null} is returned and the notification is - * subsequently removed. Otherwise only posts that are posted by friend - * Sones of the given Sone are retained; all other posts are removed. + * is {@code null} and {@code soneRequired} is {@code true}, {@code null} is + * returned and the notification is subsequently removed. Otherwise only + * posts that are posted by friend Sones of the given Sone are retained; all + * other posts are removed. * * @param newPostNotification - * The new-post notification + * The new-post notification * @param currentSone - * The current Sone, or {@code null} if not logged in + * The current Sone, or {@code null} if not logged in * @return The filtered new-post notification, or {@code null} if the - * notification should be removed + * notification should be removed */ - public static ListNotification filterNewPostNotification(ListNotification newPostNotification, Sone currentSone) { - if (currentSone == null) { - return null; - } - List newPosts = new ArrayList(); - for (Post post : newPostNotification.getElements()) { - if (currentSone.hasFriend(post.getSone().getId()) || currentSone.equals(post.getSone()) || currentSone.equals(post.getRecipient())) { - newPosts.add(post); - } - } + @Nonnull + private Optional> filterNewPostNotification(@Nonnull ListNotification newPostNotification, + @Nonnull Sone currentSone) { + List newPosts = from(newPostNotification.getElements()).filter(postVisibilityFilter.isVisible(currentSone)).toList(); if (newPosts.isEmpty()) { - return null; + return Optional.absent(); } if (newPosts.size() == newPostNotification.getElements().size()) { - return newPostNotification; + return Optional.of(newPostNotification); } ListNotification filteredNotification = new ListNotification(newPostNotification); filteredNotification.setElements(newPosts); - return filteredNotification; + filteredNotification.setLastUpdateTime(newPostNotification.getLastUpdatedTime()); + return Optional.of(filteredNotification); } /** @@ -113,57 +146,25 @@ public class ListNotificationFilters { * replies are removed. * * @param newReplyNotification - * The new-reply notification + * The new-reply notification * @param currentSone - * The current Sone, or {@code null} if not logged in + * The current Sone, or {@code null} if not logged in * @return The filtered new-reply notification, or {@code null} if the - * notification should be removed + * notification should be removed */ - public static ListNotification filterNewReplyNotification(ListNotification newReplyNotification, Sone currentSone) { - if (currentSone == null) { - return null; - } - List newReplies = new ArrayList(); - for (Reply reply : newReplyNotification.getElements()) { - if (((reply.getPost().getSone() != null) && currentSone.hasFriend(reply.getPost().getSone().getId())) || currentSone.equals(reply.getPost().getSone()) || currentSone.equals(reply.getPost().getRecipient())) { - newReplies.add(reply); - } - } + private Optional> filterNewReplyNotification(ListNotification newReplyNotification, + @Nonnull Sone currentSone) { + List newReplies = from(newReplyNotification.getElements()).filter(replyVisibilityFilter.isVisible(currentSone)).toList(); if (newReplies.isEmpty()) { - return null; + return Optional.absent(); } if (newReplies.size() == newReplyNotification.getElements().size()) { - return newReplyNotification; + return Optional.of(newReplyNotification); } - ListNotification filteredNotification = new ListNotification(newReplyNotification); + ListNotification filteredNotification = new ListNotification(newReplyNotification); filteredNotification.setElements(newReplies); - return filteredNotification; - } - - /** - * Finds the notification with the given ID in the list of notifications and - * returns it. - * - * @param - * The type of the item in the notification - * @param notifications - * The notification to search - * @param notificationId - * The ID of the requested notification - * @param notificationElementClass - * The class of the notification item - * @return The requested notification, or {@code null} if no notification - * with the given ID could be found - */ - @SuppressWarnings("unchecked") - private static ListNotification getNotification(Collection notifications, String notificationId, Class notificationElementClass) { - for (Notification notification : notifications) { - if (!notificationId.equals(notification.getId())) { - continue; - } - return (ListNotification) notification; - } - return null; + filteredNotification.setLastUpdateTime(newReplyNotification.getLastUpdatedTime()); + return Optional.of(filteredNotification); } }