Task #2265 (closed)
Opened 14 years ago
Closed 14 years ago
BUG: getEventContext does not recognize read-only group
Reported by: | atarkowska | Owned by: | jamoore |
---|---|---|---|
Priority: | major | Milestone: | OMERO-Beta4.2 |
Component: | Services | Version: | n.a. |
Keywords: | n.a. | Cc: | |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | 0.0d |
Sprint: | 2010-04-16 (7) |
Description
getEventContext returns isReadOnly = True regardless of permissions set on group.
object #0 (::omero::sys::EventContext) { ... groupId = 5 groupName = test-collaborative isAdmin = False isReadOnly = True ... } Permissions: rwrw--
Change History (6)
comment:1 Changed 14 years ago by jmoore
comment:2 Changed 14 years ago by jmoore
- Owner changed from jmoore to atarkowska
Ola, passing this back to you.
comment:3 Changed 14 years ago by jmoore
- Owner changed from atarkowska to jmoore
Ola's comment got lost. From devteam: "rename to isSessionReadOnly or sessionMode". I looked into the implementation, and isReadOnly doesn't need to exposed to clients since it only represents the transactional state of the single method (not of the whole session). Removing.
comment:4 Changed 14 years ago by jmoore
- Status changed from new to assigned
comment:5 Changed 14 years ago by atarkowska
- Priority changed from blocker to major
comment:6 Changed 14 years ago by jmoore
- Remaining Time set to 0
- Resolution set to fixed
- Status changed from assigned to closed
(In [6742]) fix #2265 - Removing isReadOnly from EventContext?
"isReadOnly" doesn't refer to the group permissions (it was there before #1434), but to rather the session is read-only (shares for example). When we add documention to the *.ice files, I'll make that clearer; do you think the value should be renamed? (If not, you can close thise.)