[jboss-cvs] JBossAS SVN: r81288 - projects/microcontainer/trunk/kernel/src/main/java/org/jboss/beans/metadata/plugins/factory.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Nov 19 09:22:41 EST 2008


Author: alesj
Date: 2008-11-19 09:22:41 -0500 (Wed, 19 Nov 2008)
New Revision: 81288

Modified:
   projects/microcontainer/trunk/kernel/src/main/java/org/jboss/beans/metadata/plugins/factory/GenericBeanFactory.java
Log:
Fix classloader usage.

Modified: projects/microcontainer/trunk/kernel/src/main/java/org/jboss/beans/metadata/plugins/factory/GenericBeanFactory.java
===================================================================
--- projects/microcontainer/trunk/kernel/src/main/java/org/jboss/beans/metadata/plugins/factory/GenericBeanFactory.java	2008-11-19 12:54:30 UTC (rev 81287)
+++ projects/microcontainer/trunk/kernel/src/main/java/org/jboss/beans/metadata/plugins/factory/GenericBeanFactory.java	2008-11-19 14:22:41 UTC (rev 81288)
@@ -125,6 +125,7 @@
     * Get the classloader from the controller context
     * 
     * @return the controller context
+    * @throws Throwable for any error
     */
    private ClassLoader getControllerContextClassLoader() throws Throwable
    {
@@ -166,11 +167,13 @@
     * Get the access control context from the controller context
     * 
     * @return the access control
+    * @throws Throwable for any error
     */
    private AccessControlContext getAccessControlContext() throws Throwable
    {
       if (context != null)
       {
+         // TODO: JBMICROCONT-386, introduce interface
          if (context instanceof AbstractKernelControllerContext == false)
             return null;
          
@@ -198,12 +201,12 @@
     */
    private Object createBean(ClassLoader cl) throws Throwable
    {
-      ClassLoader cl2 = cl;
-      if (cl2 == null)
-         cl2 = Configurator.getClassLoader(classLoader);
+      ClassLoader loader = cl;
+      if (loader == null)
+         loader = Configurator.getClassLoader(classLoader);
       BeanInfo info = null;
       if (bean != null)
-         info = configurator.getBeanInfo(bean, cl, accessMode);
+         info = configurator.getBeanInfo(bean, loader, accessMode);
 
       Joinpoint joinpoint = configurator.getConstructorJoinPoint(info, constructor, null);
       Object result = joinpoint.dispatch();
@@ -217,11 +220,11 @@
             String property = entry.getKey();
             ValueMetaData vmd = entry.getValue();
             PropertyInfo pi = info.getProperty(property);
-            pi.set(result, vmd.getValue(pi.getType(), cl));
+            pi.set(result, vmd.getValue(pi.getType(), loader));
          }
       }
-      invokeLifecycle("create", create, info, cl, result);
-      invokeLifecycle("start", start, info, cl, result);
+      invokeLifecycle("create", create, info, loader, result);
+      invokeLifecycle("start", start, info, loader, result);
       return result;
    }
 }
\ No newline at end of file




More information about the jboss-cvs-commits mailing list