Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.13
    • Fix Version/s: 2.1.14, 2.2.0-m06
    • Component/s: None
    • Labels:
      None
    • Environment:

      GlassFish 3.1.2

      Description

      Trying to upgrade from JSF 1.2 to JSF 2, I've discovered that the old JSTL namespace doesn't work: http://java.sun.com/jstl/core

      Replacing it with http://java.sun.com/jsp/jstl/core fixes the problem.

      In my view, either the old namespace must be recognized or the comment in com\sun\faces\metadata\taglib\jstl-core.taglib.xml should be updated:

      <p>The pre JSF 2.0 version of Facelets incorrectly declared the taglib
      uri to be <code>http://java.sun.com/jstl/core</code>.  For backwards
      compatibility implementations must correctly handle inclusions with the
      incorrect uri, and the correct uri, declared here.</p>
      

        Activity

        Hide
        Manfred Riem added a comment -

        The JSF 2.0 documentation states the namespace at the top of the VDL docs for Facelets, see
        http://javaserverfaces.java.net/nonav/docs/2.0/vdldocs/facelets/c/tld-summary.html

        Show
        Manfred Riem added a comment - The JSF 2.0 documentation states the namespace at the top of the VDL docs for Facelets, see http://javaserverfaces.java.net/nonav/docs/2.0/vdldocs/facelets/c/tld-summary.html
        Hide
        Manfred Riem added a comment -

        Marking this issue as invalid as the documentation correctly states the required namespace.

        Show
        Manfred Riem added a comment - Marking this issue as invalid as the documentation correctly states the required namespace.
        Hide
        Andrei Ivanov added a comment -

        So, why not chose my other proposal to update the taglib xml comment?
        Why leave it like that if it's incorrect?

        Show
        Andrei Ivanov added a comment - So, why not chose my other proposal to update the taglib xml comment? Why leave it like that if it's incorrect?
        Hide
        Andrei Ivanov added a comment -

        And the JSF documentation I found here states the same thing as that comment, about backwards compatibility:
        http://docs.oracle.com/javaee/6/javaserverfaces/2.1/docs/vdldocs/facelets/c/tld-summary.html

        Show
        Andrei Ivanov added a comment - And the JSF documentation I found here states the same thing as that comment, about backwards compatibility: http://docs.oracle.com/javaee/6/javaserverfaces/2.1/docs/vdldocs/facelets/c/tld-summary.html
        Hide
        Manfred Riem added a comment -

        Reopening after due diligence of reporter pointing out comment in 2.1 API docs. Thanks!

        Show
        Manfred Riem added a comment - Reopening after due diligence of reporter pointing out comment in 2.1 API docs. Thanks!
        Hide
        dougd added a comment -

        r=dougd

        Show
        dougd added a comment - r=dougd
        Hide
        Manfred Riem added a comment -

        Applied to 2.1 branch,

        svn commit -m "Fixes http://java.net/jira/browse/JAVASERVERFACES-2523, r=dougd, Allow the 'http://java.sun.com/jstl/core' namespace to be used for backwards compatibility."
        Sending jsf-ri\src\main\java\com\sun\faces\application\ApplicationAssociate.java
        Sending jsf-ri\src\main\java\com\sun\faces\facelets\tag\jstl\core\JstlCoreLibrary.java
        Adding test\agnostic\renderKit\basic\nbactions.xml
        Adding test\agnostic\renderKit\basic\src\main\webapp\issue2523.xhtml
        Adding test\agnostic\renderKit\basic\src\test\java\com\sun\faces\test\agnostic\renderKit\basic\Issue2523IT.java
        Transmitting file data .....
        Committed revision 10806.

        Show
        Manfred Riem added a comment - Applied to 2.1 branch, svn commit -m "Fixes http://java.net/jira/browse/JAVASERVERFACES-2523 , r=dougd, Allow the 'http://java.sun.com/jstl/core' namespace to be used for backwards compatibility." Sending jsf-ri\src\main\java\com\sun\faces\application\ApplicationAssociate.java Sending jsf-ri\src\main\java\com\sun\faces\facelets\tag\jstl\core\JstlCoreLibrary.java Adding test\agnostic\renderKit\basic\nbactions.xml Adding test\agnostic\renderKit\basic\src\main\webapp\issue2523.xhtml Adding test\agnostic\renderKit\basic\src\test\java\com\sun\faces\test\agnostic\renderKit\basic\Issue2523IT.java Transmitting file data ..... Committed revision 10806.
        Hide
        Manfred Riem added a comment -

        Applied to 2.2 trunk,

        svn commit -m "Fixes http://java.net/jira/browse/JAVASERVERFACES-2523, r=dougd, Allow the 'http://java.sun.com/jstl/core' namespace to be used for backwards compatibility."
        Sending jsf-ri\src\main\java\com\sun\faces\application\ApplicationAssociate.java
        Sending jsf-ri\src\main\java\com\sun\faces\facelets\tag\jstl\core\JstlCoreLibrary.java
        Sending test\agnostic\renderKit\basic\nbactions.xml
        Adding test\agnostic\renderKit\basic\src\main\webapp\issue2523.xhtml
        Adding test\agnostic\renderKit\basic\src\test\java\com\sun\faces\test\agnostic\renderKit\basic\Issue2523IT.java
        Transmitting file data .....
        Committed revision 10807.

        Show
        Manfred Riem added a comment - Applied to 2.2 trunk, svn commit -m "Fixes http://java.net/jira/browse/JAVASERVERFACES-2523 , r=dougd, Allow the 'http://java.sun.com/jstl/core' namespace to be used for backwards compatibility." Sending jsf-ri\src\main\java\com\sun\faces\application\ApplicationAssociate.java Sending jsf-ri\src\main\java\com\sun\faces\facelets\tag\jstl\core\JstlCoreLibrary.java Sending test\agnostic\renderKit\basic\nbactions.xml Adding test\agnostic\renderKit\basic\src\main\webapp\issue2523.xhtml Adding test\agnostic\renderKit\basic\src\test\java\com\sun\faces\test\agnostic\renderKit\basic\Issue2523IT.java Transmitting file data ..... Committed revision 10807.

          People

          • Assignee:
            Manfred Riem
            Reporter:
            Andrei Ivanov
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: