diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionCompositionDropTargetListener.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionCompositionDropTargetListener.java
index 58ac7b506e8bb8957e2594d7c961572e573220b6..b84050e3c569b6f022f3d71d40396a90f500e42b 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionCompositionDropTargetListener.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionCompositionDropTargetListener.java
@@ -25,6 +25,7 @@ import org.fortiss.tooling.base.ui.editpart.policy.RelayCreateCommandEditPolicyB
import org.fortiss.tooling.base.ui.editpart.request.CreateConnectionCompositionRequest;
import org.fortiss.tooling.kernel.service.IConnectionCompositorService;
import org.fortiss.tooling.kernel.ui.dnd.CompositionServiceLocalTransfer;
+import org.fortiss.tooling.kernel.ui.dnd.ConnectionCompositionDropTargetAdapter;
import org.fortiss.tooling.kernel.ui.dnd.ConnectionCompositionSource;
/**
@@ -34,13 +35,18 @@ import org.fortiss.tooling.kernel.ui.dnd.ConnectionCompositionSource;
* handled by the target edit part's {@link RelayCreateCommandEditPolicyBase},
* which completes the DND operation with the target information. It then relays
* the completed DND operation to the {@link IConnectionCompositorService}.
+ * <p>
+ * This class mimics the behavior of the kernel's SWT-based
+ * {@link ConnectionCompositionDropTargetAdapter}, but uses the GEF abstraction
+ * {@link AbstractTransferDropTargetListener} super class.
*
* @see RelayCreateCommandEditPolicyBase
+ * @see ConnectionCompositionDropTargetAdapter
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash:
+ * @ConQAT.Rating YELLOW Hash: 6B5AA2803FE1DC027D2BC94E176D4B37
*/
public class ConnectionCompositionDropTargetListener extends
AbstractTransferDropTargetListener {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDragContext.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDragContext.java
index bf2594268fe95ccd5aac23be3e6f4dc1c5e63887..b891296cac9c0523ecbbfeb57d56587783929c9e 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDragContext.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDragContext.java
@@ -22,17 +22,24 @@ import org.eclipse.gef.EditPart;
import org.fortiss.tooling.kernel.extension.data.IConnectionCompositionContext;
/**
- * A {@link ConnectionDragContext} stores information obtained from user's drag gesture,
- * i.e. the source edit part and the location w.r.t. this edit part and the
- * target edit part and the location w.r.t. this edit part.
+ * A {@link ConnectionDragContext} stores information obtained from user's drag
+ * gesture, i.e. the source edit part and the location w.r.t. this edit part and
+ * the target edit part and the location w.r.t. this edit part.
+ * <p>
+ * This information is usually provided to the connection compositor in order to
+ * provide the UI data necessary to construct layout information based on the
+ * drag gesture positions.
+ * <p>
+ * Note that this context is used for drag operations within a single editor,
+ * e.g. from one displayed edit part to another in the same diagram.
*
- * TODO: the class comments of {@link ConnectionDragContext} and {@link ElementDropContext} are
- * highly different. Can we make them more homogeneous?
+ * @see ConnectionDropContext
+ * @see ElementDropContext
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: 3C9A44958574E4B1F774160841886315
+ * @ConQAT.Rating YELLOW Hash: FF0ECBEE0FEA5E701F5C9CB7AB0D3B06
*/
public class ConnectionDragContext implements IConnectionCompositionContext {
@@ -49,8 +56,8 @@ public class ConnectionDragContext implements IConnectionCompositionContext {
private Point targetPosition;
/** Constructor. */
- public ConnectionDragContext(EditPart source, Point sourcePosition, EditPart target,
- Point targetPosition) {
+ public ConnectionDragContext(EditPart source, Point sourcePosition,
+ EditPart target, Point targetPosition) {
this.source = source;
this.sourcePosition = sourcePosition;
this.target = target;
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDropContext.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDropContext.java
index ac58c505286a0aa28d2c78bc08f66ae6a6627288..48b27e4ab1670dc7ab031058f0dc6eeec321c972 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDropContext.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ConnectionDropContext.java
@@ -23,12 +23,31 @@ import org.fortiss.tooling.base.ui.editpart.DiagramEditPartBase;
import org.fortiss.tooling.kernel.extension.data.IConnectionCompositionContext;
/**
- * TODO
+ * A {@link ConnectionDropContext} stores the target edit part and the relative
+ * position of the user's drag gesture, when the drag source is not in the same
+ * diagram or editor. Depending on the concrete connection compositor this
+ * information may be relevant.
+ * <p>
+ * The editpart can also be a diagram edit part, e.g. the background of some
+ * diagram editor. In this case {@link #isRoot()} returns <code>true</code>.
+ * <p>
+ * Note that this context is usually used during a drag and drop operation
+ * between a JFace viewer source (e.g a tree) and GEF-based graphical editor
+ * when the result of the gesture should be handled by the connection compositor
+ * service.
+ * <P>
+ * Example scenario: the user drags a model element from the navigator onto an
+ * edit part in some editor and drops it there. As a result the dragged element
+ * is connected to the target edit part's model element by reference link in the
+ * model.
+ *
+ * @see ConnectionDragContext
+ * @see ElementDropContext
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: 70B8E48635907E7922445C6DBE918F76
+ * @ConQAT.Rating YELLOW Hash: ED316779B97B25D6691EB4EF09E9A5DA
*/
public class ConnectionDropContext implements IConnectionCompositionContext {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementCompositionDropTargetListener.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementCompositionDropTargetListener.java
index 32fc03fea876023dc0c528f17aeedcb3e2fb5e6c..8ea440a8d9c19f630685264cf72e6b380c105073 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementCompositionDropTargetListener.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementCompositionDropTargetListener.java
@@ -25,6 +25,7 @@ import org.fortiss.tooling.base.ui.editpart.policy.RelayCreateCommandEditPolicyB
import org.fortiss.tooling.base.ui.editpart.request.CreateElementCompositionRequest;
import org.fortiss.tooling.kernel.service.IElementCompositorService;
import org.fortiss.tooling.kernel.ui.dnd.CompositionServiceLocalTransfer;
+import org.fortiss.tooling.kernel.ui.dnd.ElementCompositionDropTargetAdapter;
import org.fortiss.tooling.kernel.ui.dnd.ElementCompositionSource;
/**
@@ -34,13 +35,18 @@ import org.fortiss.tooling.kernel.ui.dnd.ElementCompositionSource;
* target edit part's {@link RelayCreateCommandEditPolicyBase}, which completes
* the DND operation with the target information. It then relays the completed
* DND operation to the {@link IElementCompositorService}.
+ * <p>
+ * This class mimics the behavior of the kernel's SWT-based
+ * {@link ElementCompositionDropTargetAdapter}, but uses the GEF abstraction
+ * {@link AbstractTransferDropTargetListener} super class.
*
* @see RelayCreateCommandEditPolicyBase
+ * @see ElementCompositionDropTargetAdapter
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash:
+ * @ConQAT.Rating YELLOW Hash: D1E8D0E560B9BF7F51F5BB2A7FA1EACA
*/
public class ElementCompositionDropTargetListener extends
AbstractTransferDropTargetListener {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementDropContext.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementDropContext.java
index d657631f570042a6a456261268f77ef8a8adb7fc..13ce4a8b7ae6dd84bb78d034f4f49542324229ec 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementDropContext.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/gef/ElementDropContext.java
@@ -23,20 +23,27 @@ import org.fortiss.tooling.base.ui.editpart.DiagramEditPartBase;
import org.fortiss.tooling.kernel.extension.data.IElementCompositionContext;
/**
- * Composition context for GEF edit parts initiated by a drop gesture. The
- * context content is defined by the editpart that was the drop target and the
- * mouse position.
+ * A {@link ElementDropContext} stores information obtained from user's drop
+ * gesture, i.e. the target edit part and the location w.r.t. this edit part.
+ * Depending on the concrete element compositor this information may be
+ * relevant, e.g., to create proper initial layout information using the user's
+ * gesture position.
* <P>
* The editpart can also be a diagram edit part, e.g. the background of some
- * diagram editor.
+ * diagram editor. In this case {@link #isRoot()} returns <code>true</code>.
+ * <p>
+ * Example scenario: an element of the model library view is dragged onto a
+ * graphical editor. The model element is composed by some element compositor
+ * and the gesture position is used to make the added element appear at the
+ * user's target location.
*
- * TODO: the class comments of {@link ConnectionDragContext} and {@link ElementDropContext} are
- * highly different. Can we make them more homogeneous?
+ * @see ConnectionDragContext
+ * @see ConnectionDropContext
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: 70B8E48635907E7922445C6DBE918F76
+ * @ConQAT.Rating YELLOW Hash: 01605EE831D52FCF0C2849835AC66C26
*/
public class ElementDropContext implements IElementCompositionContext {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/jface/ViewerElementCompositionDropAdapter.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/jface/ViewerElementCompositionDropAdapter.java
index baa3e9b8e5eb0100e7133ea564e13f0d4394e16e..4d60bc67cc221d3ddc0a513bf46b7908df1d69c3 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/jface/ViewerElementCompositionDropAdapter.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/dnd/jface/ViewerElementCompositionDropAdapter.java
@@ -24,6 +24,7 @@ import org.eclipse.jface.viewers.TreeViewer;
import org.eclipse.jface.viewers.Viewer;
import org.eclipse.jface.viewers.ViewerDropAdapter;
import org.eclipse.swt.dnd.DND;
+import org.eclipse.swt.dnd.DropTargetAdapter;
import org.eclipse.swt.dnd.DropTargetEvent;
import org.eclipse.swt.dnd.Transfer;
import org.eclipse.swt.dnd.TransferData;
@@ -34,13 +35,17 @@ import org.fortiss.tooling.kernel.ui.util.DragAndDropUtils;
import org.fortiss.tooling.kernel.utils.ModelUtils;
/**
- * {@link ElementCompositionDropTargetAdapter} implementation for a JFace
- * {@link TreeViewer}.
+ * {@link ViewerDropAdapter} implementation for a JFace {@link TreeViewer}. This
+ * implementation mimics the behavior of SWT-based
+ * {@link ElementCompositionDropTargetAdapter}, but uses the JFace super class
+ * {@link ViewerDropAdapter} instead of SWT's {@link DropTargetAdapter}.
+ *
+ * @see ElementCompositionDropTargetAdapter
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash:
+ * @ConQAT.Rating YELLOW Hash: 571453E48E7AA8DE7879B78664C0E805
*/
public class ViewerElementCompositionDropAdapter extends ViewerDropAdapter {
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/CompositionServiceLocalTransfer.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/CompositionServiceLocalTransfer.java
index 9232d59cd191b448d651d6f6b8ac6c47a50123b3..7df8e35330221f2b9861bd0deba25d17b53fd938 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/CompositionServiceLocalTransfer.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/CompositionServiceLocalTransfer.java
@@ -28,16 +28,14 @@ import org.fortiss.tooling.kernel.service.IElementCompositorService;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: E8FE5EC5C36C0A8456AEBC887D735DDF
+ * @ConQAT.Rating YELLOW Hash: 4B64D3A6F78734C6CDEC152437300DC0
*/
public class CompositionServiceLocalTransfer extends SimpleObjectTransfer {
/** Singleton instance. */
private static final CompositionServiceLocalTransfer INSTANCE = new CompositionServiceLocalTransfer();
/** Transfer type name. */
- // TODO @review CD: Why does it need such a complex name? Super class
- // already handles timing .
- private static final String TYPE_NAME = "Fortiss kernel composition transfer"
- + System.currentTimeMillis() + ":" + INSTANCE.hashCode();
+ private static final String TYPE_NAME = CompositionServiceLocalTransfer.class
+ .getName();
/** Transfer type ID. */
private static final int TYPEID = registerType(TYPE_NAME);
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDragSourceAdapter.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDragSourceAdapter.java
index 83154e58cbb507ed78d18335791d0486ba32921d..ffea60ae542ef9e6b42d5df3a4fc68e6616c9bf9 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDragSourceAdapter.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDragSourceAdapter.java
@@ -28,11 +28,14 @@ import org.eclipse.swt.dnd.DragSourceAdapter;
* {@link #getParentElement()}, and transfers an
* {@link ConnectionCompositionSource} object via
* {@link CompositionServiceLocalTransfer}.
+ * <p>
+ * This implementation uses the raw SWT DND mechanism.
+ *
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating GREEN Hash: E262C53D9770AA7BD12D6D71CEB4C586
+ * @ConQAT.Rating YELLOW Hash: 5EA6BE06BED62D8859B23FB423E02B1D
*/
public abstract class ConnectionCompositionDragSourceAdapter extends
ElementCompositionDragSourceAdapter {
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDropTargetAdapter.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDropTargetAdapter.java
index 6576073a3e50510042d209758016404338913e98..7eb5b235d915ee7f8abcf1adcc9e989e1c4a1a58 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDropTargetAdapter.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionDropTargetAdapter.java
@@ -28,18 +28,17 @@ import org.fortiss.tooling.kernel.service.IConnectionCompositorService;
* {@link DropTargetAdapter} for completing an EObject connection via a drop
* gesture. If the {@link IConnectionCompositorService} allows the composition
* the {@link DND#DROP_LINK} detail is set as user feedback.
+ * <p>
+ * This implementation uses the raw SWT DND mechanism.
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: 0BFB86B105FFC704BA4114BD223C57B8
+ * @ConQAT.Rating YELLOW Hash: EDF3A3B3D1E14E3840A6A73DA83D3D2C
*/
public abstract class ConnectionCompositionDropTargetAdapter extends
DropTargetAdapter {
- // TODO @review CD: This class sure is useful yet has never been
- // implemented. Has it ever been tested?
-
/** {@inheritDoc} */
@Override
public void drop(DropTargetEvent event) {
@@ -59,14 +58,13 @@ public abstract class ConnectionCompositionDropTargetAdapter extends
event.detail = DND.DROP_NONE;
return;
}
- // TODO @review CD: abort when canConnect succeeds?
if (IConnectionCompositorService.INSTANCE.canConnect(
dragSource.getDesignatedElement(), target,
dragSource.getDesignatedParentElement(), null)) {
- event.detail = DND.DROP_NONE;
+ event.detail = DND.DROP_LINK;
return;
}
- event.detail = DND.DROP_LINK;
+ event.detail = DND.DROP_NONE;
}
/** Returns the current target element. */
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionSource.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionSource.java
index 003aa7228bf4f879c0d8d3107331adf99c4541c6..85320e81d448cc9523c59f0144d481afa753a21a 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionSource.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ConnectionCompositionSource.java
@@ -28,25 +28,20 @@ import org.fortiss.tooling.kernel.service.IConnectionCompositorService;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: D19FC49278D9C24C478FFE4613695AD5
+ * @ConQAT.Rating YELLOW Hash: F146EB39DD806F5A40DED1CA93D74F7A
*/
public class ConnectionCompositionSource extends ElementCompositionSource {
- /** Stores the designated parent element. */
- // TODO @review CD: make final?
- private EObject designatedParentElement;
+ /** Stores the parent element for use in connection composition. */
+ private final EObject parentElement;
/** Constructor. */
public ConnectionCompositionSource(EObject element, EObject parent) {
super(element);
- this.designatedParentElement = parent;
+ this.parentElement = parent;
}
/** Returns the designated parent element. */
public EObject getDesignatedParentElement() {
- return designatedParentElement;
+ return parentElement;
}
-
- // TODO @review CD: This class is a 95% clone of ElementCompositionSource.
- // Moreover derivation does not make sense - neither from the code's nor
- // from the semantics' point of view.
}
\ No newline at end of file
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDragSourceAdapter.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDragSourceAdapter.java
index 5dfd84efe818d6e0b8f33ff48cba47a22fa30845..20fe6684d1eada0ddde9e121562073384d20f573 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDragSourceAdapter.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDragSourceAdapter.java
@@ -28,11 +28,13 @@ import org.eclipse.swt.dnd.Transfer;
* gesture. The implementation takes a source model element provided by
* {@link #getSourceElement()} and transfers an {@link ElementCompositionSource}
* object using {@link CompositionServiceLocalTransfer}.
+ * <p>
+ * This implementation uses the raw SWT DND mechanism.
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating GREEN Hash: 6F0E5F8DCFDB6BDECBC70C85507F2552
+ * @ConQAT.Rating YELLOW Hash: A8B19348C86670BE63B6242EF15F297D
*/
public abstract class ElementCompositionDragSourceAdapter extends
DragSourceAdapter {
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDropTargetAdapter.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDropTargetAdapter.java
index 49bf542d9ebbf58f9af2a380af17e8c7eff6e7fa..08acc568ed73e1b8ad2ab6d7473774ef930a65db 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDropTargetAdapter.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionDropTargetAdapter.java
@@ -28,11 +28,13 @@ import org.fortiss.tooling.kernel.service.IElementCompositorService;
* {@link DropTargetAdapter} for completing an EObject composition via a drop
* gesture. If the {@link IElementCompositorService} allows the composition the
* {@link DND#DROP_COPY} detail is set as user feedback.
+ * <p>
+ * This implementation uses the raw SWT DND mechanism.
*
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: 194552CC8315FA75482EAFCE4852E161
+ * @ConQAT.Rating YELLOW Hash: D04EA768686154B53725E567D3D70A7C
*/
public abstract class ElementCompositionDropTargetAdapter extends
DropTargetAdapter {
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionSource.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionSource.java
index 3d4af3302f7f786dd6f4e433d7e7e36bb3a41ca0..601bbf2f1d0108db426b7c29794c6f22d1b67039 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionSource.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/ElementCompositionSource.java
@@ -28,21 +28,20 @@ import org.fortiss.tooling.kernel.service.IElementCompositorService;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating RED Hash: B601637F018D099C468F5470DBE8A605
+ * @ConQAT.Rating YELLOW Hash: C8BE422248D89555D262BED1F8815C99
*/
public class ElementCompositionSource {
/** Stores the element designated for composition. */
- // TODO @review CD: make final?
- private EObject designatedElement;
+ private EObject element;
/** Constructor. */
public ElementCompositionSource(EObject element) {
- this.designatedElement = element;
+ this.element = element;
}
/** Returns the designated element. */
public EObject getDesignatedElement() {
- return designatedElement;
+ return element;
}
}
diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/package.html b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/package.html
index 0e41b06b786bcbcd1e09166a25877b54f34bfb64..7b97c1774a67f1d68bbb97dceceb075519f1c258 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/package.html
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/dnd/package.html
@@ -1,8 +1,10 @@
<!--
$Id: package.html 835 2011-06-28 07:31:00Z ratiu $
@version $Rev: 835 $
- @ConQAT.Rating RED Hash: 7CD025BCFEDE1B96A0EBDE550A6AD6EE
+ @ConQAT.Rating YELLOW Hash: CC56D6574A670E6853219571BF8D1708
-->
<body>
-Support classes for using the drag and drop mechanism.
+Support classes for using the SWT-based drag and drop mechanism.
+We distinguish between DND operation, which should result in calling the element composition, and those,
+which result in calling the connection composition.
</body>