From 0e74d28a2d77592349f55999529848a78f3860d8 Mon Sep 17 00:00:00 2001
From: Patrick Schmitz <pschmitz@berkeley.edu>
Date: Fri, 22 Jun 2012 09:49:49 -0700
Subject: [PATCH] CSPACE-5300 - Fix problem with CMIS calls. Document type must
 have schemas declared.

---
 .../src/main/resources/OSGI-INF/core-types-contrib.xml    | 3 +++
 .../nuxeo/client/java/RepositoryJavaClientImpl.java       | 8 +++-----
 .../nuxeo/ServiceGroupDocumentModelHandler.java           | 2 +-
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/3rdparty/nuxeo/nuxeo-platform-collectionspace/src/main/resources/OSGI-INF/core-types-contrib.xml b/3rdparty/nuxeo/nuxeo-platform-collectionspace/src/main/resources/OSGI-INF/core-types-contrib.xml
index a6f879e49..8be0f1619 100644
--- a/3rdparty/nuxeo/nuxeo-platform-collectionspace/src/main/resources/OSGI-INF/core-types-contrib.xml
+++ b/3rdparty/nuxeo/nuxeo-platform-collectionspace/src/main/resources/OSGI-INF/core-types-contrib.xml
@@ -6,6 +6,9 @@
   </extension>
   <extension target="org.nuxeo.ecm.core.schema.TypeService" point="doctype">
     <doctype name="CollectionSpaceDocument" extends="Document">
+      <schema name="common"/>
+      <schema name="dublincore"/>
+      <schema name="collectionspace_core"/>
     </doctype>
   </extension>  
   <extension target="org.nuxeo.ecm.core.schema.TypeService" point="doctype">
diff --git a/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java b/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java
index bf058c070..99d590ee7 100644
--- a/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java
+++ b/services/common/src/main/java/org/collectionspace/services/nuxeo/client/java/RepositoryJavaClientImpl.java
@@ -532,6 +532,9 @@ public class RepositoryJavaClientImpl implements RepositoryClient<PoxPayloadIn,
 
         DocumentFilter filter = handler.getDocumentFilter();
         String oldOrderBy = filter.getOrderByClause();
+        if (isClauseEmpty(oldOrderBy) == true){
+            filter.setOrderByClause(DocumentFilter.ORDER_BY_LAST_UPDATED);
+        }
         QueryContext queryContext = new QueryContext(ctx, handler);
 
         try {
@@ -541,15 +544,10 @@ public class RepositoryJavaClientImpl implements RepositoryClient<PoxPayloadIn,
             }
             DocumentModelList docList = null;
         	if (handler.isCMISQuery() == true) {
-        		// We have to omit the ORDER BY until we can get the "Document" table bug addressed.
-        		// CF Nuxeo JIRA NXP-9562
         		String inList = buildInListForDocTypes(docTypes);
         		ctx.getQueryParams().add(IQueryManager.SEARCH_RELATED_MATCH_OBJ_DOCTYPES, inList);
         		docList = getFilteredCMIS(repoSession, ctx, handler, queryContext);
             } else {
-                if (isClauseEmpty(oldOrderBy) == true){
-                    filter.setOrderByClause(DocumentFilter.ORDER_BY_LAST_UPDATED);
-                }
                 String query = NuxeoUtils.buildNXQLQuery(docTypes, queryContext);
                 if (logger.isDebugEnabled()) {
                     logger.debug("findDocs() NXQL: "+query);
diff --git a/services/servicegroup/service/src/main/java/org/collectionspace/services/servicegroup/nuxeo/ServiceGroupDocumentModelHandler.java b/services/servicegroup/service/src/main/java/org/collectionspace/services/servicegroup/nuxeo/ServiceGroupDocumentModelHandler.java
index 3000a1873..b32601512 100644
--- a/services/servicegroup/service/src/main/java/org/collectionspace/services/servicegroup/nuxeo/ServiceGroupDocumentModelHandler.java
+++ b/services/servicegroup/service/src/main/java/org/collectionspace/services/servicegroup/nuxeo/ServiceGroupDocumentModelHandler.java
@@ -124,7 +124,7 @@ public class ServiceGroupDocumentModelHandler
             	}
             	
             	// This should be "Document" but CMIS is gagging on that right now.
-            	ctx.getQueryParams().add(IQueryManager.SELECT_DOC_TYPE_FIELD, "cmis:document");
+            	ctx.getQueryParams().add(IQueryManager.SELECT_DOC_TYPE_FIELD, "CollectionSpaceDocument");
     	        
     	        // Now we have to issue the search
             	// findDocs qill build a QueryContext, which wants to see a docType for our context
-- 
2.47.3