ICEfaces
  1. ICEfaces
  2. ICE-1990

ViewListener.viewDisposed() not called when the window is closed

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.6
          • Fix Version/s: 1.6.2, 1.7DR#1, 1.7
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            win xp

            Description

            ViewListener.viewDisposed() is supposed to be called on window closed, but only seems to be called on session invalidation.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Mircea Toma added a comment -

              Removed ViewListener. Rectored how beans are notified to dispose their resources. Application beans need to implement com.icesoft.faces.context.DisposableBean interface to receive the notification based on what their scope is.
              When in scope:

              • request - beans are disposed when view is disposed
              • session - beans are disposed when session expires
              • application - beans are disposed when context is destroyed

              Also, for beans in request scope and concurrentDOMViews=false the view is reused on reload so the application beans are reused as well (no disposing).

              Show
              Mircea Toma added a comment - Removed ViewListener. Rectored how beans are notified to dispose their resources. Application beans need to implement com.icesoft.faces.context.DisposableBean interface to receive the notification based on what their scope is. When in scope: request - beans are disposed when view is disposed session - beans are disposed when session expires application - beans are disposed when context is destroyed Also, for beans in request scope and concurrentDOMViews=false the view is reused on reload so the application beans are reused as well (no disposing).
              Hide
              Permalink
              Mircea Toma added a comment -

              Applied fix to 1.6 branch.

              Show
              Mircea Toma added a comment - Applied fix to 1.6 branch.
              Hide
              Permalink
              Ken Fyten added a comment -

              Rolled back ViewListener removal for backwards compatibility reasons. Marked ViewListener as deprecated instead.

              Show
              Ken Fyten added a comment - Rolled back ViewListener removal for backwards compatibility reasons. Marked ViewListener as deprecated instead.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Philip Breau
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: