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

Opened 13 years ago

Closed 13 years ago

BUG: 'Delete' key in Insight

Reported by: wmoore Owned by: jburel
Priority: critical Milestone: OMERO-Beta4.3.1
Component: Insight Version: n.a.
Keywords: n.a. Cc: jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-07-07 (1)

Description

Seems that it is possible to delete another users' images in Insight (collaborative group) if you use the 'Delete' key.

See:

http://www.openmicroscopy.org.uk/community/viewtopic.php?f=4&t=726&sid=9400f55aa58225a050363935dfc500d7&p=2621

Change History (10)

comment:1 Changed 13 years ago by jmoore

  • Cc jmoore added

comment:2 Changed 13 years ago by wmoore

I see the images didn't actually get deleted, just removed from dataset (see the forum).

comment:3 Changed 13 years ago by jburel

Images are here but linked removed, This should not happen

comment:4 Changed 13 years ago by jburel

  • Remaining Time set to 0.25
  • Status changed from new to accepted

comment:5 Changed 13 years ago by jmoore

We should also be clear in on the permissions page etc. what are client restrictions and what the server enables. Most likely anyone doing this delete from the command line would have similar results, no?

comment:6 Changed 13 years ago by jburel

  • Remaining Time changed from 0.25 to 0.1

comment:7 Changed 13 years ago by jburel

they will have the same result, I don't do anything fancy there. just passing the selected images using queue delete.

comment:8 Changed 13 years ago by jburel <j.burel@…>

(In [c1efa5800499fcbbd0044d7b1771c6a9d8481754/ome.git] on branch develop) Do not allow to delete other users' images using the delete key (see #5973)

comment:9 Changed 13 years ago by jburel

Do we want to use the ticket for documentation section. Problem is now fixed in code

comment:10 Changed 13 years ago by jburel

  • Remaining Time changed from 0.1 to 0
  • Status changed from accepted to closed
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.65740 sec.)

We're Hiring!