ICEfaces
  1. ICEfaces
  2. ICE-9388

PushRenderer.removeCurrentSession does not stop push heartbeat

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.2.0.GA, 3.3
          • Fix Version/s: EE-3.3.0.GA_P01, 4.0.BETA, 4.0
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            -
          • Assignee Priority:
            P2
          • Salesforce Case Reference:

            Description

            PushRenderer.removeCurrentSession(groupName) doesn't stop the push heart beat. This method should put the push heartbeat back into "lazy" mode.

            Attached is a modified chat sample containing a commandButton that calls removeCurrentSession.

              Activity

              Ascending order - Click to sort in descending order
              Tyler Johnson created issue -
              Tyler Johnson made changes -
              Field Original Value New Value
              Salesforce Case Reference 5007000000T8YkzAAF
              Tyler Johnson made changes -
              Attachment chat.war [ 16100 ]
              Tyler Johnson made changes -
              Attachment chat.war [ 16100 ]
              Tyler Johnson made changes -
              Attachment chat.war [ 16101 ]
              Ken Fyten made changes -
              Fix Version/s 3.4 [ 10770 ]
              Ken Fyten made changes -
              Assignee Mircea Toma [ mircea.toma ]
              Assignee Priority P1 [ 10010 ]
              Ken Fyten made changes -
              Assignee Priority P1 [ 10010 ] P2 [ 10011 ]
              Ken Fyten made changes -
              Fix Version/s EE-3.3.0.GA_P01 [ 11174 ]
              Hide
              Permalink
              Mircea Toma added a comment -

              Modified BridgeSetup to trigger an update that deregisters the pushIDs whenever push is not enabled anymore. Modified LazyPushManager to use a boolean field to keep track of the push session state, thus is enough to call once addCurrentSession or removeCurrentSession on the PushRenderer.

              Show
              Mircea Toma added a comment - Modified BridgeSetup to trigger an update that deregisters the pushIDs whenever push is not enabled anymore. Modified LazyPushManager to use a boolean field to keep track of the push session state, thus is enough to call once addCurrentSession or removeCurrentSession on the PushRenderer.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #36965 Tue Jul 16 08:13:46 MDT 2013 mircea.toma ICE-9388 Modified BridgeSetup to trigger an update that deregisters the pushIDs whenever push is not enabled anymore. Modified LazyPushManager to use a boolean field to keep track of the push session state.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/BridgeSetup.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/javascript/application.js
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/java/org/icefaces/impl/application/LazyPushManager.java
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Ken Fyten made changes -
              Comment [ A comment with security level 'icesoft-internal-developers' was removed. ]
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11382 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Tyler Johnson
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: