diff options
Diffstat (limited to 'plugins/uml/diagram/org.eclipse.papyrus.uml.diagram.activity/src/org/eclipse/papyrus/uml/diagram/activity/edit/parts/ValuePinInCallOpActAsTargetEditPart.java')
-rw-r--r-- | plugins/uml/diagram/org.eclipse.papyrus.uml.diagram.activity/src/org/eclipse/papyrus/uml/diagram/activity/edit/parts/ValuePinInCallOpActAsTargetEditPart.java | 67 |
1 files changed, 31 insertions, 36 deletions
diff --git a/plugins/uml/diagram/org.eclipse.papyrus.uml.diagram.activity/src/org/eclipse/papyrus/uml/diagram/activity/edit/parts/ValuePinInCallOpActAsTargetEditPart.java b/plugins/uml/diagram/org.eclipse.papyrus.uml.diagram.activity/src/org/eclipse/papyrus/uml/diagram/activity/edit/parts/ValuePinInCallOpActAsTargetEditPart.java index e99a3b5adfd..e7d7cf4cf41 100644 --- a/plugins/uml/diagram/org.eclipse.papyrus.uml.diagram.activity/src/org/eclipse/papyrus/uml/diagram/activity/edit/parts/ValuePinInCallOpActAsTargetEditPart.java +++ b/plugins/uml/diagram/org.eclipse.papyrus.uml.diagram.activity/src/org/eclipse/papyrus/uml/diagram/activity/edit/parts/ValuePinInCallOpActAsTargetEditPart.java @@ -59,7 +59,7 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { /** * @generated */ - public static final int VISUAL_ID = 3025; + public static final String VISUAL_ID = "3025"; /** * @generated @@ -86,12 +86,9 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { super.createDefaultEditPolicies(); installEditPolicy(EditPolicy.PRIMARY_DRAG_ROLE, getPrimaryDragEditPolicy()); installEditPolicy(EditPolicyRoles.SEMANTIC_ROLE, new DefaultSemanticEditPolicy()); - installEditPolicy(EditPolicy.GRAPHICAL_NODE_ROLE, new DefaultGraphicalNodeEditPolicy()); - installEditPolicy(EditPolicy.LAYOUT_ROLE, createLayoutEditPolicy()); - installEditPolicy(EditPolicyRoles.OPEN_ROLE, - new OpenDiagramEditPolicy()); + installEditPolicy(EditPolicyRoles.OPEN_ROLE, new OpenDiagramEditPolicy()); installEditPolicy(RequestConstants.REQ_DELETE, new NoDeleteFromDiagramEditPolicy()); installEditPolicy(EditPolicy.LAYOUT_ROLE, new PinLayoutEditPolicy()); // XXX need an SCR to runtime to have another abstract superclass that would let children add reasonable editpolicies @@ -109,18 +106,16 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { * when a node have external node labels, the methods refreshChildren() remove the EditPart corresponding to the Label from the EditPart * Registry. After that, we can't reset the visibility to true (using the Show/Hide Label Action)! */ - if (NotationPackage.eINSTANCE.getView_Visible().equals(event.getFeature())) { + if(NotationPackage.eINSTANCE.getView_Visible().equals(event.getFeature())) { Object notifier = event.getNotifier(); - List<?> modelChildren = ((View) getModel()).getChildren(); - if (false == notifier instanceof Edge - && false == notifier instanceof BasicCompartment) { - if (modelChildren.contains(event.getNotifier())) { + List<?> modelChildren = ((View)getModel()).getChildren(); + if(false == notifier instanceof Edge && false == notifier instanceof BasicCompartment) { + if(modelChildren.contains(event.getNotifier())) { return; } } } super.handleNotificationEvent(event); - } /** @@ -131,23 +126,26 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { @Override protected EditPolicy createChildEditPolicy(EditPart child) { - View childView = (View) child.getModel(); - switch (UMLVisualIDRegistry.getVisualID(childView)) { - case ValuePinInCOActAsTargetLabelEditPart.VISUAL_ID: - case ValuePinInCOActAsTargetValueEditPart.VISUAL_ID: - case ValuePinInCOActAsTargetAppliedStereotypeEditPart.VISUAL_ID: - return new BorderItemSelectionEditPolicy() { + View childView = (View)child.getModel(); + String vid = UMLVisualIDRegistry.getVisualID(childView); + if(vid != null) { + switch(vid) { + case ValuePinInCOActAsTargetLabelEditPart.VISUAL_ID: + case ValuePinInCOActAsTargetValueEditPart.VISUAL_ID: + case ValuePinInCOActAsTargetAppliedStereotypeEditPart.VISUAL_ID: + return new BorderItemSelectionEditPolicy() { - @Override - protected List<?> createSelectionHandles() { - MoveHandle mh = new MoveHandle((GraphicalEditPart) getHost()); - mh.setBorder(null); - return Collections.singletonList(mh); - } - }; + @Override + protected List<?> createSelectionHandles() { + MoveHandle mh = new MoveHandle((GraphicalEditPart)getHost()); + mh.setBorder(null); + return Collections.singletonList(mh); + } + }; + } } EditPolicy result = child.getEditPolicy(EditPolicy.PRIMARY_DRAG_ROLE); - if (result == null) { + if(result == null) { result = new NonResizableEditPolicy(); } return result; @@ -178,7 +176,7 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { */ @Override public PinFigure getPrimaryShape() { - return (PinFigure) primaryShape; + return (PinFigure)primaryShape; } /** @@ -186,15 +184,13 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { */ @Override protected void addBorderItem(IFigure borderItemContainer, IBorderItemEditPart borderItemEditPart) { - if (borderItemEditPart instanceof ValuePinInCOActAsTargetLabelEditPart) { + if(borderItemEditPart instanceof ValuePinInCOActAsTargetLabelEditPart) { IBorderItemLocator locator = new ExternalLabelPositionLocator(getMainFigure()); borderItemContainer.add(borderItemEditPart.getFigure(), locator); - } else - if (borderItemEditPart instanceof ValuePinInCOActAsTargetValueEditPart) { + } else if(borderItemEditPart instanceof ValuePinInCOActAsTargetValueEditPart) { IBorderItemLocator locator = new ExternalLabelPositionLocator(getMainFigure()); borderItemContainer.add(borderItemEditPart.getFigure(), locator); - } else - if (borderItemEditPart instanceof ValuePinInCOActAsTargetAppliedStereotypeEditPart) { + } else if(borderItemEditPart instanceof ValuePinInCOActAsTargetAppliedStereotypeEditPart) { IBorderItemLocator locator = new ExternalLabelPositionLocator(getMainFigure()); borderItemContainer.add(borderItemEditPart.getFigure(), locator); } else { @@ -226,7 +222,6 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { figure.add(shape); contentPane = setupContentPane(shape); return figure; - } /** @@ -246,7 +241,7 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { */ @Override public IFigure getContentPane() { - if (contentPane != null) { + if(contentPane != null) { return contentPane; } return super.getContentPane(); @@ -257,7 +252,7 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { */ @Override protected void setForegroundColor(Color color) { - if (primaryShape != null) { + if(primaryShape != null) { primaryShape.setForegroundColor(color); } } @@ -275,8 +270,8 @@ public class ValuePinInCallOpActAsTargetEditPart extends AbstractPinEditPart { */ @Override protected void setLineType(int style) { - if (primaryShape instanceof IPapyrusNodeFigure) { - ((IPapyrusNodeFigure) primaryShape).setLineStyle(style); + if(primaryShape instanceof IPapyrusNodeFigure) { + ((IPapyrusNodeFigure)primaryShape).setLineStyle(style); } } |