svn commit: r16971 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigAssociation.java

Author: dthompson
Date: 2009-03-26 13:35:41-0700
New Revision: 16971

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigAssociation.java

Log:
Improved comment and added an assert (previously not committed during beta period).

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigAssociation.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigAssociation.java?view=diff&pathrev=16971&r1=16970&r2=16971
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigAssociation.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigAssociation.java	2009-03-26 13:35:41-0700
@@ -683,7 +683,8 @@
 
 /**
  * A Fig representing the association end role of some model element.
- * 
+ * This class is designed as a composite part and should always be 
+ * part of a FigGroup.
  * @author Bob Tarling
  */
 class FigRole extends FigSingleLineTextWithNotation {
@@ -724,12 +725,14 @@
      * Property change listener to recalculate bounds of enclosing
      * group whenever any properties of the FigRole get changed.
      * This is only really needed for the name, see issue 5621.
+     * 
      * @param pce The property change event to process.
      * @see org.argouml.uml.diagram.ui.FigSingleLineTextWithNotation#propertyChange(java.beans.PropertyChangeEvent)
      */
     @Override
     public void propertyChange(PropertyChangeEvent pce) {
         super.propertyChange(pce);
+        assert(getGroup() != null);
         this.getGroup().calcBounds();
     }

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

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