Put traces back in

git-svn-id: http://anonsvn.jboss.org/repos/weld/ri/trunk@568 1c488680-804c-0410-94cd-c6b725194a0e
weld · Dec 19, 2008 · 8b59546 · 8b59546
1 parent ed673cb
commit 8b59546
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 8 deletions.
diff --git a/webbeans-ri/src/main/java/org/jboss/webbeans/contexts/AbstractBeanMapAdaptor.java b/webbeans-ri/src/main/java/org/jboss/webbeans/contexts/AbstractBeanMapAdaptor.java
@@ -24,9 +24,15 @@
 import javax.webbeans.manager.Contextual;
 
 import org.jboss.webbeans.CurrentManager;
+import org.jboss.webbeans.log.LogProvider;
+import org.jboss.webbeans.log.Logging;
+import org.jboss.webbeans.servlet.ApplicationBeanMap;
 
 public abstract class AbstractBeanMapAdaptor implements BeanMap
 {
+   // The log provider
+   private static LogProvider log = Logging.getLogProvider(ApplicationBeanMap.class);
+
    /**
     * Gets a bean from the map
     * 
@@ -38,6 +44,7 @@ public <T> T get(Contextual<? extends T> bean)
    {
       String key = getBeanKey(bean);
       T instance = (T) getAttribute(key);
+      log.trace("Looked for " + key + " and got " + instance);
       return instance;
    }
 
@@ -52,6 +59,7 @@ public <T> T remove(Contextual<? extends T> bean)
       T instance = get(bean);
       String key = getBeanKey(bean);
       removeAttribute(key);
+      log.trace("Removed bean under key " + key);
       return instance;
    }
 
@@ -66,7 +74,9 @@ public void clear()
       {
          String name = (String) names.nextElement();
          removeAttribute(name);
+         log.trace("Cleared " + name);
       }
+      log.trace("Bean Map cleared");
    }
 
    /**
@@ -103,6 +113,7 @@ public <T> void put(Contextual<? extends T> bean, T instance)
    {
       String key = getBeanKey(bean);
       setAttribute(key, instance);
+      log.trace("Added bean " + bean + " under key " + key);
    }
 
    /**
@@ -127,6 +138,12 @@ public <T> void put(Contextual<? extends T> bean, T instance)
     */
    protected abstract Enumeration<String> getAttributeNames();
 
+   /**
+    * Sets an instance under a key in the underlying storage
+    * 
+    * @param key The key
+    * @param instance The instance
+    */
    protected abstract void setAttribute(String key, Object instance);
 
    /**

diff --git a/webbeans-ri/src/main/java/org/jboss/webbeans/servlet/ApplicationBeanMap.java b/webbeans-ri/src/main/java/org/jboss/webbeans/servlet/ApplicationBeanMap.java
@@ -23,8 +23,6 @@
 
 import org.jboss.webbeans.contexts.AbstractBeanMapAdaptor;
 import org.jboss.webbeans.contexts.ApplicationContext;
-import org.jboss.webbeans.log.LogProvider;
-import org.jboss.webbeans.log.Logging;
 
 /**
  * A BeanMap that uses a servlet context as backing map
@@ -35,8 +33,6 @@
  */
 public class ApplicationBeanMap extends AbstractBeanMapAdaptor
 {
-   // The log provider
-   private static LogProvider log = Logging.getLogProvider(ApplicationBeanMap.class);
    // The servlet context to use as backing map
    private ServletContext context;
 

diff --git a/webbeans-ri/src/main/java/org/jboss/webbeans/servlet/SessionBeanMap.java b/webbeans-ri/src/main/java/org/jboss/webbeans/servlet/SessionBeanMap.java
@@ -23,8 +23,6 @@
 
 import org.jboss.webbeans.contexts.AbstractBeanMapAdaptor;
 import org.jboss.webbeans.contexts.ApplicationContext;
-import org.jboss.webbeans.log.LogProvider;
-import org.jboss.webbeans.log.Logging;
 
 /**
  * A BeanMap that uses a HTTP session as backing map
@@ -35,8 +33,6 @@
  */
 public class SessionBeanMap extends AbstractBeanMapAdaptor
 {
-   // The log provider
-   private static LogProvider log = Logging.getLogProvider(ApplicationBeanMap.class);
    // The HTTP session context to use as backing map
    private HttpSession session;