Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #2265 (closed)

Opened 11 years ago

Closed 11 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 11 years ago by jmoore

"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.)

comment:2 Changed 11 years ago by jmoore

  • Owner changed from jmoore to atarkowska

Ola, passing this back to you.

comment:3 Changed 11 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 11 years ago by jmoore

  • Status changed from new to assigned

comment:5 Changed 11 years ago by atarkowska

  • Priority changed from blocker to major

comment:6 Changed 11 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?

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.100432 sec.)

We're Hiring!