From 80be80399f4e77a2dc7b63b3ed5b652bad926ab7 Mon Sep 17 00:00:00 2001 From: Aron Roberts Date: Tue, 11 Dec 2012 18:42:24 -0800 Subject: [PATCH] CSPACE-5727: Revised comment for clarity. --- .../services/listener/UpdateObjectLocationOnMove.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) 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. -- 2.47.3