Rename getReply() to getPostReply(), remove convenience method.
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Tue, 15 Jan 2013 08:57:43 +0000 (09:57 +0100)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Wed, 16 Jan 2013 06:14:54 +0000 (07:14 +0100)
src/main/java/net/pterodactylus/sone/core/Core.java
src/main/java/net/pterodactylus/sone/core/SoneDownloader.java
src/main/java/net/pterodactylus/sone/fcp/AbstractSoneCommand.java
src/main/java/net/pterodactylus/sone/web/DeleteReplyPage.java
src/main/java/net/pterodactylus/sone/web/MarkAsKnownPage.java
src/main/java/net/pterodactylus/sone/web/SearchPage.java
src/main/java/net/pterodactylus/sone/web/ajax/DeleteReplyAjaxPage.java
src/main/java/net/pterodactylus/sone/web/ajax/GetLikesAjaxPage.java
src/main/java/net/pterodactylus/sone/web/ajax/GetReplyAjaxPage.java
src/main/java/net/pterodactylus/sone/web/ajax/GetTimesAjaxPage.java
src/main/java/net/pterodactylus/sone/web/ajax/MarkAsKnownAjaxPage.java

index 5843b30..b25d8e3 100644 (file)
@@ -609,18 +609,6 @@ public class Core extends AbstractService implements IdentityListener, UpdateLis
 
        /**
         * Returns the reply with the given ID. If there is no reply with the given
-        * ID yet, a new one is created.
-        *
-        * @param replyId
-        *            The ID of the reply to get
-        * @return The reply
-        */
-       public PostReply getReply(String replyId) {
-               return getReply(replyId, true);
-       }
-
-       /**
-        * Returns the reply with the given ID. If there is no reply with the given
         * ID yet, a new one is created, unless {@code create} is false in which
         * case {@code null} is returned.
         *
@@ -631,7 +619,7 @@ public class Core extends AbstractService implements IdentityListener, UpdateLis
         *            to return {@code null} if no reply can be found
         * @return The reply, or {@code null} if there is no such reply
         */
-       public PostReply getReply(String replyId, boolean create) {
+       public PostReply getPostReply(String replyId, boolean create) {
                synchronized (replies) {
                        PostReply reply = replies.get(replyId);
                        if (create && (reply == null)) {
@@ -1382,7 +1370,7 @@ public class Core extends AbstractService implements IdentityListener, UpdateLis
                                logger.log(Level.WARNING, "Invalid reply found, aborting load!");
                                return;
                        }
-                       replies.add(getReply(replyId).setSone(sone).setPost(getPost(postId)).setTime(replyTime).setText(replyText));
+                       replies.add(getPostReply(replyId, true).setSone(sone).setPost(getPost(postId)).setTime(replyTime).setText(replyText));
                }
 
                /* load post likes. */
index 72ff008..39dbcd5 100644 (file)
@@ -403,7 +403,7 @@ public class SoneDownloader extends AbstractService {
                                        return null;
                                }
                                try {
-                                       replies.add(core.getReply(replyId).setSone(sone).setPost(core.getPost(replyPostId)).setTime(Long.parseLong(replyTime)).setText(replyText));
+                                       replies.add(core.getPostReply(replyId, true).setSone(sone).setPost(core.getPost(replyPostId)).setTime(Long.parseLong(replyTime)).setText(replyText));
                                } catch (NumberFormatException nfe1) {
                                        /* TODO - mark Sone as bad. */
                                        logger.log(Level.WARNING, String.format("Downloaded reply for Sone %s with invalid time: %s", sone, replyTime));
index 1944570..1eb488e 100644 (file)
@@ -209,7 +209,7 @@ public abstract class AbstractSoneCommand extends AbstractCommand {
        protected PostReply getReply(SimpleFieldSet simpleFieldSet, String parameterName) throws FcpException {
                try {
                        String replyId = simpleFieldSet.getString(parameterName);
-                       PostReply reply = core.getReply(replyId, false);
+                       PostReply reply = core.getPostReply(replyId, false);
                        if (reply == null) {
                                throw new FcpException("Could not load reply from “" + replyId + "”.");
                        }
index 900acd9..0f010b7 100644 (file)
@@ -53,7 +53,7 @@ public class DeleteReplyPage extends SoneTemplatePage {
        protected void processTemplate(FreenetRequest request, TemplateContext templateContext) throws RedirectException {
                super.processTemplate(request, templateContext);
                String replyId = request.getHttpRequest().getPartAsStringFailsafe("reply", 36);
-               PostReply reply = webInterface.getCore().getReply(replyId);
+               PostReply reply = webInterface.getCore().getPostReply(replyId, false);
                String returnPage = request.getHttpRequest().getPartAsStringFailsafe("returnPage", 256);
                if (request.getMethod() == Method.POST) {
                        if (!webInterface.getCore().isLocalSone(reply.getSone())) {
index 4f8e071..409c748 100644 (file)
@@ -71,7 +71,7 @@ public class MarkAsKnownPage extends SoneTemplatePage {
                                }
                                webInterface.getCore().markPostKnown(post);
                        } else if (type.equals("reply")) {
-                               PostReply reply = webInterface.getCore().getReply(id, false);
+                               PostReply reply = webInterface.getCore().getPostReply(id, false);
                                if (reply == null) {
                                        continue;
                                }
index 0f12abf..6d300ed 100644 (file)
@@ -346,7 +346,7 @@ public class SearchPage extends SoneTemplatePage {
         */
        private String getReplyPostId(String phrase) {
                String replyId = phrase.startsWith("reply://") ? phrase.substring(8) : phrase;
-               return (webInterface.getCore().getReply(replyId, false) != null) ? webInterface.getCore().getReply(replyId, false).getPost().getId() : null;
+               return (webInterface.getCore().getPostReply(replyId, false) != null) ? webInterface.getCore().getPostReply(replyId, false).getPost().getId() : null;
        }
 
        /**
index 0442117..857c777 100644 (file)
@@ -49,7 +49,7 @@ public class DeleteReplyAjaxPage extends JsonPage {
        @Override
        protected JsonObject createJsonObject(FreenetRequest request) {
                String replyId = request.getHttpRequest().getParam("reply");
-               PostReply reply = webInterface.getCore().getReply(replyId);
+               PostReply reply = webInterface.getCore().getPostReply(replyId, false);
                if (reply == null) {
                        return createErrorJsonObject("invalid-reply-id");
                }
index 293b4b0..d4251f1 100644 (file)
@@ -67,7 +67,7 @@ public class GetLikesAjaxPage extends JsonPage {
                        Set<Sone> sones = webInterface.getCore().getLikes(post);
                        return createSuccessJsonObject().put("likes", sones.size()).put("sones", getSones(sones));
                } else if ("reply".equals(type)) {
-                       PostReply reply = webInterface.getCore().getReply(id);
+                       PostReply reply = webInterface.getCore().getPostReply(id, false);
                        Set<Sone> sones = webInterface.getCore().getLikes(reply);
                        return createSuccessJsonObject().put("likes", sones.size()).put("sones", getSones(sones));
                }
index 38bf090..ca2996f 100644 (file)
@@ -62,7 +62,7 @@ public class GetReplyAjaxPage extends JsonPage {
        @Override
        protected JsonObject createJsonObject(FreenetRequest request) {
                String replyId = request.getHttpRequest().getParam("reply");
-               PostReply reply = webInterface.getCore().getReply(replyId);
+               PostReply reply = webInterface.getCore().getPostReply(replyId, false);
                if ((reply == null) || (reply.getSone() == null)) {
                        return createErrorJsonObject("invalid-reply-id");
                }
index f6bc05d..4d67c3c 100644 (file)
@@ -77,7 +77,7 @@ public class GetTimesAjaxPage extends JsonPage {
                if (allIds.length() > 0) {
                        String[] ids = allIds.split(",");
                        for (String id : ids) {
-                               PostReply reply = webInterface.getCore().getReply(id, false);
+                               PostReply reply = webInterface.getCore().getPostReply(id, false);
                                if (reply == null) {
                                        continue;
                                }
index 370bacd..7944e11 100644 (file)
@@ -63,7 +63,7 @@ public class MarkAsKnownAjaxPage extends JsonPage {
                                }
                                core.markPostKnown(post);
                        } else if (type.equals("reply")) {
-                               PostReply reply = core.getReply(id, false);
+                               PostReply reply = core.getPostReply(id, false);
                                if (reply == null) {
                                        continue;
                                }