Uploaded image for project: 'MyFaces Core'
  1. MyFaces Core
  2. MYFACES-1769

JSR-252 Issue #224: Corrected h:dataTable var to be a String, and not ValueExpression enabled

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.2.2
    • JSR-252
    • None
    • Tomcat 6.0.14, Myfaces 1.2.1-SNAPSHOT

    Description

      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()
        Unknown macro: { 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.

      Attachments

        Activity

          People

            matzew Matthias Wessendorf
            lu4242 Leonardo Uribe
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: