431503: [Table 2] The Property view of the table generate an exception Widget Is Disposed when we try to add Post Action...

1 files changed, 9 insertions, 8 deletions

diff --git a/plugins/infra/widget/org.eclipse.papyrus.infra.widgets/src/org/eclipse/papyrus/infra/widgets/editors/MultipleValueEditor.java b/plugins/infra/widget/org.eclipse.papyrus.infra.widgets/src/org/eclipse/papyrus/infra/widgets/editors/MultipleValueEditor.java
index b0694e00bb4..4b23273caa4 100644
--- a/plugins/infra/widget/org.eclipse.papyrus.infra.widgets/src/org/eclipse/papyrus/infra/widgets/editors/MultipleValueEditor.java
+++ b/plugins/infra/widget/org.eclipse.papyrus.infra.widgets/src/org/eclipse/papyrus/infra/widgets/editors/MultipleValueEditor.java
@@ -412,13 +412,10 @@ public class MultipleValueEditor extends AbstractListEditor implements Selection
 					@Override

 					public void run() {

 						Object newElement = referenceFactory.createObject(MultipleValueEditor.this, context);

-						if(newElement == null) {

-							// Cancel the operation

-							throw new OperationCanceledException();

+						if(newElement != null) {

+							modelProperty.add(newElement);

+							commit();

 						}

-						

-						modelProperty.add(newElement);

-						commit();

 					}

 				}, NLS.bind(Messages.MultipleValueEditor_addOperation, labelText));

 			}

@@ -469,7 +466,9 @@ public class MultipleValueEditor extends AbstractListEditor implements Selection
 	@Override

 	protected void commit() {

 		super.commit();

-		treeViewer.refresh();

+		if(!isDisposed()){

+			treeViewer.refresh();

+		}

 	}

 

 	/**

@@ -601,7 +600,9 @@ public class MultipleValueEditor extends AbstractListEditor implements Selection
 	 * @param event

 	 */

 	public void handleChange(ChangeEvent event) {

-		treeViewer.refresh();

+		if(!isDisposed()) {

+			treeViewer.refresh();

+		}

 	}

 

 	@Override