You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by rg...@apache.org on 2007/08/14 15:52:42 UTC

svn commit: r565759 - /struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Form.java

Author: rgielen
Date: Tue Aug 14 06:52:41 2007
New Revision: 565759

URL: http://svn.apache.org/viewvc?view=rev&rev=565759
Log:
WW-2109:
id resolution for <s:form> - minor cleanups, some former changes rolled back again

Modified:
    struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Form.java

Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Form.java
URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Form.java?view=diff&rev=565759&r1=565758&r2=565759
==============================================================================
--- struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Form.java (original)
+++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Form.java Tue Aug 14 06:52:41 2007
@@ -211,14 +211,10 @@
     protected void populateComponentHtmlId(Form form) {
         if (id != null) {
             addParameter("id", escape(id));
-        } else if (this.action != null) {
-            // if it isn't specified, we'll make somethig up
-            String action = findString(this.action);
-            if (action != null) {
-                addParameter("id", escape(action));
-            }
         }
 
+        // if no id given, it will be tried to generate it from the action attribute
+        // by the urlRenderer implementation
         urlRenderer.renderFormUrl(this);
     }