Don’t render posts and replies on getStatus().
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Thu, 25 Nov 2010 11:17:21 +0000 (12:17 +0100)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Thu, 25 Nov 2010 11:17:21 +0000 (12:17 +0100)
src/main/java/net/pterodactylus/sone/web/WebInterface.java
src/main/java/net/pterodactylus/sone/web/ajax/GetStatusAjaxPage.java

index 5010bad..a7b6728 100644 (file)
@@ -352,7 +352,7 @@ public class WebInterface implements CoreListener {
                pageToadlets.add(pageToadletFactory.createPageToadlet(new StaticPage("javascript/", "/static/javascript/", "text/javascript")));
                pageToadlets.add(pageToadletFactory.createPageToadlet(new StaticPage("images/", "/static/images/", "image/png")));
                pageToadlets.add(pageToadletFactory.createPageToadlet(new GetTranslationPage(this)));
-               pageToadlets.add(pageToadletFactory.createPageToadlet(new GetStatusAjaxPage(this, postTemplate, replyTemplate)));
+               pageToadlets.add(pageToadletFactory.createPageToadlet(new GetStatusAjaxPage(this)));
                pageToadlets.add(pageToadletFactory.createPageToadlet(new DismissNotificationAjaxPage(this)));
                pageToadlets.add(pageToadletFactory.createPageToadlet(new CreateReplyAjaxPage(this)));
                pageToadlets.add(pageToadletFactory.createPageToadlet(new GetReplyAjaxPage(this, replyTemplate)));
index 8b7f513..8b1612b 100644 (file)
@@ -17,7 +17,6 @@
 
 package net.pterodactylus.sone.web.ajax;
 
-import java.io.StringWriter;
 import java.text.DateFormat;
 import java.text.SimpleDateFormat;
 import java.util.ArrayList;
@@ -31,12 +30,9 @@ import net.pterodactylus.sone.data.Reply;
 import net.pterodactylus.sone.data.Sone;
 import net.pterodactylus.sone.template.SoneAccessor;
 import net.pterodactylus.sone.web.WebInterface;
-import net.pterodactylus.util.io.Closer;
 import net.pterodactylus.util.json.JsonArray;
 import net.pterodactylus.util.json.JsonObject;
 import net.pterodactylus.util.notify.Notification;
-import net.pterodactylus.util.template.Template;
-import net.pterodactylus.util.template.TemplateException;
 
 /**
  * The “get status” AJAX handler returns all information that is necessary to
@@ -49,26 +45,14 @@ public class GetStatusAjaxPage extends JsonPage {
        /** Date formatter. */
        private static final DateFormat dateFormat = new SimpleDateFormat("MMM d, yyyy, HH:mm:ss");
 
-       /** The template to render posts. */
-       private final Template postTemplate;
-
-       /** The template to render replies. */
-       private final Template replyTemplate;
-
        /**
         * Creates a new “get status” AJAX handler.
         *
         * @param webInterface
         *            The Sone web interface
-        * @param postTemplate
-        *            The template to render for posts
-        * @param replyTemplate
-        *            The template to render for replies
         */
-       public GetStatusAjaxPage(WebInterface webInterface, Template postTemplate, Template replyTemplate) {
+       public GetStatusAjaxPage(WebInterface webInterface) {
                super("ajax/getStatus.ajax", webInterface);
-               this.postTemplate = postTemplate;
-               this.replyTemplate = replyTemplate;
        }
 
        /**
@@ -97,18 +81,16 @@ public class GetStatusAjaxPage extends JsonPage {
                        jsonRemovedNotifications.add(createJsonNotification(notification));
                }
                /* load new posts. */
-               postTemplate.set("currentSone", getCurrentSone(request.getToadletContext()));
                Set<Post> newPosts = webInterface.getNewPosts();
                JsonArray jsonPosts = new JsonArray();
                for (Post post : newPosts) {
-                       jsonPosts.add(createJsonPost(post));
+                       jsonPosts.add(post.getId());
                }
                /* load new replies. */
-               replyTemplate.set("currentSone", getCurrentSone(request.getToadletContext()));
                Set<Reply> newReplies = webInterface.getNewReplies();
                JsonArray jsonReplies = new JsonArray();
                for (Reply reply : newReplies) {
-                       jsonReplies.add(createJsonReply(reply));
+                       jsonReplies.add(reply.getId());
                }
                return createSuccessJsonObject().put("sones", jsonSones).put("notifications", jsonNotifications).put("removedNotifications", jsonRemovedNotifications).put("newPosts", jsonPosts).put("newReplies", jsonReplies);
        }
@@ -148,56 +130,6 @@ public class GetStatusAjaxPage extends JsonPage {
        }
 
        /**
-        * Creates a JSON object from the given post. The JSON object will only
-        * contain the ID of the post, its time, and its rendered HTML code.
-        *
-        * @param post
-        *            The post to create a JSON object from
-        * @return The JSON representation of the post
-        */
-       private JsonObject createJsonPost(Post post) {
-               JsonObject jsonPost = new JsonObject();
-               jsonPost.put("id", post.getId());
-               jsonPost.put("time", post.getTime());
-               StringWriter stringWriter = new StringWriter();
-               postTemplate.set("post", post);
-               try {
-                       postTemplate.render(stringWriter);
-               } catch (TemplateException te1) {
-                       /* TODO - shouldn’t happen. */
-               } finally {
-                       Closer.close(stringWriter);
-               }
-               return jsonPost.put("html", stringWriter.toString());
-       }
-
-       /**
-        * Creates a JSON object from the given reply. The JSON object will only
-        * contain the ID of the reply, the ID of its post, its time, and its
-        * rendered HTML code.
-        *
-        * @param reply
-        *            The reply to create a JSON object from
-        * @return The JSON representation of the reply
-        */
-       private JsonObject createJsonReply(Reply reply) {
-               JsonObject jsonPost = new JsonObject();
-               jsonPost.put("postId", reply.getPost().getId());
-               jsonPost.put("id", reply.getId());
-               jsonPost.put("time", reply.getTime());
-               StringWriter stringWriter = new StringWriter();
-               replyTemplate.set("reply", reply);
-               try {
-                       replyTemplate.render(stringWriter);
-               } catch (TemplateException te1) {
-                       /* TODO - shouldn’t happen. */
-               } finally {
-                       Closer.close(stringWriter);
-               }
-               return jsonPost.put("html", stringWriter.toString());
-       }
-
-       /**
         * Creates a JSON object from the given notification.
         *
         * @param notification