From: Aron Roberts <aron@socrates.berkeley.edu>
Date: Thu, 15 Apr 2010 20:09:07 +0000 (+0000)
Subject: CSPACE-1494,CSPACE-873: Temporarily stub-out return of acquisitionSource in acquisiti... 
X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=7edbb7666647c9186009fb06233df674f1559114;p=tmp%2Fjakarta-migration.git

CSPACE-1494,CSPACE-873: Temporarily stub-out return of acquisitionSource in acquisitions list items, pending a more durable solution to returning one (or more) instances of repeatable field values in minimal profile of list items.  Ultimately may return the value of the 'primary' instance of a repeatable field, per Patrick.
---

diff --git a/services/acquisition/jaxb/src/main/java/org/collectionspace/services/AcquisitionListItemJAXBSchema.java b/services/acquisition/jaxb/src/main/java/org/collectionspace/services/AcquisitionListItemJAXBSchema.java
index a0d3999b2..0f816b07c 100644
--- a/services/acquisition/jaxb/src/main/java/org/collectionspace/services/AcquisitionListItemJAXBSchema.java
+++ b/services/acquisition/jaxb/src/main/java/org/collectionspace/services/AcquisitionListItemJAXBSchema.java
@@ -2,7 +2,7 @@ package org.collectionspace.services;
 
 public interface AcquisitionListItemJAXBSchema {
 	final static String ACQUISITION_REFERENCE_NUMBER = "acquisitionReferenceNumber";
-	final static String ACQUISITION_SOURCE="acquisitionSource";
+	final static String ACQUISITION_SOURCES = "acquisitionSources";
 	final static String CSID = "csid";
 	final static String URI = "url";
 }
diff --git a/services/acquisition/jaxb/src/main/resources/acquisitions_common.xsd b/services/acquisition/jaxb/src/main/resources/acquisitions_common.xsd
index fa707b376..04fd09a6e 100644
--- a/services/acquisition/jaxb/src/main/resources/acquisitions_common.xsd
+++ b/services/acquisition/jaxb/src/main/resources/acquisitions_common.xsd
@@ -81,7 +81,7 @@
                         <xs:sequence>
                             <xs:element name="acquisitionReferenceNumber" type="xs:string"
                                 minOccurs="1" />
-                            <xs:element name="acquisitionSource" type="xs:string"
+                            <xs:element name="acquisitionSources" type="acquisitionSourceList"
                                 minOccurs="1" />
                             <!-- uri to retrive acquisition details -->
                             <xs:element name="uri" type="xs:anyURI"
diff --git a/services/acquisition/service/src/main/java/org/collectionspace/services/acquisition/nuxeo/AcquisitionDocumentModelHandler.java b/services/acquisition/service/src/main/java/org/collectionspace/services/acquisition/nuxeo/AcquisitionDocumentModelHandler.java
index c79eb3890..3b1e3d4f7 100644
--- a/services/acquisition/service/src/main/java/org/collectionspace/services/acquisition/nuxeo/AcquisitionDocumentModelHandler.java
+++ b/services/acquisition/service/src/main/java/org/collectionspace/services/acquisition/nuxeo/AcquisitionDocumentModelHandler.java
@@ -32,6 +32,7 @@ import org.collectionspace.services.common.document.DocumentWrapper;
 import org.collectionspace.services.acquisition.AcquisitionsCommon;
 import org.collectionspace.services.acquisition.AcquisitionsCommonList;
 import org.collectionspace.services.acquisition.AcquisitionsCommonList.AcquisitionListItem;
+import org.collectionspace.services.acquisition.AcquisitionSourceList;
 
 import org.collectionspace.services.common.document.DocumentHandler.Action;
 import org.collectionspace.services.nuxeo.client.java.RemoteDocumentModelHandlerImpl;
@@ -120,8 +121,18 @@ public class AcquisitionDocumentModelHandler
             AcquisitionListItem listItem = new AcquisitionListItem();
             listItem.setAcquisitionReferenceNumber((String) docModel.getProperty(getServiceContext().getCommonPartLabel(),
                     AcquisitionListItemJAXBSchema.ACQUISITION_REFERENCE_NUMBER));
-            listItem.setAcquisitionSource((String) docModel.getProperty(getServiceContext().getCommonPartLabel(),
-                    AcquisitionListItemJAXBSchema.ACQUISITION_SOURCE));
+            // logger.warn("Before getting acquisitionSources ...");
+            // Investigate what is returned here by Nuxeo's DocumentModel.getProperty ... List<String> or?
+            // AcquisitionSourceList acquisitionSources =
+            //     (AcquisitionSourceList) docModel.getProperty(qetServiceContext().getCommonPartLabel(),
+            //         AcquisitionListItemJAXBSchema.ACQUISITION_SOURCES);
+            // logger.warn("acquisitionSources size = " + acquisitionSources.getAcquisitionSource().size());
+            if (logger.isDebugEnabled()) {
+              logger.debug("Before setting acquisition sources ...");
+            }
+            AcquisitionSourceList acquisitionSources = new AcquisitionSourceList();
+            acquisitionSources.getAcquisitionSource().add("Currently stubbed - see CSPACE-1494.");
+            listItem.setAcquisitionSources(acquisitionSources);
             //need fully qualified context for URI
             String id = NuxeoUtils.extractId(docModel.getPathAsString());
             listItem.setUri(getServiceContextPath() + id);