ICEfaces
  1. ICEfaces
  2. ICE-4636

Use of Java 5 API - incompatible with Java 1.4

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.1
    • Fix Version/s: 1.8.2-RC1, 1.8.2
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Any

      Description

      The BridgeFacesContext has this around line 148:

              String keepViewRoot = externalContext.getInitParameter("com.icesoft.faces.retainViewRoot");
              if (keepViewRoot != null) {
                  retainViewRoot = Boolean.parseBoolean(keepViewRoot);
              }

      The parseBoolean method is only available in Java 5+. We don't compile using 1.4 for various reasons and testing is probably not catching this due to the conditional requirement for the context parameter. Should be changed to:

              String keepViewRoot = externalContext.getInitParameter("com.icesoft.faces.retainViewRoot");
              if (keepViewRoot != null) {
                  retainViewRoot = Boolean.valueOf(keepViewRoot).booleanValue();
              }

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #18997 Thu Jun 25 11:00:47 MDT 2009 deryk.sinotte ICE-4636: Changing use of JDK 5 API to be 1.4 compatible
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/BridgeFacesContext.java
        Deryk Sinotte created issue -
        Deryk Sinotte made changes -
        Field Original Value New Value
        Assignee Deryk Sinotte [ deryk.sinotte ]
        Hide
        Deryk Sinotte added a comment -

        Modified to be 1.4 friendly.

        Show
        Deryk Sinotte added a comment - Modified to be 1.4 friendly.
        Deryk Sinotte made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.8.2 [ 10190 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 1.8.2-RC1 [ 10210 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Deryk Sinotte
            Reporter:
            Deryk Sinotte
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: