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 #3136 (closed)

Opened 9 years ago

Closed 9 years ago

Bug: changePermissions does not check for unchanged permissions

Reported by: jamoore Owned by: jamoore
Priority: blocker Milestone: OMERO-Beta4.2.1
Component: Performance Version: n.a.
Keywords: n.a. Cc: atarkowska
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2010-10-28 (18)

Description

As mentioned in a commit for #2873, the method IAdmin.updateGroup expects group.details.permissions to be null otherwise it automatically calls changePermissions. This is ok, except that it does the heavy lifting even when the permissions are the same.

Change History (2)

comment:1 Changed 9 years ago by jmoore

  • Status changed from new to assigned

comment:2 Changed 9 years ago by jmoore

  • Remaining Time changed from 0.1 to 0
  • Resolution set to fixed
  • Status changed from assigned to closed

(In [8359]) Checking Permissions.sameRights from IAdmin.changePermissions(Fix #3136)

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.79521 sec.)

We're Hiring!