[Stripes-dev] SF.net SVN: stripes: [439] trunk/stripes/src/net/sourceforge/stripes/ controller/NameBasedActionResolver.java

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[Stripes-dev] SF.net SVN: stripes: [439] trunk/stripes/src/net/sourceforge/stripes/ controller/NameBasedActionResolver.java

tfenne
Revision: 439
          http://svn.sourceforge.net/stripes/?rev=439&view=rev
Author:   tfenne
Date:     2006-10-14 05:55:57 -0700 (Sat, 14 Oct 2006)

Log Message:
-----------
Fix for STS-274: Methods with parameters should not be registered as event handlers

Modified Paths:
--------------
    trunk/stripes/src/net/sourceforge/stripes/controller/NameBasedActionResolver.java

Modified: trunk/stripes/src/net/sourceforge/stripes/controller/NameBasedActionResolver.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/controller/NameBasedActionResolver.java 2006-10-14 02:46:13 UTC (rev 438)
+++ trunk/stripes/src/net/sourceforge/stripes/controller/NameBasedActionResolver.java 2006-10-14 12:55:57 UTC (rev 439)
@@ -215,8 +215,10 @@
 
         // If the method isn't annotated, but does return a resolution and is
         // not abstract (we already know it's public) then use the method name
-        if ( name == null && !Modifier.isAbstract(handler.getModifiers())
-                && Resolution.class.isAssignableFrom(handler.getReturnType()) ) {
+        if ( name == null
+                && !Modifier.isAbstract(handler.getModifiers())
+                    && Resolution.class.isAssignableFrom(handler.getReturnType())
+                        && handler.getParameterTypes().length == 0) {
 
             name = handler.getName();
         }


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Stripes-development mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/stripes-development