myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Matthias We├čendorf (JIRA) <...@myfaces.apache.org>
Subject [jira] Reopened: (MYFACES-1769) JSR-252 Issue #224: Corrected h:dataTable var to be a String, and not ValueExpression enabled
Date Fri, 23 Nov 2007 20:05:43 GMT

     [ https://issues.apache.org/jira/browse/MYFACES-1769?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Matthias We├čendorf reopened MYFACES-1769:
-----------------------------------------


> JSR-252 Issue #224: Corrected h:dataTable var to be a String, and not ValueExpression
enabled
> ---------------------------------------------------------------------------------------------
>
>                 Key: MYFACES-1769
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1769
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-252
>         Environment: Tomcat 6.0.14, Myfaces 1.2.1-SNAPSHOT
>            Reporter: Leonardo Uribe
>            Priority: Minor
>
> Now myfaces has the following on UIData:
>   // Property: var
>   private String _var;
>   /**
>    * Gets Defines the name of the request-scope variable that will hold the current row
during iteration.  This value must be a static value.
>    *
>    * @return  the new var value
>    */
>   public String getVar()
>   {
>     if (_var != null)
>     {
>       return _var;
>     }
>     ValueExpression expression = getValueExpression("var");
>     if (expression != null)
>     {
>       return (String)expression.getValue(getFacesContext().getELContext());
>     }
>     return null;
>   }
>   /**
>    * Sets Defines the name of the request-scope variable that will hold the current row
during iteration.  This value must be a static value.
>    * 
>    * @param var  the new var value
>    */
>   public void setVar(String var)
>   {
>     this._var = var;
>   }
> Because var is a static String, according to the spec it should be as is:
>   // Property: var
>   private String _var;
>   /**
>    * Gets Defines the name of the request-scope variable that will hold the current row
during iteration.  This value must be a static value.
>    *
>    * @return  the new var value
>    */
>   public String getVar()
>   {
>       return _var;
>   }
>   /**
>    * Sets Defines the name of the request-scope variable that will hold the current row
during iteration.  This value must be a static value.
>    * 
>    * @param var  the new var value
>    */
>   public void setVar(String var)
>   {
>     this._var = var;
>   }
> HtmlDataTableTag has this:
>   private ValueExpression _var;
>   public void setVar(ValueExpression var)
>   {
>     _var = var;
>   }
> it should be this:
>   private String _var;
>   public void setVar(String var)
>   {
>    _var=var;
>   }
> The proper corrections on h.tld and others should be done.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message