Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
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 14 years ago

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

  • Status changed from new to assigned

comment:2 Changed 14 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.67854 sec.)

© 2000-2014 University of Dundee & Open Microscopy Environment. Creative Commons Attribution 3.0 Unported License
OME source code is available under the GNU General public license or through commercial license from Glencoe Software

We're Hiring!