diff options
author | Benoit Maggi | 2014-05-20 09:21:55 +0000 |
---|---|---|
committer | Benoit Maggi | 2014-05-20 09:21:55 +0000 |
commit | 9bf6e07174905891449035c7291365d12a073b10 (patch) | |
tree | 7f3a1e7c1b42b4ca369eb6e4eae7372fcd7cdb2d /plugins/sysml/diagram/org.eclipse.papyrus.sysml.diagram.common | |
parent | b0d593ed716741e7957128c79633d331db29eeed (diff) | |
download | org.eclipse.papyrus-9bf6e07174905891449035c7291365d12a073b10.tar.gz org.eclipse.papyrus-9bf6e07174905891449035c7291365d12a073b10.tar.xz org.eclipse.papyrus-9bf6e07174905891449035c7291365d12a073b10.zip |
Bug 435197 - [Model Explorer] Copy/paste generates NPE on SysML::Blocks
- patch the NullPointer in PartPasteStrategy
- migrate sysml.modelexplorer.test from fragment to plugin in order to
use the new AbstractEditorTest of junit.utils
- add test for non regression
Change-Id: I73e824848f39ac09475c9e95932822fed2136fd4
Signed-off-by: Benoit Maggi <benoit.maggi@cea.fr>
Diffstat (limited to 'plugins/sysml/diagram/org.eclipse.papyrus.sysml.diagram.common')
-rw-r--r-- | plugins/sysml/diagram/org.eclipse.papyrus.sysml.diagram.common/src/org/eclipse/papyrus/sysml/diagram/common/strategy/paste/PartPasteStrategy.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/plugins/sysml/diagram/org.eclipse.papyrus.sysml.diagram.common/src/org/eclipse/papyrus/sysml/diagram/common/strategy/paste/PartPasteStrategy.java b/plugins/sysml/diagram/org.eclipse.papyrus.sysml.diagram.common/src/org/eclipse/papyrus/sysml/diagram/common/strategy/paste/PartPasteStrategy.java index ccc8fbaa5ed..d58723a50da 100644 --- a/plugins/sysml/diagram/org.eclipse.papyrus.sysml.diagram.common/src/org/eclipse/papyrus/sysml/diagram/common/strategy/paste/PartPasteStrategy.java +++ b/plugins/sysml/diagram/org.eclipse.papyrus.sysml.diagram.common/src/org/eclipse/papyrus/sysml/diagram/common/strategy/paste/PartPasteStrategy.java @@ -170,8 +170,10 @@ public class PartPasteStrategy implements IPasteStrategy { PartBlockAdditionalData partBlockAdditionalData = (PartBlockAdditionalData)additionnalData; for(Property property : allAttributes) { Association duplicatedAssociation = partBlockAdditionalData.getDuplicatedAssociationByPropertyName(property.getName()); - restoreAssociationPartContext(classifier, property, duplicatedAssociation); - listDuplicatedAssociation.add(duplicatedAssociation); + if (duplicatedAssociation != null){ + restoreAssociationPartContext(classifier, property, duplicatedAssociation); + listDuplicatedAssociation.add(duplicatedAssociation); + } } associationContainer = targetOwner; } |