You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by mu...@apache.org on 2007/07/09 15:39:01 UTC

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

Author: musachy
Date: Mon Jul  9 06:38:57 2007
New Revision: 554643

URL: http://svn.apache.org/viewvc?view=rev&rev=554643
Log:
WW-2027 Tags that push values into the value stack should use the "var" attribute to specify the name of the variable
* 'var' cannot be required until 'id' is dropped

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

Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Set.java
URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Set.java?view=diff&rev=554643&r1=554642&r2=554643
==============================================================================
--- struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Set.java (original)
+++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/Set.java Mon Jul  9 06:38:57 2007
@@ -118,7 +118,10 @@
         return super.end(writer, body);
     }
 
-    @StrutsTagAttribute(description="Name used to reference the value pushed into the Value Stack", required=true)
+    /* 
+     * TODO: set required=true when 'id' is dropped after 2.1
+     */
+    @StrutsTagAttribute(description="Name used to reference the value pushed into the Value Stack", required=false)
     public void setVar(String var) {
        super.setVar(var);
     }