From: Aron Roberts Date: Wed, 12 Dec 2012 02:42:24 +0000 (-0800) Subject: CSPACE-5727: Revised comment for clarity. X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=80be80399f4e77a2dc7b63b3ed5b652bad926ab7;p=tmp%2Fjakarta-migration.git CSPACE-5727: Revised comment for clarity. --- diff --git a/3rdparty/nuxeo/nuxeo-platform-listener/updateobjectlocationonmove/src/main/java/org/collectionspace/services/listener/UpdateObjectLocationOnMove.java b/3rdparty/nuxeo/nuxeo-platform-listener/updateobjectlocationonmove/src/main/java/org/collectionspace/services/listener/UpdateObjectLocationOnMove.java index dd501d5ca..df55fa9a1 100644 --- a/3rdparty/nuxeo/nuxeo-platform-listener/updateobjectlocationonmove/src/main/java/org/collectionspace/services/listener/UpdateObjectLocationOnMove.java +++ b/3rdparty/nuxeo/nuxeo-platform-listener/updateobjectlocationonmove/src/main/java/org/collectionspace/services/listener/UpdateObjectLocationOnMove.java @@ -110,10 +110,16 @@ public class UpdateObjectLocationOnMove implements EventListener { // Note: currently, all Document lifecycle transitions on // the relevant doctype(s) are handled by this event handler, // not just transitions between 'soft deleted' and active states. + // // We are assuming that we'll want to re-compute current locations - // for related CollectionObjects on any such transitions. + // for related CollectionObjects on any such transitions, as the + // semantics of such transitions are opaque to this event handler, + // since arbitrary workflows can be bound to those doctype(s). // // If we need to filter out some of those lifecycle transitions, + // such as excluding transitions to the 'locked' workflow state; or, + // alternately, if we want to restrict this event handler's + // scope to handle only transitions into the 'soft deleted' state, // we can add additional checks for doing so at this point.