Why it's not working?

[ad-rocha]

Hi,

I'm starting to work wih JBoss AOP...

I' have two aspects:

package aftt.weaving;

import org.jboss.aop.joinpoint.MethodInvocation;

public class MyInstrumentation {

 public Object invoke(MethodInvocation invocation) throws Throwable {
 Object result = null;
 try {
 System.out.println("Entering method: " + invocation.getMethod());
 result = invocation.invokeNext();
 }
 finally {
 System.out.println("Leaving method: " + invocation.getMethod());
 }
 return result;
 }

}

package aftt.weaving;

public class MyAspect extends MyInstrumentation {

}

and a xml weaving file:

<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<aop>
 <bind pointcut="execution(public static boolean identifier.Identifier->verify(java.lang.String))">
 <advice name="instrument" aspect="aftt.weaving.MyAspect"/>
 </bind>
</aop>

When I try to weave I receive the following error:

[Log4jListener.java:150] [error] advice instrument cannot find aspect aftt.weaving.MyAspect file:/D:/Eclipse/workspace/AFTT/projects/Teste-Identifier/instrumentation/weaving.xml

What's wrong?

Thanks,

André

[ad-rocha]

Sorry... the correct method name in MyInstrumentation is instrument, not invoke

[bill.burke]

you need to declare the Aspect in XML before referencing it.

<aspect class="aftt.weaving.MyAspect"/>

[ad-rocha]

Thanks Bill, it's working now.

But why have I to referece the aspect if the advice tag define the aspect class?

<advice name="instrument" aspect="aftt.weaving.MyAspect"/>

[bill.burke]

Because aspect also has a scope attribute which by default is PER_VM

<aspect class="Foo" scope="PER_JOINPOINT"/>

Its also important because an aspect can take configuration as well.

Check out this doco:

http://docs.jboss.org/aop/aspect-framework/reference/en/html/xml.html#xml-aspect2

[ad-rocha]

Hum... thanks!