Stop logging when starting the plugin failed.

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

diff --git a/src/main/java/net/pterodactylus/sone/data/PostShell.java b/src/main/java/net/pterodactylus/sone/data/PostShell.java
index df66b20..05b0045 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/data/PostShell.java
+++ b/src/main/java/net/pterodactylus/sone/data/PostShell.java
@@ -17,10 +17,17 @@
 
 package net.pterodactylus.sone.data;
 
+import java.util.ArrayList;
 import java.util.Collections;
+import java.util.Comparator;
 import java.util.HashSet;
+import java.util.List;
 import java.util.Set;
 import java.util.UUID;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
+import net.pterodactylus.util.logging.Logging;
 
 /**
  * {@link Shell} around a {@link Post} that has not yet been retrieved from
@@ -30,6 +37,18 @@ import java.util.UUID;
  */
 public class PostShell extends Post implements Shell<Post> {
 
+       /** The logger. */
+       private static final Logger logger = Logging.getLogger(PostShell.class);
+
+       /** The shell creator. */
+       public static final ShellCreator<Post> creator = new ShellCreator<Post>() {
+
+               @Override
+               public Shell<Post> createShell(String id) {
+                       return new PostShell().setId(id);
+               }
+       };
+
        /** The GUID of the post. */
        private UUID id;
 
@@ -73,8 +92,13 @@ public class PostShell extends Post implements Shell<Post> {
         *            The ID of the post
         * @return This post shell (for method chaining)
         */
-       public PostShell setId(UUID id) {
-               this.id = id;
+       public PostShell setId(String id) {
+               try {
+                       this.id = UUID.fromString(id);
+               } catch (IllegalArgumentException iae1) {
+                       logger.log(Level.WARNING, "Invalid ID: “" + id + "”.", iae1);
+                       this.id = UUID.randomUUID();
+               }
                return this;
        }
 
@@ -107,7 +131,7 @@ public class PostShell extends Post implements Shell<Post> {
         */
        @Override
        public long getTime() {
-               return time;
+               return (time != null) ? time : 0;
        }
 
        /**
@@ -150,8 +174,17 @@ public class PostShell extends Post implements Shell<Post> {
         * @return All replies to this post
         */
        @Override
-       public Set<Reply> getReplies() {
-               return Collections.unmodifiableSet(replies);
+       public List<Reply> getReplies() {
+               List<Reply> sortedReplies = new ArrayList<Reply>(replies);
+               Collections.sort(sortedReplies, new Comparator<Reply>() {
+
+                       @Override
+                       public int compare(Reply leftReply, Reply rightReply) {
+                               return (int) Math.max(Integer.MIN_VALUE, Math.min(Integer.MAX_VALUE, leftReply.getTime() - rightReply.getTime()));
+                       }
+
+               });
+               return sortedReplies;
        }
 
        /**
@@ -190,7 +223,7 @@ public class PostShell extends Post implements Shell<Post> {
         */
        @Override
        public boolean canUnshell() {
-               return (id != null) && (sone != null) && (time != null) && (text != null);
+               return (id != null) && (sone != null) && (!(sone instanceof Shell<?>)) && (time != null) && (text != null);
        }
 
        /**