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

Opened 17 years ago

Closed 17 years ago

Last modified 13 years ago

IAdmin: updateExperimenter method

Reported by: jburel Owned by: cxallan
Priority: major Milestone: 3.0-Beta2.2
Component: Services Version: n.a.
Keywords: n.a. Cc: jamoore, atarkowska
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: n.a.

Description

Limitation:
The current user cannot update his/her profile.
Need to modify the security check for that method.

Change History (7)

comment:1 Changed 17 years ago by jmoore

So,...currently the method is @RolesAllowed?("system") because don't want most people changing these things, but obviously you need to be able to update your email and institution. Is name (first/middle/last) going to be editable as well?

Technically, there'll need to be a check for self and then a call to runAsAdmin().

And what do we do about editing groups? Description changeable by owner?

comment:2 Changed 17 years ago by jburel

My idea:
Allow user to modify firs/middle/last name, email and institution
and also the default group, if user member of more than one group.

comment:3 Changed 17 years ago by jmoore

ok. That'd mean mostly likely changing:

  • setDefaultGroup to setUserDefaultGroup
  • adding setDefaultGroup(ExperimenterGroup?)
  • adding updateExperimenterInfo(Experimenter) and only the mentioned fields are updated.

comment:4 Changed 17 years ago by cxallan

  • Cc atarkowska added
  • Milestone changed from 3.0-Beta2 to 3.0-Beta3
  • Resolution set to fixed
  • Status changed from new to closed
  • Summary changed from IAdim: updateExperimenter method to IAdmin: updateExperimenter method
  • Version 3.0-M3 deleted

Resolved by #688 this is largely a duplicate ticket.

comment:5 Changed 17 years ago by jmoore

  • Milestone changed from 3.0-Beta3 to 3.0-Beta2.2

comment:6 Changed 13 years ago by Melissa Linkert <melissa@…>

  • Remaining Time set to 0

(In [0bd837bac16aa7dc9a38349d3b74466b5a8a3a29/bioformats.git]) Prevent an NPE when attempting to stitch datasets that aren't stitchable. Closes #646.

(cherry picked from commit b97d88dc3d038616c68aa3bd65d96c0216665460)

comment:7 Changed 13 years ago by Melissa Linkert <melissa@…>

(In [0bd837bac16aa7dc9a38349d3b74466b5a8a3a29/bioformats.git]) Prevent an NPE when attempting to stitch datasets that aren't stitchable. Closes #646.

(cherry picked from commit b97d88dc3d038616c68aa3bd65d96c0216665460)

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

We're Hiring!