Nice change. That particular chunk of code looks way better. And
there's more functionality, double bonus :)
-David
On Dec 27, 2007, at 4:21 AM, [EMAIL PROTECTED] wrote:
Author: jlaskowski
Date: Thu Dec 27 04:21:44 2007
New Revision: 607050
URL: http://svn.apache.org/viewvc?rev=607050&view=rev
Log:
OPENEJB-740 Business method interceptor methods defined by the bean
class's superclasses are not invoked
Modified:
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/
apache/openejb/config/AnnotationDeployer.java
Modified: openejb/trunk/openejb3/container/openejb-core/src/main/
java/org/apache/openejb/config/AnnotationDeployer.java
URL:
http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java?rev=607050&r1=607049&r2=607050&view=diff
=
=
=
=
=
=
=
=
======================================================================
--- openejb/trunk/openejb3/container/openejb-core/src/main/java/org/
apache/openejb/config/AnnotationDeployer.java (original)
+++ openejb/trunk/openejb3/container/openejb-core/src/main/java/org/
apache/openejb/config/AnnotationDeployer.java Thu Dec 27 04:21:44 2007
@@ -688,8 +688,8 @@
}
}
- Interceptors interceptors =
clazz.getAnnotation(Interceptors.class);
- if (interceptors != null) {
+ for (Class<?> interceptorsAnnotatedClass :
inheritedClassFinder.findAnnotatedClasses(Interceptors.class)) {
+ Interceptors interceptors =
interceptorsAnnotatedClass.getAnnotation(Interceptors.class);
EjbJar ejbJar = ejbModule.getEjbJar();
for (Class interceptor : interceptors.value()) {
if
(ejbJar.getInterceptor(interceptor.getName()) == null) {
@@ -706,7 +706,7 @@
}
for (Method method :
classFinder.findAnnotatedMethods(Interceptors.class)) {
- interceptors =
method.getAnnotation(Interceptors.class);
+ Interceptors interceptors =
method.getAnnotation(Interceptors.class);
if (interceptors != null) {
EjbJar ejbJar = ejbModule.getEjbJar();
for (Class interceptor :
interceptors.value()) {
