Revision: 1235
http://stripes.svn.sourceforge.net/stripes/?rev=1235&view=rev
Author: bengunter
Date: 2010-05-18 18:02:42 +0000 (Tue, 18 May 2010)
Log Message:
-----------
Fixed STS-740. Fully construct the FilterMethods object and all the lists it
maintains before adding it to the cache to prevent concurrent modification as
the lists are being built at startup.
Modified Paths:
--------------
branches/1.5.x/stripes/src/net/sourceforge/stripes/controller/BeforeAfterMethodInterceptor.java
Modified:
branches/1.5.x/stripes/src/net/sourceforge/stripes/controller/BeforeAfterMethodInterceptor.java
===================================================================
---
branches/1.5.x/stripes/src/net/sourceforge/stripes/controller/BeforeAfterMethodInterceptor.java
2010-05-18 17:47:52 UTC (rev 1234)
+++
branches/1.5.x/stripes/src/net/sourceforge/stripes/controller/BeforeAfterMethodInterceptor.java
2010-05-18 18:02:42 UTC (rev 1235)
@@ -196,8 +196,7 @@
FilterMethods filterMethods = filterMethodsCache.get(beanClass);
if (filterMethods == null) {
filterMethods = new FilterMethods();
- filterMethodsCache.put(beanClass, filterMethods);
-
+
// Look for @Before and @After annotations on the
methods in the ActionBean class
Collection<Method> methods =
ReflectUtil.getMethods(beanClass);
for (Method method : methods) {
@@ -239,6 +238,8 @@
}
}
}
+
+ filterMethodsCache.put(beanClass, filterMethods);
}
return filterMethods;
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development
