ICEfaces
  1. ICEfaces
  2. ICE-8632

PushRenderer's addCurrentSession() and removeCurrentSession() aren't doing what the method name implies

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.2.BETA1
          • Fix Version/s: 3.2
          • Component/s: None
          • Labels:
            None
          • Environment:
            ICEfaces

            Description

            The PushRenderer methods addCurrentSession(String) and removeCurrentSession() aren't seem to be doing what the method name seems to imply.

            Basically, when addCurrentSession(String) is invoked it does not add all views associated with the current session to the group, instead it will start adding views to the group created after invocation of this method associated with the current session. On the other hand, when removeCurrentSession(String) is invoked it does not remove all views associated with the current session from the group, instead it will stop adding views to the group created after invocation of this method associated with the current session.

            We should either ensure the implementation of these methods are conform the method names, or change the method names.

              Activity

              Repository Revision Date User Message
              ICEsoft Public SVN Repository #31268 Wed Oct 03 09:52:26 MDT 2012 jack.van.ooststroom Fixed JIRA ICE-8632 : PushRenderer's addCurrentSession() and removeCurrentSession() aren't doing what the method name implies; clean up
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/java/org/icefaces/application/PushRenderer.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/java/org/icefaces/impl/push/SessionViewManager.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #31267 Wed Oct 03 08:37:48 MDT 2012 jack.van.ooststroom Fixed JIRA ICE-8632 : PushRenderer's addCurrentSession() and removeCurrentSession() aren't doing what the method name implies
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/java/org/icefaces/impl/push/SessionViewManager.java

                People

                • Assignee:
                  Jack Van Ooststroom
                  Reporter:
                  Jack Van Ooststroom
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: