X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fcore%2FCore.java;h=d60bbda40ee59a9fdacd15102a7886f5b046fb74;hp=7d38ed9373416acd97f26dcc04732831b94f502b;hb=563df6d3fb70d83ee8a7d2ee6b957434af7d3251;hpb=f2eaa4ab7f0055a704425f2159efaa5be46b7c5e

diff --git a/src/main/java/net/pterodactylus/sone/core/Core.java b/src/main/java/net/pterodactylus/sone/core/Core.java
index 7d38ed9..d60bbda 100644
--- a/src/main/java/net/pterodactylus/sone/core/Core.java
+++ b/src/main/java/net/pterodactylus/sone/core/Core.java
@@ -49,6 +49,8 @@ import net.pterodactylus.sone.core.ConfigurationSoneParser.InvalidParentAlbumFou
 import net.pterodactylus.sone.core.ConfigurationSoneParser.InvalidPostFound;
 import net.pterodactylus.sone.core.ConfigurationSoneParser.InvalidPostReplyFound;
 import net.pterodactylus.sone.core.Options.DefaultOption;
+import net.pterodactylus.sone.core.SoneChangeDetector.PostProcessor;
+import net.pterodactylus.sone.core.SoneChangeDetector.PostReplyProcessor;
 import net.pterodactylus.sone.core.SoneInserter.SetInsertionDelay;
 import net.pterodactylus.sone.core.event.ImageInsertFinishedEvent;
 import net.pterodactylus.sone.core.event.MarkPostKnownEvent;
@@ -344,9 +346,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	 */
 	@Override
 	public Collection<Sone> getSones() {
-		synchronized (sones) {
-			return ImmutableSet.copyOf(sones.values());
-		}
+		return database.getSones();
 	}
 
 	/**
@@ -360,9 +360,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	 */
 	@Override
 	public Optional<Sone> getSone(String id) {
-		synchronized (sones) {
-			return Optional.fromNullable(sones.get(id));
-		}
+		return database.getSone(id);
 	}
 
 	/**
@@ -370,9 +368,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	 */
 	@Override
 	public Collection<Sone> getLocalSones() {
-		synchronized (sones) {
-			return FluentIterable.from(sones.values()).filter(LOCAL_SONE_FILTER).toSet();
-		}
+		return database.getLocalSones();
 	}
 
 	/**
@@ -405,9 +401,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	 */
 	@Override
 	public Collection<Sone> getRemoteSones() {
-		synchronized (sones) {
-			return FluentIterable.from(sones.values()).filter(not(LOCAL_SONE_FILTER)).toSet();
-		}
+		return database.getRemoteSones();
 	}
 
 	/**
@@ -766,8 +760,8 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 			logger.log(Level.WARNING, "Given Identity is null!");
 			return null;
 		}
-		final Long latestEdition = fromNullable(tryParse(
-				identity.getProperty("Sone.LatestEdition"))).or(0L);
+		final Long latestEdition = tryParse(fromNullable(
+				identity.getProperty("Sone.LatestEdition")).or("0"));
 		synchronized (sones) {
 			final Sone sone = getRemoteSone(identity.getId(), true);
 			if (sone.isLocal()) {
@@ -946,7 +940,7 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 	 *            {@code true} if the stored Sone should be updated regardless
 	 *            of the age of the given Sone
 	 */
-	public void updateSone(Sone sone, boolean soneRescueMode) {
+	public void updateSone(final Sone sone, boolean soneRescueMode) {
 		Optional<Sone> storedSone = getSone(sone.getId());
 		if (storedSone.isPresent()) {
 			if (!soneRescueMode && !(sone.getTime() > storedSone.get().getTime())) {
@@ -954,72 +948,41 @@ public class Core extends AbstractService implements SoneProvider, PostProvider,
 				return;
 			}
 			/* find removed posts. */
-			Collection<Post> removedPosts = new ArrayList<Post>();
-			Collection<Post> newPosts = new ArrayList<Post>();
-			Collection<Post> existingPosts = database.getPosts(sone.getId());
-			for (Post oldPost : existingPosts) {
-				if (!sone.getPosts().contains(oldPost)) {
-					removedPosts.add(oldPost);
-				}
-			}
-			/* find new posts. */
-			for (Post newPost : sone.getPosts()) {
-				if (existingPosts.contains(newPost)) {
-					continue;
-				}
-				if (newPost.getTime() < getSoneFollowingTime(sone)) {
-					newPost.setKnown(true);
-				} else if (!newPost.isKnown()) {
-					newPosts.add(newPost);
-				}
-			}
-			/* store posts. */
-			database.storePosts(sone, sone.getPosts());
-			Collection<PostReply> newPostReplies = new ArrayList<PostReply>();
-			Collection<PostReply> removedPostReplies = new ArrayList<PostReply>();
-			if (!soneRescueMode) {
-				for (PostReply reply : storedSone.get().getReplies()) {
-					if (!sone.getReplies().contains(reply)) {
-						removedPostReplies.add(reply);
+			SoneChangeDetector soneChangeDetector = new SoneChangeDetector(storedSone.get());
+			soneChangeDetector.onNewPosts(new PostProcessor() {
+				@Override
+				public void processPost(Post post) {
+					if (post.getTime() < getSoneFollowingTime(sone)) {
+						post.setKnown(true);
+					} else if (!post.isKnown()) {
+						eventBus.post(new NewPostFoundEvent(post));
 					}
 				}
-			}
-			Set<PostReply> storedReplies = storedSone.get().getReplies();
-			for (PostReply reply : sone.getReplies()) {
-				if (storedReplies.contains(reply)) {
-					continue;
+			});
+			soneChangeDetector.onRemovedPosts(new PostProcessor() {
+				@Override
+				public void processPost(Post post) {
+					eventBus.post(new PostRemovedEvent(post));
 				}
-				if (reply.getTime() < getSoneFollowingTime(sone)) {
-					reply.setKnown(true);
-				} else if (!reply.isKnown()) {
-					newPostReplies.add(reply);
-				}
-			}
-			database.storePostReplies(sone, sone.getReplies());
-			for (Album album : storedSone.get().getRootAlbum().getAlbums()) {
-				database.removeAlbum(album);
-				for (Image image : album.getImages()) {
-					database.removeImage(image);
+			});
+			soneChangeDetector.onNewPostReplies(new PostReplyProcessor() {
+				@Override
+				public void processPostReply(PostReply postReply) {
+					if (postReply.getTime() < getSoneFollowingTime(sone)) {
+						postReply.setKnown(true);
+					} else if (!postReply.isKnown()) {
+						eventBus.post(new NewPostReplyFoundEvent(postReply));
+					}
 				}
-			}
-			for (Post removedPost : removedPosts) {
-				eventBus.post(new PostRemovedEvent(removedPost));
-			}
-			for (Post newPost : newPosts) {
-				eventBus.post(new NewPostFoundEvent(newPost));
-			}
-			for (PostReply removedPostReply : removedPostReplies) {
-				eventBus.post(new PostReplyRemovedEvent(removedPostReply));
-			}
-			for (PostReply newPostReply : newPostReplies) {
-				eventBus.post(new NewPostReplyFoundEvent(newPostReply));
-			}
-			for (Album album : toAllAlbums.apply(sone)) {
-				database.storeAlbum(album);
-				for (Image image : album.getImages()) {
-					database.storeImage(image);
+			});
+			soneChangeDetector.onRemovedPostReplies(new PostReplyProcessor() {
+				@Override
+				public void processPostReply(PostReply postReply) {
+					eventBus.post(new PostReplyRemovedEvent(postReply));
 				}
-			}
+			});
+			soneChangeDetector.detectChanges(sone);
+			database.storeSone(sone);
 			synchronized (sones) {
 				sone.setOptions(storedSone.get().getOptions());
 				sone.setKnown(storedSone.get().isKnown());