Start factoring OSGi accesses.

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

diff --git a/org.argeo.cms/src/org/argeo/cms/internal/kernel/KernelUtils.java b/org.argeo.cms/src/org/argeo/cms/internal/kernel/KernelUtils.java
index b3fb33ac3169911e02eaafe4b6df4fbd1197dfb5..a5362c64e7484686ff64a0c2631d84acdc06b1cd 100644 (file)
--- a/org.argeo.cms/src/org/argeo/cms/internal/kernel/KernelUtils.java
+++ b/org.argeo.cms/src/org/argeo/cms/internal/kernel/KernelUtils.java
@@ -24,11 +24,12 @@ import javax.security.auth.login.LoginContext;
 import javax.security.auth.login.LoginException;
 
 import org.apache.commons.logging.Log;
+import org.argeo.api.DataModelNamespace;
+import org.argeo.api.NodeConstants;
 import org.argeo.cms.CmsException;
-import org.argeo.node.NodeConstants;
 import org.osgi.framework.Bundle;
 import org.osgi.framework.BundleContext;
-import org.osgi.framework.FrameworkUtil;
+import org.osgi.util.tracker.ServiceTracker;
 
 /** Package utilities */
 class KernelUtils implements KernelConstants {
@@ -42,7 +43,7 @@ class KernelUtils implements KernelConstants {
                                        .getInstance("JavaLoginConfig", uriParameter);
                        javax.security.auth.login.Configuration.setConfiguration(jaasConfiguration);
                } catch (Exception e) {
-                       throw new CmsException("Cannot set configuration " + jaasConfigurationUrl, e);
+                       throw new IllegalArgumentException("Cannot set configuration " + jaasConfigurationUrl, e);
                }
        }
 
@@ -59,7 +60,7 @@ class KernelUtils implements KernelConstants {
                try {
                        props.load(cl.getResourceAsStream(resource));
                } catch (IOException e) {
-                       throw new CmsException("Cannot load " + resource + " from classpath", e);
+                       throw new IllegalArgumentException("Cannot load " + resource + " from classpath", e);
                }
                return asDictionary(props);
        }
@@ -71,7 +72,7 @@ class KernelUtils implements KernelConstants {
                try {
                        return new File(executionDir, relativePath).getCanonicalFile();
                } catch (IOException e) {
-                       throw new CmsException("Cannot get canonical file", e);
+                       throw new IllegalArgumentException("Cannot get canonical file", e);
                }
        }
 
@@ -86,32 +87,28 @@ class KernelUtils implements KernelConstants {
 
        static URI getOsgiInstanceUri(String relativePath) {
                String osgiInstanceBaseUri = getFrameworkProp(OSGI_INSTANCE_AREA);
-               return safeUri(osgiInstanceBaseUri + (relativePath != null ? relativePath : ""));
+               if (osgiInstanceBaseUri != null)
+                       return safeUri(osgiInstanceBaseUri + (relativePath != null ? relativePath : ""));
+               else
+                       return Paths.get(System.getProperty("user.dir")).toUri();
        }
 
-       // static String getOsgiInstancePath(String relativePath) {
-       // try {
-       // if (relativePath == null)
-       // return getOsgiInstanceDir().getCanonicalPath();
-       // else
-       // return new File(getOsgiInstanceDir(), relativePath).getCanonicalPath();
-       // } catch (IOException e) {
-       // throw new CmsException("Cannot get instance path for " + relativePath,
-       // e);
-       // }
-       // }
-
        static File getOsgiConfigurationFile(String relativePath) {
                try {
                        return new File(new URI(getBundleContext().getProperty(OSGI_CONFIGURATION_AREA) + relativePath))
                                        .getCanonicalFile();
                } catch (Exception e) {
-                       throw new CmsException("Cannot get configuration file for " + relativePath, e);
+                       throw new IllegalArgumentException("Cannot get configuration file for " + relativePath, e);
                }
        }
 
        static String getFrameworkProp(String key, String def) {
-               String value = getBundleContext().getProperty(key);
+               BundleContext bundleContext = Activator.getBundleContext();
+               String value;
+               if (bundleContext != null)
+                       value = bundleContext.getProperty(key);
+               else
+                       value = System.getProperty(key);
                if (value == null)
                        return def;
                return value;
@@ -170,7 +167,7 @@ class KernelUtils implements KernelConstants {
                        loginContext = new LoginContext(NodeConstants.LOGIN_CONTEXT_DATA_ADMIN);
                        loginContext.login();
                } catch (LoginException e1) {
-                       throw new CmsException("Could not login as data admin", e1);
+                       throw new IllegalStateException("Could not login as data admin", e1);
                } finally {
                        Thread.currentThread().setContextClassLoader(currentCl);
                }
@@ -181,38 +178,63 @@ class KernelUtils implements KernelConstants {
                                try {
                                        return repository.login(workspaceName);
                                } catch (RepositoryException e) {
-                                       throw new CmsException("Cannot open admin session", e);
+                                       throw new IllegalStateException("Cannot open admin session", e);
                                }
                        }
 
                });
        }
 
+       static void asyncOpen(ServiceTracker<?, ?> st) {
+               Runnable run = new Runnable() {
+
+                       @Override
+                       public void run() {
+                               st.open();
+                       }
+               };
+               Activator.getInternalExecutorService().execute(run);
+//             new Thread(run, "Open service tracker " + st).start();
+       }
+
        /**
-        * @return the {@link BundleContext} of the {@link Bundle} which provided
-        *         this class, never null.
-        * @throws CmsException
-        *             if the related bundle is not active
+        * @return the {@link BundleContext} of the {@link Bundle} which provided this
+        *         class, never null.
+        * @throws CmsException if the related bundle is not active
         */
-       public static BundleContext getBundleContext(Class<?> clzz) {
-               Bundle bundle = FrameworkUtil.getBundle(clzz);
-               BundleContext bc = bundle.getBundleContext();
-               if (bc == null)
-                       throw new CmsException("Bundle " + bundle.getSymbolicName() + " is not active");
-               return bc;
+//     static BundleContext getBundleContext(Class<?> clzz) {
+////           Bundle bundle = FrameworkUtil.getBundle(clzz);
+//             BundleContext bc = Activator.getBundleContext();
+//             if (bc == null)
+//                     throw new CmsException("Bundle " + bundle.getSymbolicName() + " is not active");
+//             return bc;
+//     }
+
+       static BundleContext getBundleContext() {
+               return Activator.getBundleContext();
        }
 
-       private static BundleContext getBundleContext() {
-               return getBundleContext(KernelUtils.class);
+       static boolean asBoolean(String value) {
+               if (value == null)
+                       return false;
+               switch (value) {
+               case "true":
+                       return true;
+               case "false":
+                       return false;
+               default:
+                       throw new IllegalArgumentException(
+                                       "Unsupported value for attribute " + DataModelNamespace.ABSTRACT + ": " + value);
+               }
        }
 
        private static URI safeUri(String uri) {
                if (uri == null)
-                       throw new CmsException("URI cannot be null");
+                       throw new IllegalArgumentException("URI cannot be null");
                try {
                        return new URI(uri);
                } catch (URISyntaxException e) {
-                       throw new CmsException("Dadly formatted URI " + uri, e);
+                       throw new IllegalArgumentException("Badly formatted URI " + uri, e);
                }
        }