Details
-
Type: Bug
-
Status: Closed
-
Priority: 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.
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
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Migration
created issue -
Ken Fyten
made changes -
Field | Original Value | New Value |
---|---|---|
Summary | Placeholder issue | PushRenderer's addCurrentSession() and removeCurrentSession() aren't doing what the method name implies |
Assignee | Jack Van Ooststroom [ jack.van.ooststroom ] | |
Fix Version/s | 3.2 [ 10338 ] | |
Reporter | Migration [ remote ] | Jack Van Ooststroom [ jack.van.ooststroom ] |
Affects Version/s | 3.2.BETA1 [ 10360 ] | |
Environment | Test | ICEfaces |
Description | Placeholder issue |
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. |
Affects | Documentation (User Guide, Ref. Guide, etc.),Compatibility/Configuration [ 10003, 10002 ] |
Ken Fyten
made changes -
Status | Open [ 1 ] | Closed [ 6 ] |
Affects | Documentation (User Guide, Ref. Guide, etc.),Compatibility/Configuration [ 10003, 10002 ] | |
Resolution | Fixed [ 1 ] | |
Security | Private [ 10001 ] |