Merged searchapi (5.2.1) to 5.2.N (5.2.1)

130051 gjames: SEARCH-159: Don't allow sort for cmis_alfresco language


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/BRANCHES/DEV/5.2.N/root@130298 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Gethin James
2016-09-06 15:38:52 +00:00
parent 04881cfa0c
commit 2ae8c3f90d
2 changed files with 3 additions and 2 deletions

View File

@@ -170,7 +170,7 @@ public class SearchMapper
{
if (sort != null && !sort.isEmpty())
{
if (CMIS.equals(sp.getLanguage()))
if (LANGUAGE_CMIS_ALFRESCO.equals(sp.getLanguage()))
{
throw new InvalidArgumentException(InvalidArgumentException.DEFAULT_MESSAGE_ID,
new Object[] { ": sort {} not allowed with cmis language" });

View File

@@ -46,6 +46,7 @@ import org.alfresco.rest.framework.resource.parameters.Paging;
import org.alfresco.service.cmr.repository.StoreRef;
import org.alfresco.service.cmr.search.LimitBy;
import org.alfresco.service.cmr.search.SearchParameters;
import org.alfresco.service.cmr.search.SearchService;
import org.junit.Test;
import java.util.Arrays;
@@ -211,7 +212,7 @@ public class SearchMapperTests
assertEquals("{}TYPE", searchParameters.getSortDefinitions().get(0).getField());
searchParameters = new SearchParameters();
searchParameters.setLanguage(SearchMapper.CMIS);
searchParameters.setLanguage(SearchService.LANGUAGE_CMIS_ALFRESCO);
try
{
searchMapper.fromSort(searchParameters, Arrays.asList(new SortDef("FIELD", null, false)));