struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 26192] - forward element redirect and className attributes not honored
Date Sat, 24 Jan 2004 13:40:15 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=26192>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=26192

forward element redirect and className attributes not honored

darkeye@tyrell.hu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From darkeye@tyrell.hu  2004-01-24 13:40 -------
the controller tag doesn't help, and it seems that the SuccessAction class was
problematic as well, as it implemented some process method instead of execute.

Nevertheless, I found the bug, and here's the fix. The difference between our
approaches was that I'm using wildcards in action mappings, as described in
section 4.10 of the user guide:
http://jakarta.apache.org/struts/userGuide/building_controller.html#action_mapping_wildcards

It seems that the ActionConfigMatcher object did not honor the redirect flag.
Here's a CVS diff for fixing this problem:

Index: src/share/org/apache/struts/config/ActionConfigMatcher.java
===================================================================
RCS file:
/home/cvspublic/jakarta-struts/src/share/org/apache/struts/config/ActionConfigMatcher.java,v
retrieving revision 1.7
diff -r1.7 ActionConfigMatcher.java
207a208
>             cfg.setRedirect(fConfigs[x].getRedirect());


can you please make sure this fix gets into the CVS tree? thanks.

---------------------------------------------------------------------
To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: struts-dev-help@jakarta.apache.org


Mime
View raw message