X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fdata%2Fimpl%2FSoneImpl.java;h=24792763c471032b345278998095d8ab3ae97cc0;hb=90d193f69c4aa12c232f937a9d2b7d5afdda81dd;hp=551e2298b8f62c512e2e20c534e86b693c3b0d15;hpb=56a2875b53db60fc26dd7ee089d721227508703c;p=Sone.git

diff --git a/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java b/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
index 551e229..2479276 100644
--- a/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
+++ b/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
@@ -25,6 +25,7 @@ import java.net.MalformedURLException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import java.util.concurrent.CopyOnWriteArraySet;
@@ -47,6 +48,7 @@ import net.pterodactylus.sone.freenet.wot.OwnIdentity;
 
 import freenet.keys.FreenetURI;
 
+import com.google.common.collect.FluentIterable;
 import com.google.common.hash.Hasher;
 import com.google.common.hash.Hashing;
 
@@ -78,7 +80,7 @@ public class SoneImpl implements LocalSone {
 	private volatile long latestEdition;
 
 	/** The time of the last inserted update. */
-	private volatile long time;
+	private final long time;
 
 	/** The status of this Sone. */
 	private volatile SoneStatus status = SoneStatus.unknown;
@@ -87,13 +89,13 @@ public class SoneImpl implements LocalSone {
 	private volatile Profile profile = new Profile(this);
 
 	/** The client used by the Sone. */
-	private volatile Client client;
+	private final Client client;
 
 	/** Whether this Sone is known. */
 	private volatile boolean known;
 
 	/** All posts. */
-	private final Set<Post> posts = new CopyOnWriteArraySet<Post>();
+	private final Collection<Post> posts = new HashSet<Post>();
 
 	/** All replies. */
 	private final Set<PostReply> replies = new CopyOnWriteArraySet<PostReply>();
@@ -119,11 +121,14 @@ public class SoneImpl implements LocalSone {
 	 * @param local
 	 * 		{@code true} if the Sone is a local Sone, {@code false} otherwise
 	 */
-	public SoneImpl(Database database, Identity identity, boolean local) {
+	public SoneImpl(Database database, Identity identity, boolean local, long time, Client client, Collection<Post> posts) {
 		this.database = database;
 		this.id = identity.getId();
 		this.identity = identity;
 		this.local = local;
+		this.time = time;
+		this.client = client;
+		this.posts.addAll(posts);
 	}
 
 	//
@@ -239,18 +244,6 @@ public class SoneImpl implements LocalSone {
 	}
 
 	/**
-	 * Sets the time of the last inserted update of this Sone.
-	 *
-	 * @param time
-	 * 		The time of the update (in milliseconds since Jan 1, 1970 UTC)
-	 * @return This Sone (for method chaining)
-	 */
-	public Sone setTime(long time) {
-		this.time = time;
-		return this;
-	}
-
-	/**
 	 * Returns the status of this Sone.
 	 *
 	 * @return The status of this Sone
@@ -313,7 +306,6 @@ public class SoneImpl implements LocalSone {
 	 * @return This Sone (for method chaining)
 	 */
 	public Sone setClient(Client client) {
-		this.client = client;
 		return this;
 	}
 
@@ -365,12 +357,9 @@ public class SoneImpl implements LocalSone {
 	 * @return All posts of this Sone
 	 */
 	public List<Post> getPosts() {
-		List<Post> sortedPosts;
 		synchronized (this) {
-			sortedPosts = new ArrayList<Post>(posts);
+			return FluentIterable.from(posts).toSortedList(Post.TIME_COMPARATOR);
 		}
-		Collections.sort(sortedPosts, Post.TIME_COMPARATOR);
-		return sortedPosts;
 	}
 
 	/**
@@ -381,10 +370,6 @@ public class SoneImpl implements LocalSone {
 	 * @return This Sone (for method chaining)
 	 */
 	public Sone setPosts(Collection<Post> posts) {
-		synchronized (this) {
-			this.posts.clear();
-			this.posts.addAll(posts);
-		}
 		return this;
 	}