[Uml-devel] [Bug 56184] Umbrello XMI file format doesn't conform with uml13.dtd

Oliver Kellogg okellogg at users.sourceforge.net
Tue Jan 10 11:34:00 UTC 2006


------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=56184         




------- Additional Comments From okellogg users sourceforge net  2006-01-10 20:26 -------
SVN commit 496548 by okellogg:

UMLRole::saveToXMI(): Change aggregation="shared" to "aggregate".
UMLAssociation::saveToXMI(): Save abstraction as <UML:Abstraction>.
Thanks to Tom Morris for clarifying.
CCBUG:56184
CCMAIL:tfmorris gmail com


 M  +8 -2      association.cpp  
 M  +1 -1      umlrole.cpp  


--- branches/KDE/3.5/kdesdk/umbrello/umbrello/association.cpp #496547:496548
 @ -161,8 +161,7  @
 }
 
 void UMLAssociation::saveToXMI( QDomDocument & qDoc, QDomElement & qElement ) {
-    if (m_AssocType == Uml::at_Generalization ||
-            m_AssocType == Uml::at_Realization) {
+    if (m_AssocType == Uml::at_Generalization) {
         QDomElement assocElement = UMLObject::save("UML:Generalization", qDoc);
         assocElement.setAttribute( "discriminator", "" );
         assocElement.setAttribute( "child", ID2STR(getObjectId(A)) );
 @ -170,6 +169,13  @
         qElement.appendChild( assocElement );
         return;
     }
+    if (m_AssocType == Uml::at_Realization) {
+        QDomElement assocElement = UMLObject::save("UML:Abstraction", qDoc);
+        assocElement.setAttribute( "client", ID2STR(getObjectId(A)) );
+        assocElement.setAttribute( "supplier", ID2STR(getObjectId(B)) );
+        qElement.appendChild( assocElement );
+        return;
+    }
     if (m_AssocType == Uml::at_Dependency) {
         QDomElement assocElement = UMLObject::save("UML:Dependency", qDoc);
         assocElement.setAttribute( "client", ID2STR(getObjectId(A)) );
--- branches/KDE/3.5/kdesdk/umbrello/umbrello/umlrole.cpp #496547:496548
 @ -118,7 +118,7  @
             roleElement.setAttribute("aggregation", "composite");
             break;
         case Uml::at_Aggregation:
-            roleElement.setAttribute("aggregation", "shared");
+            roleElement.setAttribute("aggregation", "aggregate");
             break;
         default:
             roleElement.setAttribute("aggregation", "none");




More information about the umbrello-devel mailing list