svn commit: r19396 - trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/FacadeEUMLImpl.java

Author: mvw
Date: 2011-05-10 23:23:38-0700
New Revision: 19396

Modified:
   trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/FacadeEUMLImpl.java

Log:
Fix for issue 6253: UML2: deletion of a state --> BasicIndexOutOfBoundsException.

Modified: trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/FacadeEUMLImpl.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/FacadeEUMLImpl.java?view=diff&pathrev=19396&r1=19395&r2=19396
==============================================================================
--- trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/FacadeEUMLImpl.java	(original)
+++ trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/FacadeEUMLImpl.java	2011-05-10 23:23:38-0700
@@ -12,6 +12,7 @@
  *    Bob Tarling
  *    Thomas Neustupny
  *    Laurent Braud
+ *    Michiel van der Wulp
  *****************************************************************************/
 
 package org.argouml.model.euml;
@@ -1637,9 +1638,12 @@
         if (!(handle instanceof Transition)) {
             throw new IllegalArgumentException();
         }
-        // TODO: Transitions can have multiple Triggers now?
-        // Need API change to handle - tfm
-        return ((Transition) handle).getTriggers().get(0);
+        // Transitions can have multiple Triggers now
+        List<Trigger> trs = ((Transition) handle).getTriggers();
+        if (trs.isEmpty()) {
+            return null;
+        }
+        return trs.get(0);
     }
 
     public List<Trigger> getTriggers(Object handle) {

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=2733920

To unsubscribe from this discussion, e-mail: [[email protected]].