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

Opened 13 years ago

Closed 9 years ago

RFE: IDelete - Skipping error message

Reported by: atarkowska Owned by: jamoore
Priority: critical Milestone: 5.1.1
Component: Services Version: 5.1.0
Keywords: n.a. Cc: jburel, jamoore, saloynton
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description (last modified by jburel)

I was importing plate using Insight importer and exidently shut the application in the middle of import. The plate object was created in the datable but non of the wells were rendered correctly. Then I decided to delete Plate and I end up with the error message in status:

Finished 0 error(s) (Skipping delete of LogicalChannel:1289 
Skipping delete of LogicalChannel:1290 Skipping delete of DetectorSettings:19 
Skipping delete of DetectorSettings:19 
Skipping delete of ObjectiveSettings:3 ConstraintViolation: fkimage_experiment_experimentConstraintViolation: fkdetectorsettings_detector_detectorSkipping delete of LogicalChannel:1289 
Skipping delete of LogicalChannel:1290 Skipping delete of DetectorSettings:19 
Skipping delete of DetectorSettings:19 Skipping delete of ObjectiveSettings:3 
ConstraintViolation: fkimage_experiment_experiment
ConstraintViolation: fkdetectorsettings_detector_detector
Skipping delete of LogicalChannel:1289 
Skipping delete of LogicalChannel:1290 Skipping delete of DetectorSettings:19 
Skipping delete of DetectorSettings:19 Skipping delete of ObjectiveSettings:3
 ConstraintViolation: fkimage_experiment_experimentConstraintViolation: 

Change History (12)

comment:1 Changed 13 years ago by atarkowska

  • Component changed from General to Services
  • Description modified (diff)
  • Priority changed from minor to major

comment:2 Changed 13 years ago by atarkowska

comment:3 Changed 13 years ago by jmoore

  • Cc jburel jmoore saloynton added

Ola, passing back to you after the conversation in devteam. I think the best choice until we improve reporting is to not show the users the warnings, but you'll need to confirm with J-M and have the strategy the same in both clients.

comment:4 Changed 13 years ago by jmoore

  • Owner changed from jmoore to atarkowska

comment:5 Changed 13 years ago by jburel

  • Description modified (diff)

comment:6 Changed 13 years ago by jburel

  • Description modified (diff)

Reduce the message to something: "Delete not completed"
but it also means that we will need to have a force delete option so corrupted data can be deleted.
From a user point of view, the import failed so he/she should be able to remove the plate.
Attaching ticket to delete phase II

comment:7 Changed 13 years ago by jmoore

It's a bit more complicated than that, since in this case, nothing has gone wrong. It's only saying that N-1 of N attempts to delete the LogicalChannel failed. The final one passed. So, in this case, there's nothing for the user to do. In the case of a user tries to delete one image that shares with another image a LogicalChannel and so the LogicalChannel really cannot be deleted, then there's no reason to offer a "force" option, since there's nothing we can do about it (other than to delete the second image).

I don't think this is related to the failed import or not.

comment:8 Changed 13 years ago by Aleksandra Tarkowska <A.Tarkowska@…>

(In [f0d726aef66dc9f091aff14c8d3210042524c6ed/ome.git] on branch develop) this replace error/warning server messages by the appropriate string, see #6003

comment:9 Changed 13 years ago by atarkowska

  • Milestone OMERO-Beta4.3.1 deleted
  • Owner atarkowska deleted
  • Sprint 2011-07-07 (1) deleted

comment:10 Changed 12 years ago by jmoore

  • Milestone set to OMERO-Beta4.4
  • Owner set to jmoore
  • Priority changed from major to critical
  • Summary changed from BUG: IDelete - Skipping error message to RFE: IDelete - Skipping error message

Changing to an RFE. I agree that it's not ideal but:

  1. it's not harming the user (though it certainly may be confusing them)
  2. it will require significant reworking to detect.

If any of you disagree and can think of something specific server-side I can do to make this better now, let me know.

comment:11 Changed 12 years ago by jmoore

  • Milestone changed from OMERO-Beta4.4 to OMERO-Beta4.4.1

Won't be making any changes to this for the moment.

comment:12 Changed 9 years ago by jamoore

  • Milestone changed from 5.x to 5.1.1
  • Resolution set to fixed
  • Status changed from new to closed
  • Version set to 5.1.0

Delete1 implementation is now all but completely gone.

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

We're Hiring!