Community
Participate
Working Groups
Draging a link of any kind from the model explorer to a diagram cause a stack overflow. This is due to the method org.eclipse.papyrus.diagram.common.editpolicies.CommonDiagramDragDropEditPolicy.dropBinaryLink(CompositeCommand, Element, Element, int, Point, Element) which return its provided command, and then the command is added to itself ... TODO: avoid returning the providing command to avoid such programming mistake.
The returned type of dropBinaryLink() has been changed to void. This will prevent to add the CompoundCommand in itself inadvertently.
Since this class is public and used by all diagrams, changing the signature of method CommonDiagramDragDropEditPolicy.dropBinaryLink is an API change and should change the plugin version. The method was intended to return the compound command for convenience for coders (old calls with return take 1 line instead of 3). In fact, there was only one incorrect call to this method that you yourself committed on 09/06/10 13:36 in commit 1859, in org.eclipse.papyrus.diagram.common.editpolicies.CommonDiagramDragDropEditPolicy.getDropObjectsCommand(DropObjectsRequest) on line 248 cc.add( dropBinaryLink(cc, source, target, linkVISUALID, dropRequest.getLocation(), (Element)droppedObject) ); Could you please either : - 1. restore the old method signature and old correct calls, correct the incorrect call line 248 , make the method's javadoc more explicit to avoid errors, or -2. update the plugin version, send a message to the devel list to inform users the signature has changed, correct the method's javadoc according to the new signature (@return), correct alll modified calls to restore the original labels of commands which have been replaced by "drop Association", correct non modified calls which still use the old signature or mention them in the message sent to the the devel list (I noticed at least activity and parameteric), so that people in charge of these diagram can adapt them. Thank you.
Ok, I restore the old code and open a bug to change the signature later.
API has been reverted. Stack overflow pb still corrected
