Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.2
    • Component/s: Components/Renderers
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: Macintosh

    • Issuezilla Id:
      757
    • Status Whiteboard:
      Hide

      cat2 javadoc size_medium importance_large

      Show
      cat2 javadoc size_medium importance_large

      Description

      The JavaDoc for FacesContext.setViewRoot() currently say:

      "Set the root component that is associated with this request. This method can only be called by the
      application handler (or a class that the handler calls), and only during the Invoke Application phase of
      the request processing lifecycle."

      However, JSF itself calls setViewRoot() during the restore view phase, which conflicts with the above
      statement. In addition, we have been recommending the use of the PreRenderView events as a
      substitute for view actions until view actions can be added in JSF 2.1. Navigation in response to
      PreRenderView events would also contradict the above statement.

      We need to re-evaluate what restrictions should apply to setViewRoot() (and, by extension,
      NavigationHandler.handleNavigation() and ConfigurableNavigation.performNavigation()).

        Activity

        Hide
        Ed Burns added a comment -

        cat2

        Show
        Ed Burns added a comment - cat2
        Hide
        Ed Burns added a comment -

        javadoc

        Show
        Ed Burns added a comment - javadoc
        Hide
        Ed Burns added a comment -

        components

        Show
        Ed Burns added a comment - components
        Hide
        Ed Burns added a comment -

        These are targeted at 2.1.

        Show
        Ed Burns added a comment - These are targeted at 2.1.
        Hide
        Ed Burns added a comment -

        triage

        Show
        Ed Burns added a comment - triage
        Hide
        Ed Burns added a comment -

        rogerk

        Show
        Ed Burns added a comment - rogerk
        Hide
        rogerk added a comment -

        triage

        Show
        rogerk added a comment - triage
        Hide
        rogerk added a comment -

        Fixed in 2.1 (reviswion r8577).

        Show
        rogerk added a comment - Fixed in 2.1 (reviswion r8577).
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

          • Assignee:
            rogerk
            Reporter:
            aschwart
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: