Task #8581 (closed)
setDefaultGroup on switch group
Reported by: | wmoore | Owned by: | wmoore |
---|---|---|---|
Priority: | minor | Milestone: | OMERO-4.4 |
Component: | Web | Version: | n.a. |
Keywords: | n.a. | Cc: | atarkowska, cxallan |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | 0.0d |
Sprint: | 2012-05-08 (14) |
Description
When we change the 'active_group' we are now just changing the value of conn.CONFIGSERVICE_OPTS?omero.group? for each call. In order to preserve the existing behaviour, we should also change the default group, so we come back to the same group next time we log in etc.
Change History (7)
comment:1 Changed 12 years ago by jmoore
comment:2 Changed 12 years ago by wmoore
But does setDefaultGroup() have any effect on your current context? I'd have thought your default group is only a flag, and only taken into account when you next login?
comment:3 Changed 12 years ago by atarkowska
- Sprint changed from 2012-04-24 (13) to 2012-05-08 (14)
comment:4 Changed 12 years ago by wmoore
- Remaining Time changed from 0.2 to 0
- Resolution set to wontfix
- Status changed from new to closed
Since a user can edit their default group under their settings page, we don't NEED to update each time we switch 'active_group'.
comment:5 Changed 12 years ago by wmoore
- Resolution wontfix deleted
- Status changed from closed to reopened
OK, let's simply setDefaultGroup() on logout, so that the group we were currently viewing 'active_group' is rememberd. This is same logic as Insight.
comment:6 Changed 12 years ago by wmoore
- Resolution set to fixed
- Status changed from reopened to closed
setDefaultGroup on logout. https://github.com/aleksandra-tarkowska/openmicroscopy/commit/e9b2b36ae4fd5b3a8eb648bee027a066b43ca841
Closing...
comment:7 Changed 12 years ago by Will Moore <will@…>
(In [e9b2b36ae4fd5b3a8eb648bee027a066b43ca841/ome.git] on branch develop) setDefaultGroup on logout(). Closes #8581
I disagree. I worry about race conditions with constantly changing the default group.