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

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Thu Sep 18 01:25:11 EDT 2008


Author: flavia.rainone at jboss.com
Date: 2008-09-18 01:25:11 -0400 (Thu, 18 Sep 2008)
New Revision: 78663

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/Advisor.java
Log:
[JBAOP-646] The bug is fixed (added a binding.addAdvisor(this) call to every binding involved in the chain
rebuild process.

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/Advisor.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/Advisor.java	2008-09-18 05:09:08 UTC (rev 78662)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/Advisor.java	2008-09-18 05:25:11 UTC (rev 78663)
@@ -1407,6 +1407,7 @@
                if ((!write && ab.getPointcut().matchesGet(this, field))
                      || (write && ab.getPointcut().matchesSet(this, field)))
                {    
+                  ab.addAdvisor(this);
                   pointcutResolved(fieldInfos[i], ab, new FieldJoinpoint(field));
                }
             }
@@ -1437,6 +1438,7 @@
                {
                   if (ab.getPointcut().matchesExecution(this, constructor))
                   {
+                     ab.addAdvisor(this);
                      pointcutResolved(constructorInfos[i], ab, new ConstructorJoinpoint(constructor));
                   }
                }
@@ -1467,6 +1469,7 @@
                {
                   if (binding.getPointcut().matchesConstruction(this, constructor))
                   {
+                     ab.addAdvisor(this);
                      pointcutResolved(constructionInfos[i], ab, new ConstructorJoinpoint(constructor));
                   }
                }




More information about the jboss-cvs-commits mailing list