Jira-ID: CRNT-26324

JMS Application type: click on Restore default after adding multiple Response throws exception

git-svn-id: http://emeafrazerg/svn/ipp/product/trunk/stardust/ide@59711 8100b5e0-4d52-466c-ae9c-bdeccbdeaf6b

1 files changed, 12 insertions, 8 deletions

diff --git a/modeling/org.eclipse.stardust.modeling.core/src/org/eclipse/stardust/modeling/core/spi/applicationTypes/jms/JmsPropertyPage.java b/modeling/org.eclipse.stardust.modeling.core/src/org/eclipse/stardust/modeling/core/spi/applicationTypes/jms/JmsPropertyPage.java
index e10326c..bd2fb7b 100644
--- a/modeling/org.eclipse.stardust.modeling.core/src/org/eclipse/stardust/modeling/core/spi/applicationTypes/jms/JmsPropertyPage.java
+++ b/modeling/org.eclipse.stardust.modeling.core/src/org/eclipse/stardust/modeling/core/spi/applicationTypes/jms/JmsPropertyPage.java
@@ -81,7 +81,7 @@ public class JmsPropertyPage extends AbstractModelElementPropertyPage
    private Button requestButton;
    private Button responseButton;
    private EObjectLabelProvider labelProvider;
-   
+
    public void dispose()
    {
       requestOutlineSynchronizer.dispose();
@@ -114,7 +114,7 @@ public class JmsPropertyPage extends AbstractModelElementPropertyPage
    {
       if (requestButton.getSelection())
       {
-         
+
          addJmsType(REQUEST_NODE, REQUEST,
               RequestPropertyPage.class.getName(), application);
       }
@@ -242,7 +242,7 @@ public class JmsPropertyPage extends AbstractModelElementPropertyPage
       };
       TableUtil.setLabelProvider(viewer, labelProvider, labelProperties);
       viewer.setContentProvider(new JmsAccessPointsTreeContentProvider());
-      
+
       requestOutlineSynchronizer = new ModelElementsOutlineSynchronizer(
             new DefaultOutlineProvider(this,
                CarnotWorkflowModelPackage.eINSTANCE.getIAccessPointOwner_AccessPoint(),
@@ -254,8 +254,12 @@ public class JmsPropertyPage extends AbstractModelElementPropertyPage
                {
                   public boolean select(Object toTest)
                   {
-                     return AccessPointUtil.isDirectionCompatible(
-                           (AccessPointType) toTest, true);
+                        if (toTest instanceof AccessPointType)
+                        {
+                           return AccessPointUtil.isDirectionCompatible(
+                                 (AccessPointType) toTest, true);
+                        }
+                        return false;
                   }
                })
             {
@@ -280,7 +284,7 @@ public class JmsPropertyPage extends AbstractModelElementPropertyPage
                }
             });
       addModelElementsOutlineSynchronizer(requestOutlineSynchronizer);
-      
+
       responseOutlineSynchronizer = new ModelElementsOutlineSynchronizer(
             new DefaultOutlineProvider(this,
                CarnotWorkflowModelPackage.eINSTANCE.getIAccessPointOwner_AccessPoint(),
@@ -453,13 +457,13 @@ public class JmsPropertyPage extends AbstractModelElementPropertyPage
          factory.getId(), factory.getName(), null,
          isIn ? DirectionType.IN_LITERAL : DirectionType.OUT_LITERAL, !isIn, null,
          ModelUtils.getDataType(application, CarnotConstants.SERIALIZABLE_DATA_ID));
-      
+
       AttributeUtil.setAttribute(ap, CarnotConstants.BROWSABLE_ATT, BOOLEAN_TYPE,
             Boolean.FALSE.toString());
 
       ap.setElementOid(ModelUtils.getElementOid(ap,
             ModelUtils.findContainingModel(application)));
-      
+
       application.getAccessPoint().add(ap);
       if (preselect)
       {