ICEfaces
  1. ICEfaces
  2. ICE-1934

f:attribute does not work with JSF 1.2 runtime

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.6.3, 1.7Beta1, 1.7
    • Component/s: None
    • Labels:
      None
    • Environment:
      Firefox/IE, ICEFaces 1.6, JSF 1.2

      Description

      f:attribute does not work with JSF 1.2 runtime. I have the following in a page:

      <ice:commandButton value="Test" actionListener="#{myBean.onTest}">
               <f:attribute name="i" value="testval"/>
       </ice:commandButton>

      When I try to load the page in the browser, it throws the following error:

      javax.faces.FacesException: java.lang.IllegalArgumentException: Cannot invoke com.sun.faces.taglib.jsf_core.AttributeTag.setName - argument type mismatch

      The same page works fine with JSF 1.1 runtime.

        Activity

        Alok Mittal created issue -
        Hide
        Brendan Billingsley added a comment -

        It is not clear to me if this is a JSF 1.2 issue in the implementation or a icefaces issue.

        Show
        Brendan Billingsley added a comment - It is not clear to me if this is a JSF 1.2 issue in the implementation or a icefaces issue.
        Hide
        Brendan Billingsley added a comment -

        This is a very serious issue, please give us some update on it.

        Show
        Brendan Billingsley added a comment - This is a very serious issue, please give us some update on it.
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Mandeep Hayher [ mandeep.hayher ]
        Ken Fyten made changes -
        Assignee Mandeep Hayher [ mandeep.hayher ] Ted Goddard [ ted.goddard ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15520 Wed Jan 09 11:31:43 MST 2008 ted.goddard reflectively checking for name ValueExpressions (ICE-1934)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/parser/ELSetPropertiesRule.java
        Hide
        Ted Goddard added a comment -

        A fix is checked in to the trunk. Is this also needed in the 1.6 branch?

        The basic idea is to reflectively check for the type expected by setName on the tag, and if the tag is expecting a ValueExpression, to pass a ValueExpression rather than a String.

        Show
        Ted Goddard added a comment - A fix is checked in to the trunk. Is this also needed in the 1.6 branch? The basic idea is to reflectively check for the type expected by setName on the tag, and if the tag is expecting a ValueExpression, to pass a ValueExpression rather than a String.
        Ted Goddard made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.7 [ 10080 ]
        Resolution Fixed [ 1 ]
        Hide
        Ken Fyten added a comment -

        Please backport to 1.6 brand and add 1.6.3 as a fix target.

        Show
        Ken Fyten added a comment - Please backport to 1.6 brand and add 1.6.3 as a fix target.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15524 Wed Jan 09 17:04:34 MST 2008 ted.goddard reflectively checking for name ValueExpressions (ICE-1934)
        Files Changed
        Commit graph MODIFY /icefaces/branches/icefaces-1.6/icefaces/core/src/com/icesoft/faces/webapp/parser/ELSetPropertiesRule.java
        Hide
        Ted Goddard added a comment -

        fix ported to icefaces-1.6 branch

        Show
        Ted Goddard added a comment - fix ported to icefaces-1.6 branch
        Ted Goddard made changes -
        Fix Version/s 1.6.3 [ 10120 ]
        Ted Goddard made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 1.7Beta1 [ 10121 ]
        Fix Version/s 1.7 [ 10080 ]
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Ted Goddard [ ted.goddard ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Alok Mittal
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: