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 5e971a367721a18d80e854944f34a4c59d9fb52d..fc2f7f77e71f26d8dd87719bc5ce2d53c8e17a77 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
@@ -46,7 +46,7 @@ import org.fortiss.tooling.kernel.ui.dnd.ConnectionCompositionSource;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 354302A4CE71F645B69D34B798AD66F3
+ * @ConQAT.Rating GREEN Hash: B59CA1269FD90295352657E58548197A
*/
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 b891296cac9c0523ecbbfeb57d56587783929c9e..21e55b3bc7eeff5eb631445798bf543a22b931bc 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
@@ -39,7 +39,7 @@ import org.fortiss.tooling.kernel.extension.data.IConnectionCompositionContext;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: FF0ECBEE0FEA5E701F5C9CB7AB0D3B06
+ * @ConQAT.Rating GREEN Hash: 1EAAF7FF6B88FDF5782B65149233492A
*/
public class ConnectionDragContext implements IConnectionCompositionContext {
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 48b27e4ab1670dc7ab031058f0dc6eeec321c972..3126cc7532dbf4828347ecfb47bfff808799fe26 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
@@ -47,7 +47,7 @@ import org.fortiss.tooling.kernel.extension.data.IConnectionCompositionContext;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: ED316779B97B25D6691EB4EF09E9A5DA
+ * @ConQAT.Rating GREEN Hash: 21F730770C8506BE3F3A014923B502AF
*/
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 306284cffc9e92616e5e54300232d54bee9b0ed5..61242c8984230f70409a09f3f6a22bc44f99953b 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
@@ -46,7 +46,7 @@ import org.fortiss.tooling.kernel.ui.dnd.ElementCompositionSource;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 6C2E7A710D0DEC0A1524C999D70FAE86
+ * @ConQAT.Rating GREEN Hash: B1E79AD54A1B385D6BFD5058759AF9A9
*/
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 13ce4a8b7ae6dd84bb78d034f4f49542324229ec..2c26873a9bac3b9dbe26319f42dd08fcd2e21fe1 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
@@ -43,7 +43,7 @@ import org.fortiss.tooling.kernel.extension.data.IElementCompositionContext;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 01605EE831D52FCF0C2849835AC66C26
+ * @ConQAT.Rating GREEN Hash: 9ED47424D63579597B8F2874EB085624
*/
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 b7153fd4bd9d63dcf69d0c301ddee5da0bddd52a..617bf8e3dc02babfc2a3827cbd0978fdbbdd742f 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
@@ -18,6 +18,7 @@ $Id$
package org.fortiss.tooling.base.ui.dnd.jface;
import static org.fortiss.tooling.kernel.ui.util.DragAndDropUtils.extractDroppedEObject;
+import static org.fortiss.tooling.kernel.ui.util.DragAndDropUtils.getCompositionServiceTransfers;
import org.eclipse.emf.ecore.EObject;
import org.eclipse.jface.viewers.TreeViewer;
@@ -45,7 +46,7 @@ import org.fortiss.tooling.kernel.utils.ModelUtils;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 30E98158032F24051E401A044E775F4D
+ * @ConQAT.Rating GREEN Hash: C92271848C93CF412A754D2801FF3BC3
*/
public class ViewerElementCompositionDropAdapter extends ViewerDropAdapter {
@@ -113,6 +114,6 @@ public class ViewerElementCompositionDropAdapter extends ViewerDropAdapter {
/** Returns the preferred transfer for the DND operation. */
public Transfer[] getPreferredTransfers() {
- return DragAndDropUtils.getCompositionServiceTransfers();
+ return getCompositionServiceTransfers();
}
}
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/DiagramEditorBase.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/DiagramEditorBase.java
index fc5b6c7812b6c11795777f92ec47a4ed09781c0c..c1bfd88637a35f31ce5148613efbad8052c7a395 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/DiagramEditorBase.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/DiagramEditorBase.java
@@ -17,6 +17,10 @@ $Id$
+--------------------------------------------------------------------------*/
package org.fortiss.tooling.base.ui.editor;
+import static org.conqat.ide.commons.gef.editpart.EditPartUtils.fullRefresh;
+import static org.eclipse.gef.editparts.ZoomManager.FIT_ALL;
+import static org.eclipse.gef.editparts.ZoomManager.FIT_HEIGHT;
+import static org.eclipse.gef.editparts.ZoomManager.FIT_WIDTH;
import static org.fortiss.tooling.base.layout.DefaultLayoutConstants.DEFAULT_GRID_ORIGIN_X;
import static org.fortiss.tooling.base.layout.DefaultLayoutConstants.DEFAULT_GRID_ORIGIN_Y;
import static org.fortiss.tooling.base.layout.DefaultLayoutConstants.DEFAULT_GRID_SIZE;
@@ -28,7 +32,6 @@ import java.util.HashSet;
import java.util.List;
import java.util.Set;
-import org.conqat.ide.commons.gef.editpart.EditPartUtils;
import org.eclipse.draw2d.ColorConstants;
import org.eclipse.draw2d.IFigure;
import org.eclipse.draw2d.geometry.Dimension;
@@ -99,7 +102,7 @@ import org.fortiss.tooling.kernel.ui.util.EObjectSelectionUtils;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 7B15AF8EF35DA949FB4FCC3345C5988B
+ * @ConQAT.Rating RED Hash: 7787CCD0721B1100ED0F87095D599DD1
*/
public class DiagramEditorBase<T extends EObject> extends GEFEditorBase<T>
implements IPostSelectionProvider, ContextMenuContextProvider {
@@ -250,7 +253,7 @@ public class DiagramEditorBase<T extends EObject> extends GEFEditorBase<T>
@Override
public void propertyChange(PropertyChangeEvent event) {
if (event.getProperty().equals(EVisualStyle.ID)) {
- EditPartUtils.fullRefresh(viewer.getRootEditPart());
+ fullRefresh(viewer.getRootEditPart());
}
}
};
@@ -328,8 +331,7 @@ public class DiagramEditorBase<T extends EObject> extends GEFEditorBase<T>
* Default zoom level contributions.
*/
private static final List<String> defaultZoomLevelContribution = Arrays
- .asList(new String[] { ZoomManager.FIT_ALL, ZoomManager.FIT_WIDTH,
- ZoomManager.FIT_HEIGHT });
+ .asList(new String[] { FIT_ALL, FIT_WIDTH, FIT_HEIGHT });
/** Registers global actions (including zoom action). */
@Override
@@ -473,6 +475,11 @@ public class DiagramEditorBase<T extends EObject> extends GEFEditorBase<T>
* Highlights a figure by setting its color to be RED. This implementation
* can be changed by more specific editors.
*/
+
+ // TODO @review CD: unsafe code
+ // IMO double invocation of highlight(true) overwrites the normal color in
+ // the map and makes de-highlighting impossible
+
protected void setHighlight(IFigure fig, boolean highlighted) {
if (highlighted) {
highlightedFigures2NormalColor.put(fig, fig.getForegroundColor());
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/FormsEditorBase.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/FormsEditorBase.java
index a54f68dd2816b37b66ea2e394d4cfa45f2474537..a05914a07b36a79100a5ce858eeb407186f11d82 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/FormsEditorBase.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/FormsEditorBase.java
@@ -40,7 +40,7 @@ import org.fortiss.tooling.kernel.ui.extension.base.EditorBase;
* @author ratiu
* @author $Author: hoelzl $
* @version $Rev: 18709 $
- * @ConQAT.Rating YELLOW Hash: 77237937DC7CBA4AA9BD8B671E8A9926
+ * @ConQAT.Rating GREEN Hash: BD71C31A22A4525094D90909D612FEB1
*/
public abstract class FormsEditorBase<T extends EObject> extends EditorBase<T> {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/GEFEditorBase.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/GEFEditorBase.java
index 10ebaa57c1f7c708fadb5766db4a1a41baab6b7d..7ac2e16435f84b7f63e9aad97a537d2a9f7d9b49 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/GEFEditorBase.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editor/GEFEditorBase.java
@@ -39,7 +39,7 @@ import org.fortiss.tooling.kernel.service.IPersistencyService;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 394C38363F275D47D9B146E156A62135
+ * @ConQAT.Rating GREEN Hash: DC9C2F53C31BA1266F540BA4615E67F1
*/
public abstract class GEFEditorBase<T extends EObject> extends
org.fortiss.tooling.kernel.ui.extension.base.EditorBase<T> {