[jboss-cvs] JBossAS SVN: r67150 - projects/aop/trunk/aop/src/main/org/jboss/aop.

[jboss-cvs-commits at lists.jboss.org]

Author: kabir.khan at jboss.com
Date: 2007-11-15 17:18:42 -0500 (Thu, 15 Nov 2007)
New Revision: 67150

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/AspectManager.java
   projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java
Log:
[JBAOP-483] Class(Proxy)Container does not populate the class metadata - DOmain overrides attachMetaData()

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/AspectManager.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/AspectManager.java	2007-11-15 20:49:01 UTC (rev 67149)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/AspectManager.java	2007-11-15 22:18:42 UTC (rev 67150)
@@ -1103,12 +1103,14 @@
 
    protected void attachMetaData(Advisor advisor, Class clazz)
    {
-      Map metaData = getClassMetaData();
-      Iterator it = metaData.values().iterator();
-      while (it.hasNext())
+      synchronized (classMetaData)
       {
-         ClassMetaDataBinding data = (ClassMetaDataBinding) it.next();
-         addAdvisorToClassMetaDataBinding(data, clazz, advisor, clazz);
+         Iterator it = classMetaData.values().iterator();
+         while (it.hasNext())
+         {
+            ClassMetaDataBinding data = (ClassMetaDataBinding) it.next();
+            addAdvisorToClassMetaDataBinding(data, clazz, advisor, clazz);
+         }
       }
    }
 

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java	2007-11-15 20:49:01 UTC (rev 67149)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/Domain.java	2007-11-15 22:18:42 UTC (rev 67150)
@@ -830,7 +830,8 @@
       return parent.getPointcut(name);
    }
 
-   public void attachMetaData(ClassAdvisor advisor, Class clazz)
+   @Override
+   public void attachMetaData(Advisor advisor, Class clazz)
    {
       if (inheritsBindings)
       {