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

Move SystemEventListener implementation from UIComponent down to UIComponentBase.

    Details

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

      Operating System: All
      Platform: Macintosh

    • Issuezilla Id:
      808
    • Status Whiteboard:
      Hide

      size_small importance_large

      Show
      size_small importance_large

      Description

      UIComponent provides implementations for:

      • subscribeToEvent
      • unsubscribeFromEvent
      • getListenersForEventClass

      And also provides storage for system event listeners.

      However, UIComponent does not provdie state saving for system event listeners. This is implemented
      by UIComponentBase.

      As a result, components that extend UIComponent directly cannot leverage the default system event
      listener method implementations, since:

      1. The listeners will not be state saved by UIComponent.
      2. There is no way for a subclass to access the listeners for custom state saving purposes.

      These concrete method implementations do not belong on UIComponent. They should be pushed
      down to UIComponentBase. Components that subclass UIComponent can provide their own system
      event listener storage/state saving.

        Activity

        Hide
        rogerk added a comment -

        Checked in.

        Show
        rogerk added a comment - 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 -

        Update status for JCP ChangeLog.

        Show
        Ed Burns added a comment - Update status for JCP ChangeLog.
        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:
            rogerk
            Reporter:
            aschwart
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: