Revision: 1152
http://stripes.svn.sourceforge.net/stripes/?rev=1152&view=rev
Author: fdaoud
Date: 2009-09-27 11:28:21 +0000 (Sun, 27 Sep 2009)
Log Message:
-----------
Fix for STS-696. Thanks to Paul White.
Modified Paths:
--------------
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
trunk/tests/src/net/sourceforge/stripes/controller/BasicResolverTests.java
Modified:
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
===================================================================
---
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
2009-09-27 02:41:08 UTC (rev 1151)
+++
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
2009-09-27 11:28:21 UTC (rev 1152)
@@ -215,7 +215,7 @@
Method[] methods = clazz.getDeclaredMethods();
for (Method method : methods) {
- if ( Modifier.isPublic(method.getModifiers()) ) {
+ if ( Modifier.isPublic(method.getModifiers()) &&
!method.isBridge() ) {
String eventName = getHandledEvent(method);
// look for duplicate event names within the current class
Modified:
trunk/tests/src/net/sourceforge/stripes/controller/BasicResolverTests.java
===================================================================
--- trunk/tests/src/net/sourceforge/stripes/controller/BasicResolverTests.java
2009-09-27 02:41:08 UTC (rev 1151)
+++ trunk/tests/src/net/sourceforge/stripes/controller/BasicResolverTests.java
2009-09-27 11:28:21 UTC (rev 1152)
@@ -1,12 +1,16 @@
package net.sourceforge.stripes.controller;
-import net.sourceforge.stripes.action.UrlBinding;
+import java.lang.reflect.Method;
+import java.util.HashMap;
+import java.util.Map;
+import net.sourceforge.stripes.StripesTestFixture;
import net.sourceforge.stripes.action.ActionBean;
import net.sourceforge.stripes.action.ActionBeanContext;
-import net.sourceforge.stripes.action.Resolution;
import net.sourceforge.stripes.action.DefaultHandler;
+import net.sourceforge.stripes.action.ForwardResolution;
import net.sourceforge.stripes.action.HandlesEvent;
-import net.sourceforge.stripes.StripesTestFixture;
+import net.sourceforge.stripes.action.Resolution;
+import net.sourceforge.stripes.action.UrlBinding;
import net.sourceforge.stripes.mock.MockRoundtrip;
import org.testng.annotations.Test;
import org.testng.Assert;
@@ -39,6 +43,10 @@
return null;
}
+ public Resolution process() {
+ return null;
+ }
+
// Start of Test Methods
@Test(groups="fast")
@@ -88,4 +96,18 @@
Assert.assertEquals(bean.getNumber(), 2);
Assert.assertEquals(bean.getContext().getEventName(), "two");
}
+
+ @Test(groups="fast")
+ public void testOverrideHandlerMethodReturnsSubtype() throws
SecurityException, NoSuchMethodException {
+ NameBasedActionResolver resolver = new NameBasedActionResolver();
+ Map<String, Method> classMappings = new HashMap<String, Method>();
+ resolver.processMethods(ExtendedBaseAction.class, classMappings);
+ }
+
+ public static class ExtendedBaseAction extends BasicResolverTests {
+ @Override
+ public ForwardResolution process() {
+ return null;
+ }
+ }
}
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development
