diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/contentprovider/TableContentProviderBase.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/contentprovider/TableContentProviderBase.java
index cec248f61deeb68dba6f9541103e14f40fb19ba9..a8ae42b7cf0141d9c9f24be6514e1d73f6c21927 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/contentprovider/TableContentProviderBase.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/contentprovider/TableContentProviderBase.java
@@ -32,7 +32,7 @@ import org.eclipse.swt.widgets.Table;
* @author ratiu
* @author $Author: hoelzl $
* @version $Rev: 18709 $
- * @ConQAT.Rating YELLOW Hash: A0EBE838F6C48C637BF66BDCC7BECCF6
+ * @ConQAT.Rating GREEN Hash: 9BD7A35B68F2A0C579302785D426E260
*/
public abstract class TableContentProviderBase implements
IStructuredContentProvider {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractDecoratedTextCellDatabindingEditingSupport.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractDecoratedTextCellDatabindingEditingSupport.java
index 86b433ece35eed08ab75d6ac096cbc68f7cd636e..a7a67746c4f781b294d4af0d6b8cf2d480a0f833 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractDecoratedTextCellDatabindingEditingSupport.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractDecoratedTextCellDatabindingEditingSupport.java
@@ -40,7 +40,7 @@ import org.fortiss.tooling.kernel.ui.util.DataBindingUtils;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: F74F00C2F5462687008A52D13BA242A6
+ * @ConQAT.Rating GREEN Hash: 2C4AFF788977B23F89AC273AB9E26FFA
*/
public abstract class AbstractDecoratedTextCellDatabindingEditingSupport extends
AbstractTextCellDatabindingEditingSupport {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractTextCellDatabindingEditingSupport.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractTextCellDatabindingEditingSupport.java
index a22f9ad35f011fe4194b0da3b10ae6981f378d09..2b6c6c5271b4083bae3a1bdb7aca44e54b05fec8 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractTextCellDatabindingEditingSupport.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/databinding/AbstractTextCellDatabindingEditingSupport.java
@@ -38,7 +38,7 @@ import org.fortiss.tooling.kernel.ui.util.DataBindingUtils;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 97B2B23D7EF5DA85AEF4EF792A1C394D
+ * @ConQAT.Rating GREEN Hash: 111846AC9BFA6A0AFD10DED43942929C
*/
public abstract class AbstractTextCellDatabindingEditingSupport extends
EditingSupport {
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 57668728fdb059fcee65bf67e70cf7ed594fb298..b3ecac5b1f75d2bf8ac870f6d6050533183fcd2f 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
@@ -102,7 +102,7 @@ import org.fortiss.tooling.kernel.ui.util.EObjectSelectionUtils;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 29B690AA556FE2BDF57B347269A25E91
+ * @ConQAT.Rating GREEN Hash: 2BE9DAC51D493CE6884BC5780CCBA3BD
*/
public class DiagramEditorBase<T extends EObject> extends GEFEditorBase<T>
implements IPostSelectionProvider, ContextMenuContextProvider {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/command/ReconnectCommand.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/command/ReconnectCommand.java
index ceac76af54d18d93e82ff403ad58952e0a98a7ae..0362b36d117cd63502abaeb28c69442587e2fa9a 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/command/ReconnectCommand.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/command/ReconnectCommand.java
@@ -31,7 +31,7 @@ import org.fortiss.tooling.kernel.service.IConnectionCompositorService;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 24421D2AC104331D9746EEAF90095300
+ * @ConQAT.Rating GREEN Hash: B647C020917AB70601507C6FE044FB31
*/
public class ReconnectCommand extends ConnectCommand {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyInsetEllipse.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyInsetEllipse.java
index bfb328f708d3d7ae2e2b6ec9ae1fb2d307ce69e0..9802a436c6d0f08ce0ab844ea9737e91ad26280b 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyInsetEllipse.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyInsetEllipse.java
@@ -38,7 +38,7 @@ import org.eclipse.swt.graphics.Path;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 7E86CA664B6B561EE22AEC45CC29C1E2
+ * @ConQAT.Rating RED Hash: 84A2AF464E9E0229F704456147BCFAFD
*/
public class PrettyInsetEllipse extends Shape {
@@ -89,6 +89,9 @@ public class PrettyInsetEllipse extends Shape {
EVisualStyle style = EVisualStyle.get();
gfx.pushState();
gfx.setAntialias(SWT.ON);
+ // TODO @review CD: During last iteration most of the glossy look
+ // implementation disappeared. Yet this remained. As seen before the
+ // only effect are ugly borders. I suggest removal.
if (style == EVisualStyle.Glossy) {
// Glossy style.
gfx.setLineCap(SWT.CAP_ROUND);
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyRoundedRectangle.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyRoundedRectangle.java
index f6c6a4b0b52e12a84656f8b0ab06d194c60b9cba..d5b15913b2556cc3fb0feb36d7a76dba28d06138 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyRoundedRectangle.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/figure/PrettyRoundedRectangle.java
@@ -36,7 +36,7 @@ import org.eclipse.swt.graphics.Color;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 34D4F4B5FBB7A3565AE4065D4D7EFED2
+ * @ConQAT.Rating RED Hash: 366867FD2CA94416DC46D9459E4F5F15
*/
public class PrettyRoundedRectangle extends Shape {
@@ -79,6 +79,9 @@ public class PrettyRoundedRectangle extends Shape {
final EVisualStyle style = EVisualStyle.get();
gfx.pushState();
gfx.setAntialias(SWT.ON);
+ // TODO @review CD: During last iteration most of the glossy look
+ // implementation disappeared. Yet this remained. As seen before the
+ // only effect are ugly borders. I suggest removal.
if (style == EVisualStyle.Glossy) {
// Glossy style.
gfx.setLineCap(SWT.CAP_ROUND);
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/DiagramLayoutEditPolicy.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/DiagramLayoutEditPolicy.java
index 8c2bda19fc96f5204d91a51596b2e3443bc6e8a7..94c216f8af21ef220693c3b9a80020cb3a7cab7c 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/DiagramLayoutEditPolicy.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/DiagramLayoutEditPolicy.java
@@ -39,7 +39,7 @@ import org.fortiss.tooling.base.ui.editpart.command.MoveWithinDiagramCommand;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: DC3E049397CBC5F1BA2B9181C1B6C9B7
+ * @ConQAT.Rating GREEN Hash: C38D33442463E6AF17EEE984600F870A
*/
public class DiagramLayoutEditPolicy extends RelayCreateCommandEditPolicyBase {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/ElementLayoutEditPolicy.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/ElementLayoutEditPolicy.java
index 6904920a1fef9a7f67e8b1d2ab8ac75d4121a2cb..84f12d7c5f706b1385289fea73126a62f9c525de 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/ElementLayoutEditPolicy.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/ElementLayoutEditPolicy.java
@@ -39,7 +39,7 @@ import org.fortiss.tooling.kernel.model.INamedCommentedElement;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: F419FCA55F52F78F6EB0C7D80AF32627
+ * @ConQAT.Rating GREEN Hash: 2002CCBC71CE58099D0C1BACCB5D4CE6
*/
public class ElementLayoutEditPolicy<T extends ILayoutedModelElement & INamedCommentedElement>
extends RelayCreateCommandEditPolicyBase {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/RelayCreateCommandEditPolicyBase.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/RelayCreateCommandEditPolicyBase.java
index 9fb6005f0fdda75405d562fea9db4de008f3e5cb..51ead6f2f30f33daed29184b2961670754a636d6 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/RelayCreateCommandEditPolicyBase.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/editpart/policy/RelayCreateCommandEditPolicyBase.java
@@ -43,7 +43,7 @@ import org.fortiss.tooling.kernel.ui.dnd.ElementCompositionSource;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 66E79A7E8FC55B782B9141E8473B8A7B
+ * @ConQAT.Rating GREEN Hash: C903DB73D3ADA8738234F94935C0CDDE
*/
public abstract class RelayCreateCommandEditPolicyBase extends LayoutEditPolicy {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/properties/view/PropertySectionWithCopyPasteBase.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/properties/view/PropertySectionWithCopyPasteBase.java
index a59d9b1ca9cf4391f28a49fafdec0e26a11d81d8..f7205c067c00e5bc8110ad9973aa4a272425122e 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/properties/view/PropertySectionWithCopyPasteBase.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/properties/view/PropertySectionWithCopyPasteBase.java
@@ -32,7 +32,7 @@ import org.fortiss.tooling.kernel.ui.extension.base.PropertySectionBase;
* @author ratiu
* @author $Author: hoelzl $
* @version $Rev: 18709 $
- * @ConQAT.Rating YELLOW Hash: 28164A1500812F5F071D29F4ECBD38B9
+ * @ConQAT.Rating GREEN Hash: 6428F3826328029F86C7993F10E23746
*/
public abstract class PropertySectionWithCopyPasteBase<T extends IModelElementSpecification>
extends PropertySectionBase {
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/CheckBoxLabelProvider.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/CheckBoxLabelProvider.java
index 6657d3d78e6fed49f3f94e80b36cf5d67f23ed2b..e763bf2ed508e153e4ad21c4f6c9b6f5787fef4f 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/CheckBoxLabelProvider.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/CheckBoxLabelProvider.java
@@ -38,7 +38,7 @@ import org.eclipse.swt.widgets.Shell;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: 1879A8B4DD69C9B0CA8F5E9C62920F71
+ * @ConQAT.Rating GREEN Hash: C473EA62A46BAA323662C9046C2921B5
*/
// Using drawn images may seem strange, but internet sources could not provide
// any other solution to having checkbox in cell editors
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/IconNameLabelProvider.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/IconNameLabelProvider.java
index 45785f5c18f73c8c2e0e759c497ded31877d9c2c..f633665cb5023856e05e0f2b84eb66f3b24734ed 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/IconNameLabelProvider.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/tablecell/IconNameLabelProvider.java
@@ -35,7 +35,7 @@ import org.fortiss.tooling.kernel.ui.service.IModelElementHandlerService;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: CA5333015D50B981971B61CD6B9D5B89
+ * @ConQAT.Rating GREEN Hash: C7F3355267F72F318C897785861BF6AB
*/
public class IconNameLabelProvider implements IStyledLabelProvider {
/** {@inheritDoc} */
diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/ActionUtils.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/ActionUtils.java
index 1ec2dd5221bedad8c9c820f4936008e68296a529..4cc2b6c5369dc4aa6e051813ef15130a9b019ded 100644
--- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/ActionUtils.java
+++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/ActionUtils.java
@@ -17,9 +17,10 @@ $Id$
+--------------------------------------------------------------------------*/
package org.fortiss.tooling.base.ui.utils;
+import static org.junit.Assert.assertNotNull;
+
import org.eclipse.jface.action.Action;
import org.eclipse.jface.resource.ImageDescriptor;
-import org.junit.Assert;
/**
* Utility methods for handling JFace {@link Action}s.
@@ -27,7 +28,7 @@ import org.junit.Assert;
* @author hoelzl
* @author $Author$
* @version $Rev$
- * @ConQAT.Rating YELLOW Hash: C8DACEDA16494E5C0312EFC88604471D
+ * @ConQAT.Rating GREEN Hash: 17F433D987A785A1914ADD5B8B899795
*/
public class ActionUtils {
@@ -51,8 +52,8 @@ public class ActionUtils {
action.setToolTipText(toolTip);
action.setEnabled(true);
- Assert.assertNotNull(icon);
- Assert.assertNotNull(disabledIcon);
+ assertNotNull(icon);
+ assertNotNull(disabledIcon);
action.setImageDescriptor(icon);
action.setDisabledImageDescriptor(disabledIcon);