Add test for storing IDs on known post replies

[Sone.git] / src / main / java / net / pterodactylus / sone / data / impl / PostImpl.java

diff --git a/src/main/java/net/pterodactylus/sone/data/impl/PostImpl.java b/src/main/java/net/pterodactylus/sone/data/impl/PostImpl.java
index 2269738..d8d5086 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/data/impl/PostImpl.java
+++ b/src/main/java/net/pterodactylus/sone/data/impl/PostImpl.java
@@ -1,5 +1,5 @@
 /*
- * Sone - PostImpl.java - Copyright © 2010–2012 David Roden
+ * Sone - PostImpl.java - Copyright © 2010–2019 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,33 +17,37 @@
 
 package net.pterodactylus.sone.data.impl;
 
-import java.util.UUID;
+import static com.google.common.base.Optional.fromNullable;
 
 import net.pterodactylus.sone.data.Post;
 import net.pterodactylus.sone.data.Sone;
+import net.pterodactylus.sone.database.SoneProvider;
+
+import com.google.common.base.Optional;
 
 /**
  * A post is a short message that a user writes in his Sone to let other users
  * know what is going on.
- *
- * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
 public class PostImpl implements Post {
 
+       /** The Sone provider. */
+       private final SoneProvider soneProvider;
+
        /** The GUID of the post. */
-       private final UUID id;
+       private final String id;
 
-       /** The Sone this post belongs to. */
-       private volatile Sone sone;
+       /** The ID of the owning Sone. */
+       private final String soneId;
 
-       /** The Sone of the recipient. */
-       private volatile Sone recipient;
+       /** The ID of the recipient Sone. */
+       private final String recipientId;
 
        /** The time of the post (in milliseconds since Jan 1, 1970 UTC). */
-       private volatile long time;
+       private final long time;
 
        /** The text of the post. */
-       private volatile String text;
+       private final String text;
 
        /** Whether the post is known. */
        private volatile boolean known;
@@ -51,54 +55,24 @@ public class PostImpl implements Post {
        /**
         * Creates a new post.
         *
+        * @param soneProvider
+        *            The Sone provider
         * @param id
         *            The ID of the post
-        */
-       public PostImpl(String id) {
-               this(id, null, 0, null);
-       }
-
-       /**
-        * Creates a new post.
-        *
-        * @param sone
-        *            The Sone this post belongs to
-        * @param text
-        *            The text of the post
-        */
-       public PostImpl(Sone sone, String text) {
-               this(sone, System.currentTimeMillis(), text);
-       }
-
-       /**
-        * Creates a new post.
-        *
-        * @param sone
-        *            The Sone this post belongs to
-        * @param time
-        *            The time of the post (in milliseconds since Jan 1, 1970 UTC)
-        * @param text
-        *            The text of the post
-        */
-       public PostImpl(Sone sone, long time, String text) {
-               this(UUID.randomUUID().toString(), sone, time, text);
-       }
-
-       /**
-        * Creates a new post.
-        *
-        * @param id
-        *            The ID of the post
-        * @param sone
-        *            The Sone this post belongs to
+        * @param soneId
+        *            The ID of the Sone this post belongs to
+        * @param recipientId
+        *            The ID of the recipient of the post
         * @param time
         *            The time of the post (in milliseconds since Jan 1, 1970 UTC)
         * @param text
         *            The text of the post
         */
-       public PostImpl(String id, Sone sone, long time, String text) {
-               this.id = UUID.fromString(id);
-               this.sone = sone;
+       public PostImpl(SoneProvider soneProvider, String id, String soneId, String recipientId, long time, String text) {
+               this.soneProvider = soneProvider;
+               this.id = id;
+               this.soneId = soneId;
+               this.recipientId = recipientId;
                this.time = time;
                this.text = text;
        }
@@ -112,43 +86,36 @@ public class PostImpl implements Post {
         */
        @Override
        public String getId() {
-               return id.toString();
+               return id;
        }
 
-       /**
-        * {@inheritDoc}
-        */
        @Override
-       public Sone getSone() {
-               return sone;
+       public boolean isLoaded() {
+               return true;
        }
 
        /**
         * {@inheritDoc}
         */
        @Override
-       public PostImpl setSone(Sone sone) {
-               this.sone = sone;
-               return this;
+       public Sone getSone() {
+               return soneProvider.getSone(soneId);
        }
 
        /**
-        * {@inheritDoc}
+        * {@inheritDocs}
         */
        @Override
-       public Sone getRecipient() {
-               return recipient;
+       public Optional<String> getRecipientId() {
+               return fromNullable(recipientId);
        }
 
        /**
         * {@inheritDoc}
         */
        @Override
-       public PostImpl setRecipient(Sone recipient) {
-               if (!sone.equals(recipient)) {
-                       this.recipient = recipient;
-               }
-               return this;
+       public Optional<Sone> getRecipient() {
+               return fromNullable(soneProvider.getSone(recipientId));
        }
 
        /**
@@ -163,15 +130,6 @@ public class PostImpl implements Post {
         * {@inheritDoc}
         */
        @Override
-       public PostImpl setTime(long time) {
-               this.time = time;
-               return this;
-       }
-
-       /**
-        * {@inheritDoc}
-        */
-       @Override
        public String getText() {
                return text;
        }
@@ -180,15 +138,6 @@ public class PostImpl implements Post {
         * {@inheritDoc}
         */
        @Override
-       public PostImpl setText(String text) {
-               this.text = text;
-               return this;
-       }
-
-       /**
-        * {@inheritDoc}
-        */
-       @Override
        public boolean isKnown() {
                return known;
        }
@@ -231,7 +180,7 @@ public class PostImpl implements Post {
         */
        @Override
        public String toString() {
-               return getClass().getName() + "[id=" + id + ",sone=" + sone + ",time=" + time + ",text=" + text + "]";
+               return String.format("%s[id=%s,sone=%s,recipient=%s,time=%d,text=%s]", getClass().getName(), id, soneId, recipientId, time, text);
        }
 
 }