(no commit message)
authorErik Brakkee <erik@brakkee.org>
Sat, 17 Jul 2010 21:36:12 +0000 (21:36 +0000)
committerErik Brakkee <erik@brakkee.org>
Sat, 17 Jul 2010 21:36:12 +0000 (21:36 +0000)
support/cdi/src/main/java/org/wamblee/cdi/BeanManagerLookup.java
support/cdi/src/main/java/org/wamblee/cdi/CdiInjectorFactory.java

index 50bed00a907aae1be4913e5a3108da8b75f6944b..afa42332afe1c016ac7ef40905510b719e8e3c07 100644 (file)
@@ -36,7 +36,7 @@ public class BeanManagerLookup {
         .getLogger(BeanManagerLookup.class.getName());
 
     public static final String BEAN_MANAGER_JNDI = "java:comp/BeanManager";
-    private static BeanManager mgr = null;
+    private static BeanManager BEAN_MANAGER = null;
 
     /**
      * Sets the bean manager (mainly for testability).
@@ -45,7 +45,7 @@ public class BeanManagerLookup {
      *            Bean manager.
      */
     public static void setBeanManager(BeanManager aMgr) {
-        mgr = aMgr;
+        BEAN_MANAGER = aMgr;
     }
 
     /**
@@ -54,16 +54,16 @@ public class BeanManagerLookup {
      * @return Bean manager.
      */
     public static BeanManager lookup() {
-        if (mgr == null) {
+        if (BEAN_MANAGER == null) {
             try {
                 InitialContext ctx = new InitialContext();
-                mgr = (BeanManager) ctx.lookup(BEAN_MANAGER_JNDI);
+                BEAN_MANAGER = (BeanManager) ctx.lookup(BEAN_MANAGER_JNDI);
                 LOGGER.info("Beanmanager successfully located");
             } catch (NamingException e) {
                 LOGGER.warning("No beanmanager was found, using null");
-                mgr = null;
+                BEAN_MANAGER = null;
             }
         }
-        return mgr;
+        return BEAN_MANAGER;
     }
 }
index de13ef54df162641c6552350a1508e25cef209ce..a4e575aa7f8b7bee581808a737948df1dc6e7d6b 100644 (file)
@@ -23,8 +23,8 @@ import org.wamblee.inject.Injector;
 import org.wamblee.inject.InjectorFactory;
 
 /**
- * Factory that creates CDI injectors. This class may be subclassed for testing to
- * override the injectors that are returned.
+ * Factory that creates CDI injectors. This class may be subclassed for testing
+ * to override the injectors that are returned.
  * 
  * @author Erik Brakkee
  */
@@ -34,11 +34,12 @@ public class CdiInjectorFactory implements InjectorFactory {
         .getLogger(CdiInjectorFactory.class.getName());
 
     private BeanManager beanManager;
-    
+
     /**
-     * Constructs the factory using a default bean manager. 
+     * Constructs the factory using a default bean manager.
      * 
-     * @throws IllegalArgumentException If bean manager is null. 
+     * @throws IllegalArgumentException
+     *             If bean manager is null.
      */
     public CdiInjectorFactory() {
         this(BeanManagerLookup.lookup());
@@ -48,10 +49,11 @@ public class CdiInjectorFactory implements InjectorFactory {
      * Constructs the factory using an explicit bean manager.
      * 
      * @param aBeanManager
-     * @throws IllegalArgumentException If bean manager is null. 
+     * @throws IllegalArgumentException
+     *             If bean manager is null.
      */
     public CdiInjectorFactory(BeanManager aBeanManager) {
-        if ( aBeanManager == null ) { 
+        if (aBeanManager == null) {
             throw new IllegalArgumentException("Bean manager is null");
         }
         beanManager = aBeanManager;