It wasn't. I didn't realise you had fixed it?

The problem is there is no real dependency.

The Aspect definition is deployed to AOP immediatley
but there is no mechanism to link the redeploy of the advice factory
to a reworking of what AOP has cached.

The rebind() is an ugly way to do this.

What we really need is for the advice factory installation into the
AspectManager to be under the control of MC such that it
gets reworked when it is redeployed. This also removes the
need to do the lazy lookup.

A related problem, that I am going to write a test for,
is that we need to differentiate "static" advice factories configured
through an -aop.xml and those done under the control of the MC.
The former should NOT be returned from getDependencies().

The smoke test works now. Thanks.

I've started those tests on the advisor implementation, I mentioned above.
To run them

ejort@ejort:~/jboss-head/aspects$ ./build.sh -f build-test50.xml test -Dtest=microcontainer/advisor/test

I mean if we change the logic to a push then there is no need for
the lookup (pull).

There are three pieces:
pointcut definition
advice instance(s)
advice factory

Only the pointcut definition must be registered with AOP directly.
If the act of re-installing the advice factory pushed itself into AOP
then it could update the advice instance(s) at that point.

This is what the install does.

But this is just more complication and moving even further away from IOC.
And it is completely unnecessary except for when it is a PER_VM advice.

All I want is to be able the inject advice factory into the aop config
when it is fully constructed. The order of what happens should be:

1) Add pointcut definition to AOP
2) Deploy bean that matches the pointcut (waits for advice factory)
3) AdviceFactory dependencies resolved -> update aop config with advice factory
4) Create beans that use the advice factory

For redeploy
1) Undeploy beans using advice factory
2) Remove advice factory from aop config
3) Undeploy dependency of advice factory
4) Redeploy dependency
5) Add new advice factory back into aop config
6) Redeploy beans using advice factory

The only reason this fails is because my prototype doesn't do
(2) and (5).

It did until we realised this meant that the pointcut definition
was undeployed as well. Which meant aop didn't know there
was a dependency because as far as it was concerned the pointcut
didn't apply.
To fix that we broke the connection and used the lazy lookup
rather than injection.

"kabir.khan@jboss.com" wrote:
The "listener" manages the dependencies

Now that I've done this feature:
http://jira.jboss.com/jira/browse/JBMICROCONT-73

We can do a proper version of the aop beans.

e.g. instead of

<deployment xmlns="urn:jboss:bean-deployer:2.0">

 <beanfactory name="InterceptedAdvice" class="org.jboss.test.microcontainer.support.InterceptorWithDependency">
 <property name="simpleBean"><inject bean="Dependency"/></property>
 <install bean="InterceptedAspect" method="rebind"/>
 <install bean="InterceptedBinding" method="rebind">
 <parameter><inject bean="InterceptedAspect" property="definition"/></parameter>
 </install>
 </beanfactory>

 <bean name="InterceptedAspect" class="org.jboss.aop.microcontainer.prototype.Aspect">
 <property name="adviceBean">InterceptedAdvice</property>
 <property name="manager"><inject bean="AspectManager"/></property>
 </bean>

 <bean name="InterceptedBinding" class="org.jboss.aop.microcontainer.prototype.AspectBinding">
 <property name="pointcut">execution(* @org.jboss.test.microcontainer.support.Test->*(..))</property>
 <property name="aspect"><inject bean="InterceptedAspect" property="definition"/></property>
 <property name="manager"><inject bean="AspectManager"/></property>
 </bean>

<deployment xmlns="urn:jboss:bean-deployer:2.0">

 <aop:advice xmlns:aop="urn:jboss:aop-deployer:1.0"
 name="InterceptedAdvice"
 class="org.jboss.test.microcontainer.support.InterceptorWithDependency"
 >
 <property name="simpleBean"><inject bean="Dependency"/></property>
 <pointcut>execution(* @org.jboss.test.microcontainer.support.Test->*(..))</pointcut>
 </aop:advice>

Yes please.

P.S. Feel free to change the prototype beans.
But I'd like to have them IOC based (as much as possible).

Although the user won't see them if they use the "use case xml"
the Microcontainer does see the three beans that make up the
aspect definition.

I was going to make a start on the aop xml today,
but obviously this is really something you should be happy with
and capable of maintianing.

I also want to add an

<aop:introduction>
 <interface>com.acme.Blah</interface>
 <pointcut>@com.acme.BlahAnnotation</pointcut>
</aop:introduction>

<bean>
 <annotation>com.acme.BlahAnnotation</annotation>
</bean>