DO NOT REPLY [Bug 10194] New: - Servlet Mapping bug in FormTag

[bu...@apache.org]

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=10194>.
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=10194

Servlet Mapping bug in FormTag

           Summary: Servlet Mapping bug in FormTag
           Product: Struts
           Version: 1.1 Beta 1
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Custom Tags
        AssignedTo: struts-dev@jakarta.apache.org
        ReportedBy: jfarley@studentadvantage.com


Struts requires that for any application deployed on the framework you must map 
the controller servlet to some URL via the web.xml file.  The current 
implementation suggests that you map the servlet to a URL that includes either 
some prefix or suffix, such as "/exec" or "*.do".  Placing either of these 
mappings in the URL was unacceptable to our business people (go fig).  They 
wanted straight URLs like this: "/app/some/action" (basically map the 
controller servlet to "/").  This works fine for the entire struts framework 
*except* for the FormTag which generates no "action=" parameter in the 
resultant HTML <form> tag.  I will post the fix here & to the struts-dev 
mailing list.

Index: FormTag.java
===================================================================
RCS file: /home/cvspublic/jakarta-
struts/src/share/org/apache/struts/taglib/html/FormTag.java,v
retrieving revision 1.22
diff -u -r1.22 FormTag.java
--- FormTag.java	23 Jun 2002 03:38:12 -0000	1.22
+++ FormTag.java	24 Jun 2002 20:20:13 -0000
@@ -789,6 +789,8 @@
                 value.append(servletMapping.substring
                              (0, servletMapping.length() - 2));
                 value.append(actionMapping);
+            } else if (servletMapping.equals("/")) {
+                value.append(actionMapping);
             }
             if (queryString != null) {
                 value.append(queryString);

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>