[Uml-devel] kdesdk/umbrello/umbrello

[Oliver Kellogg]

CVS commit by okellogg: 

operator==(): Test the ID.
(Beats me how this equality test could ever work before.)


  M +7 -1      umlwidget.cpp   1.54


--- kdesdk/umbrello/umbrello/umlwidget.cpp  #1.53:1.54
@@ -158,5 +158,11 @@ bool UMLWidget::operator==(const UMLWidg
         }
 
-        if(m_Assocs.count() != other.m_Assocs.count()) {
+        if (getID() != other.getID())
+                return false;
+
+        /* Testing the associations is already an exaggeration, no?
+           The type and ID should uniquely identify an UMLWidget.
+         */
+        if (m_Assocs.count() != other.m_Assocs.count()) {
                 return false;
         }