Remove hasRecipient method.

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

diff --git a/src/main/java/net/pterodactylus/sone/data/Post.java b/src/main/java/net/pterodactylus/sone/data/Post.java
index 3486bed..6523e9b 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/data/Post.java
+++ b/src/main/java/net/pterodactylus/sone/data/Post.java
@@ -19,6 +19,7 @@ package net.pterodactylus.sone.data;
 
 import java.util.Comparator;
 
+import com.google.common.base.Optional;
 import com.google.common.base.Predicate;
 
 /**
@@ -68,29 +69,23 @@ public interface Post {
        public Sone getSone();
 
        /**
-        * Sets the Sone of this post.
+        * Returns the ID of the recipient {@link Sone}, or
+        * {@link Optional#absent()} if this post does not have a recipient.
         *
-        * @param sone
-        *            The Sone of this post
-        * @return This post (for method chaining)
+        * @return The ID of the recipient, or {@link Optional#absent()}
         */
-       public Post setSone(Sone sone);
+       public Optional<String> getRecipientId();
 
        /**
-        * Returns the recipient of this post, if any.
+        * Returns the recipient of this post, if any. As this method can return
+        * {@link Optional#absent()} if the post has a recipient which has not yet
+        * been loaded, it is recommended to use {@link #hasRecipient()} to check
+        * for the presence of a recipient.
         *
-        * @return The recipient of this post, or {@code null}
+        * @return The recipient of this post, or {@link Optional#absent()} if there
+        *         is no recipient
         */
-       public Sone getRecipient();
-
-       /**
-        * Sets the recipient of this post.
-        *
-        * @param recipient
-        *            The recipient of this post, or {@code null}
-        * @return This post (for method chaining)
-        */
-       public Post setRecipient(Sone recipient);
+       public Optional<Sone> getRecipient();
 
        /**
         * Returns the time of the post.
@@ -100,15 +95,6 @@ public interface Post {
        public long getTime();
 
        /**
-        * Sets the time of this post.
-        *
-        * @param time
-        *            The time of this post (in milliseconds since Jan 1, 1970 UTC)
-        * @return This post (for method chaining)
-        */
-       public Post setTime(long time);
-
-       /**
         * Returns the text of the post.
         *
         * @return The text of the post
@@ -116,15 +102,6 @@ public interface Post {
        public String getText();
 
        /**
-        * Sets the text of this post.
-        *
-        * @param text
-        *            The text of this post
-        * @return This post (for method chaining)
-        */
-       public Post setText(String text);
-
-       /**
         * Returns whether this post is known.
         *
         * @return {@code true} if this post is known, {@code false} otherwise