mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-08-14 17:58:59 +00:00
ALF-9153 Convert the user discussions listing webscript to be Java backed and lucene free
git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@29950 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
@@ -1,44 +0,0 @@
|
||||
<import resource="classpath:alfresco/templates/webscripts/org/alfresco/repository/requestutils.lib.js">
|
||||
<import resource="classpath:alfresco/templates/webscripts/org/alfresco/repository/generic-paged-results.lib.js">
|
||||
<import resource="classpath:alfresco/templates/webscripts/org/alfresco/repository/discussions/topicpost.lib.js">
|
||||
|
||||
/**
|
||||
* Fetches all posts added to the forum in the last numdays days
|
||||
*/
|
||||
function getTopicPostList(node, index, count)
|
||||
{
|
||||
// query information
|
||||
var luceneQuery = "+TYPE:\"{http://www.alfresco.org/model/forum/1.0}topic\" " +
|
||||
"+PATH:\"" + node.qnamePath + "/*\" " +
|
||||
"+@cm\\:creator:\"" + person.properties.userName + "\"";
|
||||
var sortAttribute = "@{http://www.alfresco.org/model/content/1.0}published";
|
||||
|
||||
// get the data
|
||||
return getPagedResultsDataByLuceneQuery(node, luceneQuery, sortAttribute, false, index, count, getTopicPostData);
|
||||
}
|
||||
|
||||
function main()
|
||||
{
|
||||
// get requested node
|
||||
var node = getRequestNode();
|
||||
if (status.getCode() != status.STATUS_OK)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
// process additional parameters
|
||||
var index = args["startIndex"] != undefined ? parseInt(args["startIndex"]) : 0;
|
||||
var count = args["pageSize"] != undefined ? parseInt(args["pageSize"]) : 10;
|
||||
|
||||
// fetch the data and assign it to the model
|
||||
model.data = getTopicPostList(node, index, count);
|
||||
|
||||
// fetch the contentLength param
|
||||
var contentLength = args["contentLength"] != undefined ? parseInt(args["contentLength"]) : -1;
|
||||
model.contentLength = isNaN(contentLength) ? -1 : contentLength;
|
||||
|
||||
// also set the forum node
|
||||
model.forum = node;
|
||||
}
|
||||
|
||||
main();
|
@@ -1614,6 +1614,12 @@
|
||||
parent="abstractDiscussionWebScript">
|
||||
</bean>
|
||||
|
||||
<!-- Lists the discussion topics for the user of a site -->
|
||||
<bean id="webscript.org.alfresco.repository.discussions.forum.forum-posts-mine.get"
|
||||
class="org.alfresco.repo.web.scripts.discussion.ForumTopicsMineGet"
|
||||
parent="abstractDiscussionWebScript">
|
||||
</bean>
|
||||
|
||||
|
||||
<!-- -->
|
||||
<!-- Slingshot Calendar REST API -->
|
||||
|
@@ -190,11 +190,9 @@ public class DiscussionRestApiTest extends BaseWebScriptTest
|
||||
// add the user as a member with the given role
|
||||
this.siteService.setMembership(SITE_SHORT_NAME_DISCUSSION, userName, role);
|
||||
|
||||
// Give them access to the test node
|
||||
// TODO This shouldn't be needed when the webscripts use the service properly
|
||||
// (We only need it because they go about adding tag scopes and aspects)
|
||||
// Give the test user access to the test node
|
||||
// They need to be able to read it, and create children of it
|
||||
permissionService.setPermission(FORUM_NODE, userName, PermissionService.READ, true);
|
||||
permissionService.setPermission(FORUM_NODE, userName, PermissionService.WRITE, true);
|
||||
permissionService.setPermission(FORUM_NODE, userName, PermissionService.CREATE_CHILDREN, true);
|
||||
}
|
||||
|
||||
@@ -1099,11 +1097,8 @@ public class DiscussionRestApiTest extends BaseWebScriptTest
|
||||
assertEquals(2, result.getInt("total"));
|
||||
assertEquals(2, result.getInt("itemCount"));
|
||||
assertEquals(2, result.getJSONArray("items").length());
|
||||
// TODO This appears to be incorrect...
|
||||
assertEquals("NodeTitle3", result.getJSONArray("items").getJSONObject(0).getString("title"));
|
||||
assertEquals("NodeTitle2", result.getJSONArray("items").getJSONObject(1).getString("title"));
|
||||
// assertEquals("NodeTitle2", result.getJSONArray("items").getJSONObject(0).getString("title"));
|
||||
// assertEquals("NodeTitle3", result.getJSONArray("items").getJSONObject(1).getString("title"));
|
||||
assertEquals("NodeTitle2", result.getJSONArray("items").getJSONObject(0).getString("title"));
|
||||
assertEquals("NodeTitle3", result.getJSONArray("items").getJSONObject(1).getString("title"));
|
||||
assertEquals(0, result.getJSONArray("items").getJSONObject(0).getInt("replyCount"));
|
||||
assertEquals(0, result.getJSONArray("items").getJSONObject(1).getInt("replyCount"));
|
||||
|
||||
|
@@ -0,0 +1,90 @@
|
||||
/*
|
||||
* 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.Map;
|
||||
|
||||
import org.alfresco.query.PagingRequest;
|
||||
import org.alfresco.query.PagingResults;
|
||||
import org.alfresco.repo.security.authentication.AuthenticationUtil;
|
||||
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.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 topics fetching forum-posts-mine.get webscript.
|
||||
*
|
||||
* @author Nick Burch
|
||||
* @since 4.0
|
||||
*/
|
||||
public class ForumTopicsMineGet 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 trying to list of an existing Post or Topic
|
||||
if(topic != null || post != null)
|
||||
{
|
||||
String error = "Can't list Topics inside an existing Topic or Post";
|
||||
throw new WebScriptException(Status.STATUS_BAD_REQUEST, error);
|
||||
}
|
||||
|
||||
// Grab the current user to list for
|
||||
String username = AuthenticationUtil.getFullyAuthenticatedUser();
|
||||
|
||||
// Get the topics for the user
|
||||
PagingResults<TopicInfo> topics = null;
|
||||
PagingRequest paging = buildPagingRequest(req);
|
||||
if(site != null)
|
||||
{
|
||||
topics = discussionService.listTopics(site.getShortName(), username, paging);
|
||||
}
|
||||
else
|
||||
{
|
||||
topics = discussionService.listTopics(nodeRef, username, paging);
|
||||
}
|
||||
|
||||
|
||||
// If they did a site based search, and the component hasn't
|
||||
// been created yet, use the site for the permissions checking
|
||||
if(site != null && nodeRef == null)
|
||||
{
|
||||
nodeRef = site.getNodeRef();
|
||||
}
|
||||
|
||||
|
||||
// Build the common model parts
|
||||
Map<String, Object> model = buildCommonModel(site, topic, post, req);
|
||||
model.put("forum", nodeRef);
|
||||
|
||||
// Have the topics rendered
|
||||
model.put("data", renderTopics(topics, paging, site));
|
||||
|
||||
// All done
|
||||
return model;
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user