javaserverfaces
  1. javaserverfaces
  2. JAVASERVERFACES-1200

f:phaseListener not set-up when embedded in composite component

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-BETA2
    • Fix Version/s: 2.0.0-b15
    • Component/s: None
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      1,200

      Description

      A <f:phaseListener> tag has no more effect (not called any more) when moved from
      the xhtml page into a composite component definition file.

      A small web application will be attached to this issue for testing.

      The task was to implement a file-upload component. According to the solution
      pattern I know, I wanted to implement a phase listener which intercepts and
      processes the 'multi-part form data' stream from the request. This phase
      listener should automatically be installed when the composite component is used,
      therefore the tag location in the composites definition.

        Activity

        Hide
        Ryan Lubke added a comment -

        Re-opening as it seems that changes related to issue 1186 have caused the test
        to fail. It seems the processing relied on composite:implementation being
        processed during the metadata gathering phase of the composite component
        lifecycle.

        With 1186, this no longer occurs and now listeners aren't registered.

        Show
        Ryan Lubke added a comment - Re-opening as it seems that changes related to issue 1186 have caused the test to fail. It seems the processing relied on composite:implementation being processed during the metadata gathering phase of the composite component lifecycle. With 1186, this no longer occurs and now listeners aren't registered.
        Hide
        Ryan Lubke added a comment -

        Created an attachment (id=927)
        Alternate patch

        Show
        Ryan Lubke added a comment - Created an attachment (id=927) Alternate patch
        Hide
        Ed Burns added a comment -

        r=edburns

        Show
        Ed Burns added a comment - r=edburns
        Hide
        Ryan Lubke added a comment -

        New set of changes applied (r7594) to resolve this issue.

        Show
        Ryan Lubke added a comment - New set of changes applied (r7594) to resolve this issue.
        Hide
        Manfred Riem added a comment -

        Closing issue out

        Show
        Manfred Riem added a comment - Closing issue out

          People

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

            Dates

            • Created:
              Updated:
              Resolved: