Refactor JCR exceptions.

[lgpl/argeo-commons.git] / org.argeo.cms / src / org / argeo / cms / internal / kernel / NodeRepositoryFactory.java

diff --git a/org.argeo.cms/src/org/argeo/cms/internal/kernel/NodeRepositoryFactory.java b/org.argeo.cms/src/org/argeo/cms/internal/kernel/NodeRepositoryFactory.java
index f83eb9476f6005c725d95817782d4b8ac9911681..a6c722015f7fd6d9c3ec077f5ff522fc34659aad 100644 (file)
--- a/org.argeo.cms/src/org/argeo/cms/internal/kernel/NodeRepositoryFactory.java
+++ b/org.argeo.cms/src/org/argeo/cms/internal/kernel/NodeRepositoryFactory.java
@@ -1,18 +1,3 @@
-/*
- * Copyright (C) 2007-2012 Argeo GmbH
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *         http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
 package org.argeo.cms.internal.kernel;
 
 import java.net.URI;
@@ -28,14 +13,12 @@ import javax.jcr.RepositoryFactory;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.jackrabbit.jcr2dav.Jcr2davRepositoryFactory;
+import org.argeo.api.NodeConstants;
 import org.argeo.cms.internal.jcr.RepoConf;
-import org.argeo.jcr.ArgeoJcrException;
-import org.argeo.node.NodeConstants;
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.FrameworkUtil;
 import org.osgi.framework.InvalidSyntaxException;
 import org.osgi.framework.ServiceReference;
-import org.springframework.core.io.Resource;
 
 /**
  * OSGi-aware Jackrabbit repository factory which can retrieve/publish
@@ -53,13 +36,13 @@ class NodeRepositoryFactory implements RepositoryFactory {
                        Collection<ServiceReference<Repository>> srs = bundleContext.getServiceReferences(Repository.class,
                                        "(" + NodeConstants.CN + "=" + alias + ")");
                        if (srs.size() == 0)
-                               throw new ArgeoJcrException("No repository with alias " + alias + " found in OSGi registry");
+                               throw new IllegalArgumentException("No repository with alias " + alias + " found in OSGi registry");
                        else if (srs.size() > 1)
-                               throw new ArgeoJcrException(
+                               throw new IllegalArgumentException(
                                                srs.size() + " repositories with alias " + alias + " found in OSGi registry");
                        return bundleContext.getService(srs.iterator().next());
                } catch (InvalidSyntaxException e) {
-                       throw new ArgeoJcrException("Cannot find repository with alias " + alias, e);
+                       throw new IllegalArgumentException("Cannot find repository with alias " + alias, e);
                }
        }
 
@@ -101,7 +84,7 @@ class NodeRepositoryFactory implements RepositoryFactory {
                                // JCR repository factory");
                                repository = getRepositoryByAlias(getAliasFromURI(uri));
                        } else
-                               throw new ArgeoJcrException("Unrecognized URI format " + uri);
+                               throw new IllegalArgumentException("Unrecognized URI format " + uri);
 
                }
 
@@ -114,10 +97,10 @@ class NodeRepositoryFactory implements RepositoryFactory {
                        // with properties " + properties);
                        repository = getRepositoryByAlias(alias);
                } else
-                       throw new ArgeoJcrException("Not enough information in " + parameters);
+                       throw new IllegalArgumentException("Not enough information in " + parameters);
 
                if (repository == null)
-                       throw new ArgeoJcrException("Repository not found " + parameters);
+                       throw new IllegalArgumentException("Repository not found " + parameters);
 
                return repository;
        }
@@ -129,7 +112,7 @@ class NodeRepositoryFactory implements RepositoryFactory {
                        params.put(KernelConstants.JACKRABBIT_REMOTE_DEFAULT_WORKSPACE, defaultWorkspace);
                Repository repository = new Jcr2davRepositoryFactory().getRepository(params);
                if (repository == null)
-                       throw new ArgeoJcrException("Remote Davex repository " + uri + " not found");
+                       throw new IllegalArgumentException("Remote Davex repository " + uri + " not found");
                log.info("Initialized remote Jackrabbit repository from uri " + uri);
                return repository;
        }
@@ -189,7 +172,7 @@ class NodeRepositoryFactory implements RepositoryFactory {
                                alias = alias.substring(0, alias.length() - 1);
                        return alias;
                } catch (URISyntaxException e) {
-                       throw new ArgeoJcrException("Cannot interpret URI " + uri, e);
+                       throw new IllegalArgumentException("Cannot interpret URI " + uri, e);
                }
        }
 
@@ -200,9 +183,4 @@ class NodeRepositoryFactory implements RepositoryFactory {
        protected void postInitialization(Repository repository, Map parameters) {
 
        }
-
-       public void setFileRepositoryConfiguration(Resource fileRepositoryConfiguration) {
-               // this.fileRepositoryConfiguration = fileRepositoryConfiguration;
-       }
-
 }