svn commit: r12911 - trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java

Author: tfmorris
Date: 2007-06-27 11:06:22-0700
New Revision: 12911

Modified:
   trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java

Log:
Add TODO - no functional change

Modified: trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java?view=diff&rev=12911&p1=trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java&p2=trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java&r1=12910&r2=12911
==============================================================================
--- trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java	(original)
+++ trunk/src_new/org/argouml/uml/ui/UMLModelElementListModel2.java	2007-06-27 11:06:22-0700
@@ -416,6 +416,8 @@
     protected boolean isValidEvent(PropertyChangeEvent e) {
         boolean valid = false;
         if (!(getChangedElement(e) instanceof Collection)) {
+            // TODO: Considering all delete events to be valid like below
+            // is going to cause lots of unecessary work and some problems
             if ((e.getNewValue() == null && e.getOldValue() != null)
                     // Don't test changed element if it was deleted
                     || isValidElement(getChangedElement(e))) {