Merged HEAD-BUG-FIX (5.1/Cloud) to HEAD (5.1/Cloud)

98899: Merged 5.0.N (5.0.2) to HEAD-BUG-FIX (5.1/Cloud)
      98890: Merged V4.2-BUG-FIX (4.2.5) to 5.0.N (5.0.2)
         98886: Reverse merged V4.1-BUG-FIX (4.1.10)
            << Original commits for this issue caused a performance problem found in 5.0.1 testing >>
            98334: Merged DEV (4.2.5) to V4.2-BUG-FIX (4.2.5)
               98326: MNT-13198: BM-0012: Run v420b1494_01: (CMIS) GetSites is Slow
                 - Changed the PersonSitesRelation to use a faster API (SiteServiceImpl.listSiteMemberships(String, int)).
            98452: MNT-13198: BM-0012: Run v420b1494_01: (CMIS) GetSites is Slow
               - Corrected the solution to fix the build failures.


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@98955 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Alan Davis
2015-03-06 22:36:57 +00:00
parent 29e6b306b1
commit 021c0d71d9

View File

@@ -415,44 +415,22 @@ public class SitesImpl implements Sites
personId = people.validatePerson(personId);
PagingRequest pagingRequest = Util.getPagingRequest(paging);
final Collator collator = Collator.getInstance();
final Set<SiteMembership> sortedSiteMembers = new TreeSet<SiteMembership>(new Comparator<SiteMembership>()
{
@Override
public int compare(SiteMembership o1, SiteMembership o2)
{
return collator.compare(o1.getSiteInfo().getTitle(), o2.getSiteInfo().getTitle());
}
});
List<SiteMembership> siteMembers = siteService.listSiteMemberships (personId, -1);
int totalSize = siteMembers.size();
sortedSiteMembers.addAll(siteMembers);
PageDetails pageDetails = PageDetails.getPageDetails(pagingRequest, totalSize);
List<MemberOfSite> ret = new ArrayList<MemberOfSite>(totalSize);
Iterator<SiteMembership> it = sortedSiteMembers.iterator();
for(int counter = 0; counter < pageDetails.getEnd() && it.hasNext(); counter++)
{
SiteMembership siteMember = it.next();
if(counter < pageDetails.getSkipCount())
{
continue;
}
if(counter > pageDetails.getEnd() - 1)
{
break;
}
final List<Pair<SiteService.SortFields, Boolean>> sort = new ArrayList<Pair<SiteService.SortFields, Boolean>>();
sort.add(new Pair<SiteService.SortFields, Boolean>(SiteService.SortFields.SiteTitle, Boolean.TRUE));
sort.add(new Pair<SiteService.SortFields, Boolean>(SiteService.SortFields.Role, Boolean.TRUE));
SiteInfo siteInfo = siteMember.getSiteInfo();
MemberOfSite memberOfSite = new MemberOfSite(siteInfo.getShortName(), siteInfo.getNodeRef(), siteMember.getRole());
ret.add(memberOfSite);
}
return CollectionWithPagingInfo.asPaged(paging, ret, pageDetails.hasMoreItems(), null);
PagingResults<SiteMembership> results = siteService.listSitesPaged(personId, sort, pagingRequest);
List<SiteMembership> siteMembers = results.getPage();
List<MemberOfSite> ret = new ArrayList<MemberOfSite>(siteMembers.size());
for(SiteMembership siteMember : siteMembers)
{
SiteInfo siteInfo = siteMember.getSiteInfo();
MemberOfSite memberOfSite = new MemberOfSite(siteInfo.getShortName(), siteInfo.getNodeRef(), siteMember.getRole());
ret.add(memberOfSite);
}
return CollectionWithPagingInfo.asPaged(paging, ret, results.hasMoreItems(), null);
}
public SiteContainer getSiteContainer(String siteId, String containerId)