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

Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

Move images into new dataset on chgrp

Reported by: jamoore Owned by: jburel
Priority: major Milestone: OMERO-4.4
Component: Services Version: n.a.
Keywords: n.a. Cc: omero-team@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: n.a.

Description

Suggested chgrp workflow feature from Manda: specify dataset when moving an image to another group so that it's not orphaned.

See: https://www.openmicroscopy.org/community/viewtopic.php?f=6&t=1028

Change History (7)

comment:1 Changed 8 years ago by jmoore

  • Cc team removed

comment:2 Changed 7 years ago by jburel

  • Cc omero-team@… added; python-team@… removed

This should be allowed for any type of objects e.g Dataset to Project, Plate to screen.

comment:3 Changed 7 years ago by jmoore

  • Owner changed from jmoore to jburel

Implemented and pushing to my 2874-chmod branch. Passing off to Jean-Marie for testing in insight. (#8340 looks to be the web ticket)

comment:4 Changed 7 years ago by jmoore <josh@…>

(In [a84d7b127b4f12a3b37b37e8a6bd6295a4f0a9ff/ome.git] on branch develop) Initial DoAll? impl. (See #7805)

comment:5 Changed 7 years ago by jmoore <josh@…>

  • Remaining Time changed from 1.0 to 0
  • Resolution set to fixed
  • Status changed from new to closed

(In [2aa4749727e0b95d38f8e7090ac5098ff9b5eee0/ome.git] on branch develop) Refactor IRequest and Helper for DoAll? (Fix #7805)

In order to have SaveI return internal objects, it was
necessary to rework how HandleI interacts with IRequest
instances. Rather than running IceMapper? within the TX
(which is a big no-no) the raw Hibernate objects are return
via "step(int i)" and then in bulk passed through the AOP
layers wrapping Executor.execute.

IRequest.finish() was previously run within the TX, but
now it has been renamed buildResponse and takes the Object
passed out of the step method. IRequest implementors will
likely want to make use of helper.isLast(step) to know when
execution is finished.

All IRequest implementations were reviewed and refactored
to make use of new Helper methods.

comment:6 Changed 7 years ago by jmoore <josh@…>

(In [a84d7b127b4f12a3b37b37e8a6bd6295a4f0a9ff/ome.git] on branch develop) Initial DoAll? impl. (See #7805)

comment:7 Changed 7 years ago by jmoore <josh@…>

(In [2aa4749727e0b95d38f8e7090ac5098ff9b5eee0/ome.git] on branch develop) Refactor IRequest and Helper for DoAll? (Fix #7805)

In order to have SaveI return internal objects, it was
necessary to rework how HandleI interacts with IRequest
instances. Rather than running IceMapper? within the TX
(which is a big no-no) the raw Hibernate objects are return
via "step(int i)" and then in bulk passed through the AOP
layers wrapping Executor.execute.

IRequest.finish() was previously run within the TX, but
now it has been renamed buildResponse and takes the Object
passed out of the step method. IRequest implementors will
likely want to make use of helper.isLast(step) to know when
execution is finished.

All IRequest implementations were reviewed and refactored
to make use of new Helper methods.

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

We're Hiring!