From: Sanjay Dalal <sanjay.dalal@berkeley.edu>
Date: Tue, 23 Jun 2009 18:19:12 +0000 (+0000)
Subject: CSPACE-242 checks if nuxeo-client exists using jboss.server.home.dir system property... 
X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=53aa495370f0b1e192598f82c561ca2dea491b06;p=tmp%2Fjakarta-migration.git

CSPACE-242 checks if nuxeo-client exists using jboss.server.home.dir system property.If the dir does not exist, it logs an error message and throws IllegalStateException with the same message as well.
---

diff --git a/services/common/src/main/java/org/collectionspace/services/nuxeo/NuxeoConnector.java b/services/common/src/main/java/org/collectionspace/services/nuxeo/NuxeoConnector.java
index 1767f228c..87a55d1c0 100644
--- a/services/common/src/main/java/org/collectionspace/services/nuxeo/NuxeoConnector.java
+++ b/services/common/src/main/java/org/collectionspace/services/nuxeo/NuxeoConnector.java
@@ -83,9 +83,20 @@ public class NuxeoConnector {
 
     private void loadBundles() throws Exception {
         String bundles = "nuxeo-client/lib/nuxeo-runtime-*:nuxeo-client/lib/nuxeo-*";
+        String serverRootDir = System.getProperty("jboss.server.home.dir");
+        if(serverRootDir == null){
+            serverRootDir = "."; //assume server is started from server root, e.g. server/cspace
+        }
+        File clientLibDir = new File(serverRootDir + File.separator + "nuxeo-client");
+        if(!clientLibDir.exists()){
+            String msg = "Library bundles requried to deploy Nuxeo client not found: " +
+                    " directory named nuxeo-client with bundles does not exist in " + serverRootDir;
+            logger.error(msg);
+            throw new IllegalStateException(msg);
+        }
         Collection<File> files = null;
         if(bundles != null){
-            files = NuxeoApp.getBundleFiles(new File("."), bundles, ":");
+            files = NuxeoApp.getBundleFiles(new File(serverRootDir), bundles, ":");
         }
         if(logger.isDebugEnabled()){
             logger.debug("loadBundles(): deploying bundles: " + files);
@@ -154,10 +165,9 @@ public class NuxeoConnector {
     public void releaseRepositorySession(RepositoryInstance repoSession) throws Exception {
         if(repoSession != null){
             getClient().releaseRepository(repoSession);
+
             if(logger.isDebugEnabled()){
-                if(logger.isDebugEnabled()){
-                    logger.debug("releaseRepositorySession() released repository session");
-                }
+                logger.debug("releaseRepositorySession() released repository session");
             }
         }
     }
@@ -179,7 +189,7 @@ public class NuxeoConnector {
                 DocumentModel domain = riter.next();
                 DocumentModelList domainChildrenList = repoSession.getChildren(domain.getRef());
                 Iterator<DocumentModel> diter = domainChildrenList.iterator();
-                if(diter.hasNext()){
+                while(diter.hasNext()){
                     DocumentModel childNode = diter.next();
                     if("Workspaces".equalsIgnoreCase(childNode.getName())){
                         DocumentModelList workspaceList = repoSession.getChildren(childNode.getRef());
@@ -195,6 +205,11 @@ public class NuxeoConnector {
                     }
                 }
             }
+        }catch(Exception e){
+            if(logger.isDebugEnabled()){
+                logger.debug("retrieveWorkspaceIds() caught exception ", e);
+            }
+            throw e;
         }finally{
             if(repoSession != null){
                 releaseRepositorySession(repoSession);