Transaction demarcation works with TransactionProxy but not AOP

[Santosh Ramachandrula]

Hello, I have a spring configuration in my app that works with TransactionProxyFactoryBean, now I am trying to use AOP instead of Proxy and it does not work. I am getting following error with AOP configuration

  org.hibernate.HibernateException: No Hibernate Session bound to thread, and configuration does not allow creation of non-transactional one here
	at org.springframework.orm.hibernate3.SpringSessionContext.currentSession(SpringSessionContext.java:63)
	at org.hibernate.impl.SessionFactoryImpl.getCurrentSession(SessionFactoryImpl.java:574)
	at com.santosh.model.dao.AbstractCRUDDao.findAll(AbstractCRUDDao.java:117)
	at com.santosh.model.dao.PersonDao.findAll(PersonDao.java:16)
	at com.santosh.delegate.PersonDelegate.findAll(PersonDelegate.java:27)

What do I need to do in AOP decleration to get this to work?

Working version using TransactionProxyFactoryBean

  	<bean id="personDelegate" class="org.springframework.transaction.interceptor.TransactionProxyFactoryBean" >
	<property name="target" ref="personDelegateTarget" />
	<property name="transactionManager" ref="txManager" />
	<property name="transactionAttributes">
	<props>
	    <prop key="*find*">PROPAGATION_REQUIRED</prop>
	    <prop key="*">PROPAGATION_SUPPORTS,readOnly</prop>
	</props>
	</property>
	</bean>  

AOP configuration thats not working.

   <tx:advice id="txAdvice" transaction-manager="txManager">
    <tx:attributes>
  	 	<tx:method name="*find*" propagation="REQUIRED" read-only="true"/>
  	 	<tx:method name="*" propagation="REQUIRED" read-only="false"/>
  	 </tx:attributes>
  </tx:advice>
  
  <aop:config>
  	<aop:advisor pointcut="execution(* *Delegate.*(..))" advice-ref="txAdvice" />
  </aop:config>  

[Christophe Verré]

I think that you have to declare the pointcut like this :

   <aop:config>  
     <aop:advisor id="rancherOperation" pointcut="execution(* *..*Delegate.*(..))" advice-ref="txAdviceRanch" />  
   </aop:config>   

To tell that you want to apply the advice on any class ending with "Delegate" from any package.

[Santosh Ramachandrula]

That worked. Thanks Chris.