X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fcore%2FCore.java;h=8c05fe4ab856fc22c4af7fae7206aaccd4e8b644;hp=ccf633c180b3a3a01fe7c5445df3d70c6411fd4f;hb=2bacfa78f3191fd9847574a6c8b218a4882844a4;hpb=4b42574c9ec4490100ac3e87b9c2f3aac965a202

diff --git a/src/main/java/net/pterodactylus/sone/core/Core.java b/src/main/java/net/pterodactylus/sone/core/Core.java
index ccf633c..8c05fe4 100644
--- a/src/main/java/net/pterodactylus/sone/core/Core.java
+++ b/src/main/java/net/pterodactylus/sone/core/Core.java
@@ -48,8 +48,6 @@ import net.pterodactylus.sone.core.ConfigurationSoneParser.InvalidImageFound;
 import net.pterodactylus.sone.core.ConfigurationSoneParser.InvalidParentAlbumFound;
 import net.pterodactylus.sone.core.ConfigurationSoneParser.InvalidPostFound;
 import net.pterodactylus.sone.core.ConfigurationSoneParser.InvalidPostReplyFound;
-import net.pterodactylus.sone.core.SoneChangeDetector.PostProcessor;
-import net.pterodactylus.sone.core.SoneChangeDetector.PostReplyProcessor;
 import net.pterodactylus.sone.core.event.ImageInsertFinishedEvent;
 import net.pterodactylus.sone.core.event.InsertionDelayChangedEvent;
 import net.pterodactylus.sone.core.event.MarkPostKnownEvent;
@@ -113,8 +111,6 @@ import kotlin.jvm.functions.Function1;
 
 /**
  * The Sone core.
- *
- * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
 @Singleton
 public class Core extends AbstractService implements SoneProvider, PostProvider, PostReplyProvider {
@@ -160,18 +156,18 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 
 	/** Locked local Sones. */
 	/* synchronize on itself. */
-	private final Set<Sone> lockedSones = new HashSet<Sone>();
+	private final Set<Sone> lockedSones = new HashSet<>();
 
 	/** Sone inserters. */
 	/* synchronize access on this on sones. */
-	private final Map<Sone, SoneInserter> soneInserters = new HashMap<Sone, SoneInserter>();
+	private final Map<Sone, SoneInserter> soneInserters = new HashMap<>();
 
 	/** Sone rescuers. */
 	/* synchronize access on this on sones. */
-	private final Map<Sone, SoneRescuer> soneRescuers = new HashMap<Sone, SoneRescuer>();
+	private final Map<Sone, SoneRescuer> soneRescuers = new HashMap<>();
 
 	/** All known Sones. */
-	private final Set<String> knownSones = new HashSet<String>();
+	private final Set<String> knownSones = new HashSet<>();
 
 	/** The post database. */
 	private final Database database;
@@ -180,7 +176,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	private final Multimap<OwnIdentity, Identity> trustedIdentities = Multimaps.synchronizedSetMultimap(HashMultimap.<OwnIdentity, Identity>create());
 
 	/** All temporary images. */
-	private final Map<String, TemporaryImage> temporaryImages = new HashMap<String, TemporaryImage>();
+	private final Map<String, TemporaryImage> temporaryImages = new HashMap<>();
 
 	/** Ticker for threads that mark own elements as known. */
 	private final ScheduledExecutorService localElementTicker = Executors.newScheduledThreadPool(1);
@@ -448,7 +444,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	 * @return The Sones that like the given post
 	 */
 	public Set<Sone> getLikes(Post post) {
-		Set<Sone> sones = new HashSet<Sone>();
+		Set<Sone> sones = new HashSet<>();
 		for (Sone sone : getSones()) {
 			if (sone.getLikedPostIds().contains(post.getId())) {
 				sones.add(sone);
@@ -465,7 +461,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	 * @return The Sones that like the given reply
 	 */
 	public Set<Sone> getLikes(PostReply reply) {
-		Set<Sone> sones = new HashSet<Sone>();
+		Set<Sone> sones = new HashSet<>();
 		for (Sone sone : getSones()) {
 			if (sone.getLikedReplyIds().contains(reply.getId())) {
 				sones.add(sone);
@@ -853,44 +849,33 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 		}
 	}
 
-	private List<Object> collectEventsForChangesInSone(Sone oldSone,
-			final Sone newSone) {
-		final List<Object> events = new ArrayList<Object>();
-		SoneChangeDetector soneChangeDetector = new SoneChangeDetector(
-				oldSone);
-		soneChangeDetector.onNewPosts(new PostProcessor() {
-			@Override
-			public void processPost(Post post) {
-				if (post.getTime() < database.getFollowingTime(newSone.getId())) {
-					post.setKnown(true);
-				} else if (!post.isKnown()) {
-					events.add(new NewPostFoundEvent(post));
-				}
-			}
-		});
-		soneChangeDetector.onRemovedPosts(new PostProcessor() {
-			@Override
-			public void processPost(Post post) {
-				events.add(new PostRemovedEvent(post));
-			}
-		});
-		soneChangeDetector.onNewPostReplies(new PostReplyProcessor() {
-			@Override
-			public void processPostReply(PostReply postReply) {
-				if (postReply.getTime() < database.getFollowingTime(newSone.getId())) {
-					postReply.setKnown(true);
-				} else if (!postReply.isKnown()) {
-					events.add(new NewPostReplyFoundEvent(postReply));
-				}
+	private List<Object> collectEventsForChangesInSone(Sone oldSone, Sone newSone) {
+		List<Object> events = new ArrayList<>();
+		SoneComparison soneComparison = new SoneComparison(oldSone, newSone);
+		for (Post newPost : soneComparison.getNewPosts()) {
+			if (newPost.getSone().equals(newSone)) {
+				newPost.setKnown(true);
+			} else if (newPost.getTime() < database.getFollowingTime(newSone.getId())) {
+				newPost.setKnown(true);
+			} else if (!newPost.isKnown()) {
+				events.add(new NewPostFoundEvent(newPost));
 			}
-		});
-		soneChangeDetector.onRemovedPostReplies(new PostReplyProcessor() {
-			@Override
-			public void processPostReply(PostReply postReply) {
-				events.add(new PostReplyRemovedEvent(postReply));
+		}
+		for (Post post : soneComparison.getRemovedPosts()) {
+			events.add(new PostRemovedEvent(post));
+		}
+		for (PostReply postReply : soneComparison.getNewPostReplies()) {
+			if (postReply.getSone().equals(newSone)) {
+				postReply.setKnown(true);
+			} else if (postReply.getTime() < database.getFollowingTime(newSone.getId())) {
+				postReply.setKnown(true);
+			} else if (!postReply.isKnown()) {
+				events.add(new NewPostReplyFoundEvent(postReply));
 			}
-		});
-		soneChangeDetector.detectChanges(newSone);
+		}
+		for (PostReply postReply : soneComparison.getRemovedPostReplies()) {
+			events.add(new PostReplyRemovedEvent(postReply));
+		}
 		return events;
 	}