From c6c2788fc324af036fa7b5cd40fbf3e32a2b29b5 Mon Sep 17 00:00:00 2001
From: Nick Burch <nick.burch@quanticate.com>
Date: Thu, 18 Aug 2011 14:38:58 +0000
Subject: [PATCH] ALF-9153 Convert the topic+post creation webscript to be Java
 backed and Lucene free

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@29884 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../forum/forum-posts.post.json.js            | 100 --------------
 .../web-scripts-application-context.xml       |   7 +
 .../scripts/discussion/ForumTopicPost.java    | 122 ++++++++++++++++++
 3 files changed, 129 insertions(+), 100 deletions(-)
 delete mode 100644 config/alfresco/templates/webscripts/org/alfresco/repository/discussions/forum/forum-posts.post.json.js
 create mode 100644 source/java/org/alfresco/repo/web/scripts/discussion/ForumTopicPost.java

diff --git a/config/alfresco/templates/webscripts/org/alfresco/repository/discussions/forum/forum-posts.post.json.js b/config/alfresco/templates/webscripts/org/alfresco/repository/discussions/forum/forum-posts.post.json.js
deleted file mode 100644
index 3965a7f67f..0000000000
--- a/config/alfresco/templates/webscripts/org/alfresco/repository/discussions/forum/forum-posts.post.json.js
+++ /dev/null
@@ -1,100 +0,0 @@
-<import resource="classpath:alfresco/templates/webscripts/org/alfresco/repository/requestutils.lib.js">
-<import resource="classpath:alfresco/templates/webscripts/org/alfresco/repository/nodenameutils.lib.js">
-<import resource="classpath:alfresco/templates/webscripts/org/alfresco/repository/discussions/topicpost.lib.js">
-
-function ensureTagScope(node)
-{
-   if (! node.isTagScope)
-   {
-      node.isTagScope = true;
-   }
-   
-   // also check the parent (the site!)
-   if (! node.parent.isTagScope)
-   {
-      node.parent.isTagScope = true;
-   }
-}
-
-/**
- * Adds a post to the passed forum node.
- */
-function createPost(forumNode)
-{
-   // fetch the data required to create a topic
-   var title = "";
-   if (json.has("title"))
-   {
-      title = json.get("title");
-   }
-   var content = "";
-   if (json.has("content"))
-   {
-      content = json.get("content");
-   }
-   var tags = [];
-   if (json.has("tags"))
-   {
-      // get the tags JSONArray and copy it into a real javascript array object
-      var tmp = json.get("tags");
-      for (var x=0; x < tmp.length(); x++)
-      {
-          tags.push(tmp.get(x));
-      }
-   }
-   
-   // create the topic node, and add the first child node representing the topic text
-   // NOTE: this is a change from the old web client, where the topic title was used as name
-   //       for the topic node. We will use generated names to make sure we won't have naming
-   //       clashes.
-   var name = getUniqueChildName(forumNode, "post");
-   var topicNode = forumNode.createNode(name, "fm:topic");
-   topicNode.properties.title = title;
-   topicNode.save();
-
-   // We use twice the same name for the topic and the post in it
-   var contentNode = topicNode.createNode(name, "fm:post");
-   contentNode.mimetype = "text/html";
-   contentNode.properties.title = title;
-   contentNode.content = content;
-   contentNode.save();
-
-   // add the cm:syndication aspect
-   var props = new Array();
-   props["cm:published"] = new Date();
-   contentNode.addAspect("cm:syndication", props);
-
-   // add the tags to the topic node for now
-   topicNode.tags = tags;
-
-   return topicNode;
-}
-
-function main()
-{
-   // get requested node
-   var node = getRequestNode();
-   if (status.getCode() != status.STATUS_OK)
-   {
-      return;
-   }
-
-   ensureTagScope(node);
-
-   var topicPost = createPost(node);
-   
-   model.postData = getTopicPostData(topicPost);
-   
-   // create an activity entry
-   if (json.has("site") && json.has("page"))
-   {
-      var data =
-      {
-         title: model.postData.post.properties.title,
-         page: json.get("page") + "?topicId=" + model.postData.topic.name
-      }
-      activities.postActivity("org.alfresco.discussions.post-created", json.get("site"), "discussions", jsonUtils.toJSONString(data));
-   }   
-}
-
-main();
diff --git a/config/alfresco/web-scripts-application-context.xml b/config/alfresco/web-scripts-application-context.xml
index e9aad1e26a..523d71c2e9 100644
--- a/config/alfresco/web-scripts-application-context.xml
+++ b/config/alfresco/web-scripts-application-context.xml
@@ -1600,6 +1600,13 @@
          class="org.alfresco.repo.web.scripts.discussion.ForumPostRepliesGet"
          parent="abstractDiscussionWebScript">
    </bean>
+
+   <!-- Creates a new discussions topic and primary post -->
+   <bean id="webscript.org.alfresco.repository.discussions.forum.forum-posts.post"
+         class="org.alfresco.repo.web.scripts.discussion.ForumTopicPost"
+         parent="abstractDiscussionWebScript">
+   </bean>
+   
    
    <!-- Lists the Discussions for a site -->
 <!--
diff --git a/source/java/org/alfresco/repo/web/scripts/discussion/ForumTopicPost.java b/source/java/org/alfresco/repo/web/scripts/discussion/ForumTopicPost.java
new file mode 100644
index 0000000000..70958bf80f
--- /dev/null
+++ b/source/java/org/alfresco/repo/web/scripts/discussion/ForumTopicPost.java
@@ -0,0 +1,122 @@
+/*
+ * Copyright (C) 2005-2011 Alfresco Software Limited.
+ *
+ * This file is part of Alfresco
+ *
+ * Alfresco is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Alfresco is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ */
+package org.alfresco.repo.web.scripts.discussion;
+
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Map;
+
+import org.alfresco.service.cmr.discussion.PostInfo;
+import org.alfresco.service.cmr.discussion.TopicInfo;
+import org.alfresco.service.cmr.repository.NodeRef;
+import org.alfresco.service.cmr.site.SiteInfo;
+import org.json.JSONArray;
+import org.json.JSONException;
+import org.json.JSONObject;
+import org.springframework.extensions.webscripts.Cache;
+import org.springframework.extensions.webscripts.Status;
+import org.springframework.extensions.webscripts.WebScriptException;
+import org.springframework.extensions.webscripts.WebScriptRequest;
+
+/**
+ * This class is the controller for the discussions page editing forum-posts.postt webscript.
+ * 
+ * @author Nick Burch
+ * @since 4.0
+ */
+public class ForumTopicPost extends AbstractDiscussionWebScript
+{
+   @Override
+   protected Map<String, Object> executeImpl(SiteInfo site, NodeRef nodeRef,
+         TopicInfo topic, PostInfo post, WebScriptRequest req, JSONObject json,
+         Status status, Cache cache) 
+   {
+      // They shouldn't be adding to an existing Post or Topic
+      if(topic != null || post != null)
+      {
+         String error = "Can't create a new Topic inside an existing Topic or Post";
+         throw new WebScriptException(Status.STATUS_BAD_REQUEST, error);
+      }
+      
+      
+      // Grab the details of the new Topic and Post
+      String title = "";
+      String contents = "";
+      List<String> tags = null;
+      try
+      {
+         if(json.has("title"))
+         {
+            title = json.getString("title");
+         }
+         if(json.has("content"))
+         {
+            contents = json.getString("content");
+         }
+         if(json.has("tags"))
+         {
+            tags = new ArrayList<String>();
+            JSONArray jsTags = json.getJSONArray("tags");
+            for(int i=0; i<jsTags.length(); i++)
+            {
+               tags.add( jsTags.getString(i) );
+            }
+         }
+      }
+      catch(JSONException e)
+      {
+         throw new WebScriptException("Invalid JSON: " + e.getMessage());
+      }
+      
+      
+      // Have the topic created
+      if(site != null)
+      {
+         topic = discussionService.createTopic(site.getShortName(), title);
+      }
+      else
+      {
+         topic = discussionService.createTopic(nodeRef, title);
+      }
+      if(tags != null && tags.size() > 0)
+      {
+         topic.getTags().clear();
+         topic.getTags().addAll(tags);
+         discussionService.updateTopic(topic);
+      }
+      
+      
+      // Have the primary post created
+      post = discussionService.createPost(topic, contents);
+      
+      
+      // Record the activity
+      addActivityEntry("post", "created", topic, post, site, req, json);
+      
+      
+      // Build the common model parts
+      Map<String, Object> model = buildCommonModel(site, topic, post, req);
+      
+      // Build the JSON for the whole topic
+      model.put(KEY_POSTDATA, renderTopic(topic, site));
+      
+      // All done
+      return model;
+   }
+}