javaserverfaces-spec-public
  1. javaserverfaces-spec-public
  2. JAVASERVERFACES_SPEC_PUBLIC-663

apply() needs to be called twice: restore-view, after invoke application

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.1
    • Component/s: Facelets/VDL
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      663
    • Status Whiteboard:
      Hide

      cat2 frame size_small importance_large

      Show
      cat2 frame size_small importance_large

      Description

      Hi,

      we use c:if with an attribute which changes in invokeApplication, but remain on
      the same page. With Facelets old, this meant that the tag-handlers were
      reapplied, and we saw the changed component tree. With Facelets included in
      Faces itself, there is the following check:

      FaceletViewHandlingStrategy.buildView():

      if (Util.isViewPopulated(ctx, view))

      { return; }

      which effectively means that the tag-handlers are NOT reapplied, and the
      component tree stays the same, even if the value of the test-condition of a c:if
      toggled.

      A toggling test-condition in c:if's is widely used in our applications; we can
      therefore not upgrade to 2.0 without a fix for this.

      I wrongly filed this as an implementation bug first:

      https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1408

      regards,

      Martin

        Activity

        Hide
        Ed Burns added a comment -

        Target Milestone 4

        Show
        Ed Burns added a comment - Target Milestone 4
        Hide
        Ed Burns added a comment -

        Fix checked in.

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

        add changelog_2_1 keyword

        Show
        Ed Burns added a comment - add changelog_2_1 keyword
        Hide
        Ed Burns added a comment -

        Ensure changelog_2_1 is in keywords list

        Show
        Ed Burns added a comment - Ensure changelog_2_1 is in keywords list
        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:
            mmarinschek
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: