Details

    • Issuezilla Id:
      805
    • Status Whiteboard:
      Hide

      changelog

      Show
      changelog

      Description

      UIComponent.getChildren() says the following about PostAddToViewEvent delivery:

      "After the child component has been added to the view, if the following condition is not met:

      FacesContext.isPostback() returns true and FacesContext.getCurrentPhaseId() returns
      PhaseId.RESTORE_VIEW

      Application.publishEvent(javax.faces.context.FacesContext, java.lang.Class, java.lang.Object) must be
      called, passing PostAddToViewEvent.class as the first argument and the newly added component as the
      second argument."

      However:

      1. We do actually deliver PostAddToViewEvents during restore view when partial state saving is
      enabled.
      2. We don't actually want to deliver PostAddToViewEvents during render response when Facelets
      temporarily removes/re-adds existing components from the tree.
      3. These requirements are not specified in UIComponent.setParent(), which also discusses
      PostAddToViewEvent delivery:

      "This method will cause an PostAddToViewEvent to be published and if parent.isInView() returns true an
      PostAddToViewEvent will be published as well."

      The getChildren()/setParent() documentation should either be consistent, or we should remove the doc
      from one of these locations.

        Activity

        Hide
        Ed Burns added a comment -

        p2

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

        take ownership.

        Show
        Ed Burns added a comment - take ownership.
        Hide
        Ed Burns added a comment -

        Fix checked in.

        Show
        Ed Burns added a comment - Fix checked in.
        Hide
        rogerk added a comment -

        changelog

        Show
        rogerk added a comment - changelog
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

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

            Dates

            • Created:
              Updated:
              Resolved: